[llvm] r263823 - AMDGPU: add missing braces around multi-line if block

Nicolai Haehnle via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 18 13:32:04 PDT 2016


Author: nha
Date: Fri Mar 18 15:32:04 2016
New Revision: 263823

URL: http://llvm.org/viewvc/llvm-project?rev=263823&view=rev
Log:
AMDGPU: add missing braces around multi-line if block

This fixes an issue with rL263658 pointed out by Tom Stellard.

Modified:
    llvm/trunk/lib/Target/AMDGPU/SILowerControlFlow.cpp

Modified: llvm/trunk/lib/Target/AMDGPU/SILowerControlFlow.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AMDGPU/SILowerControlFlow.cpp?rev=263823&r1=263822&r2=263823&view=diff
==============================================================================
--- llvm/trunk/lib/Target/AMDGPU/SILowerControlFlow.cpp (original)
+++ llvm/trunk/lib/Target/AMDGPU/SILowerControlFlow.cpp Fri Mar 18 15:32:04 2016
@@ -136,7 +136,7 @@ bool SILowerControlFlow::shouldSkip(Mach
     for (MachineBasicBlock::iterator I = MBB->begin(), E = MBB->end();
          NumInstr < SkipThreshold && I != E; ++I) {
 
-      if (I->isBundle() || !I->isBundled())
+      if (I->isBundle() || !I->isBundled()) {
         // When a uniform loop is inside non-uniform control flow, the branch
         // leaving the loop might be an S_CBRANCH_VCCNZ, which is never taken
         // when EXEC = 0. We should skip the loop lest it becomes infinite.
@@ -145,6 +145,7 @@ bool SILowerControlFlow::shouldSkip(Mach
 
         if (++NumInstr >= SkipThreshold)
           return true;
+      }
     }
   }
 




More information about the llvm-commits mailing list