[PATCH] D81211: [AMDGPU] Enable structurizer workarounds by default

Changpeng Fang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 4 20:56:24 PDT 2020


cfang added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp:197
 static cl::opt<bool> EnableStructurizerWorkarounds(
     "amdgpu-enable-structurizer-workarounds",
+    cl::desc("Enable workarounds for the StructurizeCFG pass"), cl::init(true),
----------------
Can we come out with a name that is closer to what was implemented, like amdgpu-unify-loop-exits, or even amdgpu-normalize-loops(cfg)? "workarounds" is kind of misleading.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81211/new/

https://reviews.llvm.org/D81211





More information about the llvm-commits mailing list