[llvm-commits] [llvm] r115064 - /llvm/trunk/lib/Target/X86/X86MCCodeEmitter.cpp

Nick Lewycky nicholas at mxc.ca
Wed Sep 29 11:56:58 PDT 2010


Author: nicholas
Date: Wed Sep 29 13:56:57 2010
New Revision: 115064

URL: http://llvm.org/viewvc/llvm-project?rev=115064&view=rev
Log:
Add parens to fix GCC warning:
  lib/Target/X86/X86MCCodeEmitter.cpp: 190: error: suggest parentheses around '&&' within '||'

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

Modified: llvm/trunk/lib/Target/X86/X86MCCodeEmitter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86MCCodeEmitter.cpp?rev=115064&r1=115063&r2=115064&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86MCCodeEmitter.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86MCCodeEmitter.cpp Wed Sep 29 13:56:57 2010
@@ -186,8 +186,8 @@
   const MCOperand &BaseReg  = MI.getOperand(Op+X86::AddrBaseReg);
   const MCOperand &IndexReg = MI.getOperand(Op+X86::AddrIndexReg);
   
-  if (BaseReg.getReg() != 0 && X86::GR32RegClass.contains(BaseReg.getReg()) ||
-      IndexReg.getReg() != 0 && X86::GR32RegClass.contains(IndexReg.getReg()))
+  if ((BaseReg.getReg() != 0 && X86::GR32RegClass.contains(BaseReg.getReg())) ||
+      (IndexReg.getReg() != 0 && X86::GR32RegClass.contains(IndexReg.getReg())))
     return true;
   return false;
 }





More information about the llvm-commits mailing list