[PATCH] D64642: [AMDGPU] Tune inlining parameters for AMDGPU target
Easwaran Raman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 16 21:27:21 PDT 2019
eraman added inline comments.
================
Comment at: llvm/lib/Analysis/InlineCost.cpp:883
//
- // Vector bonuses: We want to more aggressively inline vector-dense kernels
- // and apply this bonus based on the percentage of vector instructions. A
----------------
The comment block explaining vector bonuses is still relevant after this change. Instead of removing it, you should modify it to say the bonus percentage is target dependent.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64642/new/
https://reviews.llvm.org/D64642
More information about the llvm-commits
mailing list