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

[Easwaran Raman via llvm-commits]

eraman added a comment.

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

main() {

  hot_call_1();
  ...
  hot_call_N()
  exit(0);

}

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.


http://reviews.llvm.org/D16616