[PATCH] D96805: [AMDGPU][CostModel] Refine cost model for control-flow instructions.
Stanislav Mekhanoshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 16 11:42:59 PST 2021
rampitec added a comment.
AMDGPU estimates look reasonable, but bumping unroll threshold twice looks suspicious. Is that to accommodate some loops with switches?
I am afraid this can have quite unpredictable impact on overall performance. Did you run any perf tests?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96805/new/
https://reviews.llvm.org/D96805
More information about the llvm-commits
mailing list