[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