X86: silence warning (-Wparentheses)
authorSaleem Abdulrasool <compnerd@compnerd.org>
Sun, 3 Aug 2014 23:00:39 +0000 (23:00 +0000)
committerSaleem Abdulrasool <compnerd@compnerd.org>
Sun, 3 Aug 2014 23:00:39 +0000 (23:00 +0000)
GCC 4.8.2 points out the ambiguity in evaluation of the assertion condition:

lib/Target/X86/X86FloatingPoint.cpp:949:49: warning: suggest parentheses around ‘&&’ within ‘||’ [-Wparentheses]
   assert(STReturns == 0 || isMask_32(STReturns) && N <= 2);

llvm-svn: 214672

llvm/lib/Target/X86/X86FloatingPoint.cpp

index e74613b..aac7ec7 100644 (file)
@@ -946,7 +946,7 @@ void FPS::handleCall(MachineBasicBlock::iterator &I) {
 
   // FP registers used for function return must be consecutive starting at
   // FP0.
-  assert(STReturns == 0 || isMask_32(STReturns) && N <= 2);
+  assert((STReturns == 0 || isMask_32(STReturns)) && N <= 2);
 
   for (unsigned I = 0; I < N; ++I)
     pushReg(N - I - 1);