[PATCH] D68873: [AMDGPU] Amend target loop unroll defaults

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 14 09:38:45 PDT 2019


nhaehnle added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp:62
 
 static cl::opt<unsigned> UnrollThresholdLocal(
   "amdgpu-unroll-threshold-local",
----------------
rampitec wrote:
> This change penalizes loops which should have unroll boosted instead. Your new default thresholds are now higher than boosted.
I see now change here. Is something weird going on with the diff?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68873





More information about the llvm-commits mailing list