[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