[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