[PATCH] D63198: [StructurizeCFG] Enable -structurizecfg-relaxed-uniform-regions by default

[Tim Renouf via Phabricator via llvm-commits]

tpr created this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

D62198 <https://reviews.llvm.org/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.

Change-Id: I9cbff002a1e74d3b7eb96b4192dc8129936d537d


Repository:
  rL LLVM

https://reviews.llvm.org/D63198

Files:
  lib/Transforms/Scalar/StructurizeCFG.cpp


Index: lib/Transforms/Scalar/StructurizeCFG.cpp
===================================================================
--- lib/Transforms/Scalar/StructurizeCFG.cpp
+++ lib/Transforms/Scalar/StructurizeCFG.cpp
@@ -65,7 +65,7 @@
 static cl::opt<bool>
     RelaxedUniformRegions("structurizecfg-relaxed-uniform-regions", cl::Hidden,
                           cl::desc("Allow relaxed uniform region checks"),
-                          cl::init(false));
+                          cl::init(true));
 
 // Definition of the complex types used in this pass.
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63198.204275.patch
Type: text/x-patch
Size: 551 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190612/bdd023b9/attachment.bin>