[llvm] r225154 - [X86] Remove a strange fixme referring to a hack that doesn't seem to exist since the code is in a comment. Can't figure out what the body of the 'if' was supposed to be anyway.

Craig Topper craig.topper at gmail.com
Mon Jan 5 00:19:05 PST 2015


Author: ctopper
Date: Mon Jan  5 02:19:05 2015
New Revision: 225154

URL: http://llvm.org/viewvc/llvm-project?rev=225154&view=rev
Log:
[X86] Remove a strange fixme referring to a hack that doesn't seem to exist since the code is in a comment. Can't figure out what the body of the 'if' was supposed to be anyway.

Modified:
    llvm/trunk/lib/Target/X86/MCTargetDesc/X86BaseInfo.h

Modified: llvm/trunk/lib/Target/X86/MCTargetDesc/X86BaseInfo.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/MCTargetDesc/X86BaseInfo.h?rev=225154&r1=225153&r2=225154&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/MCTargetDesc/X86BaseInfo.h (original)
+++ llvm/trunk/lib/Target/X86/MCTargetDesc/X86BaseInfo.h Mon Jan  5 02:19:05 2015
@@ -684,9 +684,6 @@ namespace X86II {
         ++FirstMemOp;// Skip the register source (which is encoded in I8IMM).
       if (HasEVEX_K)
         ++FirstMemOp;// Skip the mask register
-      // FIXME: Maybe lea should have its own form?  This is a horrible hack.
-      //if (Opcode == X86::LEA64r || Opcode == X86::LEA64_32r ||
-      //    Opcode == X86::LEA16r || Opcode == X86::LEA32r)
       return FirstMemOp;
     }
     case X86II::MRMXr:





More information about the llvm-commits mailing list