[PATCH] D76635: Make "Skip Uniform Regions" optimization in the Structurizer pass support "-opt-bisect-limit" option.

Konstantin Pyzhov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 23 12:00:58 PDT 2020


kpyzhov created this revision.
kpyzhov added a reviewer: sameerds.
kpyzhov added a project: AMDGPU.
Herald added subscribers: llvm-commits, hiraditya.
Herald added a project: LLVM.

Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D76635

Files:
  llvm/lib/Transforms/Scalar/StructurizeCFG.cpp


Index: llvm/lib/Transforms/Scalar/StructurizeCFG.cpp
===================================================================
--- llvm/lib/Transforms/Scalar/StructurizeCFG.cpp
+++ llvm/lib/Transforms/Scalar/StructurizeCFG.cpp
@@ -1013,7 +1013,7 @@
 
   DA = nullptr;
 
-  if (SkipUniformRegions) {
+  if (SkipUniformRegions && !skipRegion(*R)) {
     // TODO: We could probably be smarter here with how we handle sub-regions.
     // We currently rely on the fact that metadata is set by earlier invocations
     // of the pass on sub-regions, and that this metadata doesn't get lost --


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D76635.252103.patch
Type: text/x-patch
Size: 582 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200323/ff4a4ad9/attachment.bin>


More information about the llvm-commits mailing list