Adding signed types to slct_impl, fixes http://www.gpgpu-sim.org/bugs/show_bug.cgi?id=78

review: 123001, LGTM: 2

[git-p4: depot-paths = "//depot/gpgpu_sim_research/fermi/distribution/": change = 17692]

2 files changed, 7 insertions, 1 deletions

diff --git a/CHANGES b/CHANGES
index 35e04da..ebdfed1 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,4 +1,7 @@
 LOG:
+Version 3.2.3 versus 3.2.2
+- Fixed bug #78, added signed types to implementation of slct instruction.
+
 Version 3.2.2 versus 3.2.1
 - Added NVIDIA Quadro FX5600 GPGPU-Sim and GPUWattch configuration files.
 - Added cache_stats class to record all memory accesses and access outcomes 
diff --git a/src/cuda-sim/instructions.cc b/src/cuda-sim/instructions.cc
index 89212f2..1998765 100644
--- a/src/cuda-sim/instructions.cc
+++ b/src/cuda-sim/instructions.cc
@@ -3498,13 +3498,16 @@ void slct_impl( const ptx_instruction *pI, ptx_thread_info *thread )
 
    switch ( i_type ) {
    case B16_TYPE:
-   case U16_TYPE:              d.u16 = t?a.u16:b.u16; break;
+   case S16_TYPE:
+   case U16_TYPE: d.u16 = t?a.u16:b.u16; break;
    case F32_TYPE:
    case B32_TYPE:
+   case S32_TYPE:
    case U32_TYPE: d.u32 = t?a.u32:b.u32; break;
    case F64_TYPE:
    case FF64_TYPE:
    case B64_TYPE:
+   case S64_TYPE:
    case U64_TYPE: d.u64 = t?a.u64:b.u64; break;
    default: assert(0);
    }