[PATCH] D53611: [hot-cold-split] Only perform splitting in ThinLTO backend post-link

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 23 15:59:32 PDT 2018


tejohnson added a comment.

In https://reviews.llvm.org/D53611#1273295, @vsk wrote:

> Thank you! For completeness, it might help to have a test for the case 'Phase = PostLink'. Happy to leave it to you to decide.


I did initially, but it caused a large number of perturbations of the expected sequence since hot cold splitting invalidated a bunch of analysis passes, so I decided just to test the change being made here.


Repository:
  rL LLVM

https://reviews.llvm.org/D53611





More information about the llvm-commits mailing list