[llvm] r228578 - fix comment that didn't match the code; remove unnecessary braces; NFC

Sanjay Patel spatel at rotateright.com
Mon Feb 9 08:04:53 PST 2015


Author: spatel
Date: Mon Feb  9 10:04:52 2015
New Revision: 228578

URL: http://llvm.org/viewvc/llvm-project?rev=228578&view=rev
Log:
fix comment that didn't match the code; remove unnecessary braces; NFC

Modified:
    llvm/trunk/lib/Target/X86/X86InstrInfo.cpp

Modified: llvm/trunk/lib/Target/X86/X86InstrInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86InstrInfo.cpp?rev=228578&r1=228577&r2=228578&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86InstrInfo.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86InstrInfo.cpp Mon Feb  9 10:04:52 2015
@@ -4494,12 +4494,11 @@ X86InstrInfo::foldMemoryOperandImpl(Mach
   bool isCallRegIndirect = Subtarget.callRegIndirect();
   bool isTwoAddrFold = false;
 
-  // Atom favors register form of call. So, we do not fold loads into calls
-  // when X86Subtarget is Atom.
+  // For CPUs that favor the register form of a call,
+  // do not fold loads into calls.
   if (isCallRegIndirect &&
-    (MI->getOpcode() == X86::CALL32r || MI->getOpcode() == X86::CALL64r)) {
+    (MI->getOpcode() == X86::CALL32r || MI->getOpcode() == X86::CALL64r))
     return nullptr;
-  }
 
   unsigned NumOps = MI->getDesc().getNumOperands();
   bool isTwoAddr = NumOps > 1 &&





More information about the llvm-commits mailing list