[PATCH] D36710: [InlineCost] NFC - refactor the checks for different analyses to be a bit more localized to the code that uses those analyses.

Easwaran Raman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 14 14:17:14 PDT 2017


eraman added a comment.

LGTM

This is not strictly NFC since the entire hot/cold threshold update code used to be guarded by if (PSI) even though PSI is not needed for local hot/cold callsite updates, but practically all the callers within LLVM does pass a non-null PSI.


https://reviews.llvm.org/D36710





More information about the llvm-commits mailing list