HomePhabricator

[GlobalOpt] Swap the expensive check for cold calls with the cheap TTI check

Description

[GlobalOpt] Swap the expensive check for cold calls with the cheap TTI check

isValidCandidateForColdCC is much more expensive than
TTI.useColdCCForColdCall, which by default just returns false. Avoid
doing this work if we're not going to look at the answer anyway.

This change is NFC, but I see significant compile time improvements on
some code with pathologically many functions.

Details

Committed
bognerApr 25 2019, 5:12 PM
Parents
rL359252: [ORC] Remove symbols from dependency lists when failing materialization.
Branches
Unknown
Tags
Unknown