Skip to content

Commit 5a07943

Browse files
author
Tim Renouf
committed
[StructurizeCFG] Enable -structurizecfg-relaxed-uniform-regions by default
D62198 introduced an option to relax the checks for hasOnlyUniformBranches. This commit turns the option on by default, for better code generation in some cases in AMDGPU. Differential Revision: https://reviews.llvm.org/D63198 Change-Id: I9cbff002a1e74d3b7eb96b4192dc8129936d537d llvm-svn: 368042
1 parent 7f320d4 commit 5a07943

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

llvm/lib/Transforms/Scalar/StructurizeCFG.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ static cl::opt<bool> ForceSkipUniformRegions(
6565
static cl::opt<bool>
6666
RelaxedUniformRegions("structurizecfg-relaxed-uniform-regions", cl::Hidden,
6767
cl::desc("Allow relaxed uniform region checks"),
68-
cl::init(false));
68+
cl::init(true));
6969

7070
// Definition of the complex types used in this pass.
7171

llvm/test/CodeGen/AMDGPU/control-flow-optnone.ll

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@
1515
; GCN: s_mov_b64 exec
1616

1717
; GCN: s_or_b64 exec, exec
18-
; GCN: v_cmp_eq_u32
19-
; GCN: s_cbranch_vccnz
18+
; GCN: s_cmp_eq_u32
19+
; GCN: s_cbranch_scc1
2020
; GCN-NEXT: s_branch
2121
define amdgpu_kernel void @copytoreg_divergent_brcond(i32 %arg, i32 %arg1, i32 %arg2) #0 {
2222
bb:

0 commit comments

Comments
 (0)