X86: Move a comment to be next to the code it describes.
authorGabe Black <gblack@eecs.umich.edu>
Mon, 10 Sep 2007 18:01:52 +0000 (11:01 -0700)
committerGabe Black <gblack@eecs.umich.edu>
Mon, 10 Sep 2007 18:01:52 +0000 (11:01 -0700)
--HG--
extra : convert_revision : c384391175babb7cfdd3885ae9d9f1a9405ea44f

src/arch/x86/isa/microops/regop.isa

index 7ce9dc27fc7e3928cce0a6f4a7dbf4fa8b46f646..d2cfff9d423e84a5026cf65d76c9c40b73c106ed 100644 (file)
@@ -460,6 +460,11 @@ let {{
     class Xor(LogicRegOp):
         code = 'DestReg = merge(DestReg, psrc1 ^ op2, dataSize)'
 
+    # Neither of these is quite correct because it assumes that right shifting
+    # a signed or unsigned value does sign or zero extension respectively.
+    # The C standard says that what happens on a right shift with a 1 in the
+    # MSB position is undefined. On x86 and under likely most compilers the
+    # "right thing" happens, but this isn't a guarantee.
     class Mul1s(WrRegOp):
         code = '''
             ProdLow = psrc1 * op2;
@@ -489,11 +494,6 @@ let {{
     class Mulel(RdRegOp):
         code = 'DestReg = merge(SrcReg1, ProdLow, dataSize);'
 
-    # Neither of these is quite correct because it assumes that right shifting
-    # a signed or unsigned value does sign or zero extension respectively.
-    # The C standard says that what happens on a right shift with a 1 in the
-    # MSB position is undefined. On x86 and under likely most compilers the
-    # "right thing" happens, but this isn't a guarantee.
     class Muleh(RdRegOp):
         def __init__(self, dest, src1=None, flags=None, dataSize="env.dataSize"):
             if not src1: