[PATCH] D62707: [AMDGPU] Use InliningThresholdMultiplier for inline hint

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 30 15:34:54 PDT 2019


arsenm added a comment.

Needs test. I'm also not sure I follow how getInliningThresholdMultiplier is used. It looks like it's applied after any of the thresholds are computed in CallAnalyzer::updateThreshold?

Also why do we actually override getInlineThreshold? The alloca object check seems like it shouldn't really be fundamentally different than the SROA checks the default InlineCost does


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

https://reviews.llvm.org/D62707





More information about the llvm-commits mailing list