[PATCH] D16616: Avoid inlining call sites in unreachable-terminated block

Easwaran Raman via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 28 11:20:03 PST 2016

eraman added a comment.

The only concern I have is the suppression of inlining of hot calls in the following hypothetical example:

main() {



Even though each of the hot_call_X calls are executed once, inlining may be beneficial (due to constprop for example). Short of checking for caller to be main (as you've done in your previous patch), is there a way to distinguish this case? I suppose this pattern is very rare in real code to worry about this.


More information about the llvm-commits mailing list