Author: shihui
Date: 2011-04-11 03:19:32 -0400 (Mon, 11 Apr 2011)
New Revision: 3543

Modified:
   trunk/osprey/common/com/x8664/targ_const.cxx
Log:
Fix no side effect expression:

This will not cause any bug but the original thought is accumulate all integer 
values to make the hash value.
Fix is directly add two expressions. 

Code Review: Jian-Xin


Modified: trunk/osprey/common/com/x8664/targ_const.cxx
===================================================================
--- trunk/osprey/common/com/x8664/targ_const.cxx        2011-04-08 02:01:04 UTC 
(rev 3542)
+++ trunk/osprey/common/com/x8664/targ_const.cxx        2011-04-11 07:19:32 UTC 
(rev 3543)
@@ -5943,7 +5943,7 @@
       break;
     case MTYPE_V32F4:
     case MTYPE_V32F8:
-      hash += TCON_v0(*t) + TCON_v1(*t) + TCON_v2(*t) + TCON_v3(*t);
+      hash += TCON_v0(*t) + TCON_v1(*t) + TCON_v2(*t) + TCON_v3(*t) +
               TCON_v4(*t) + TCON_v5(*t) + TCON_v6(*t) + TCON_v7(*t);
       break;
     default:


------------------------------------------------------------------------------
Xperia(TM) PLAY
It's a major breakthrough. An authentic gaming
smartphone on the nation's most reliable network.
And it wants your games.
http://p.sf.net/sfu/verizon-sfdev
_______________________________________________
Open64-devel mailing list
Open64-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open64-devel

Reply via email to