[PATCH] D93838: [LLVM] [SCCP] : Add Function Specialization pass

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

I could have elaborated a bit more on my plans in my previous message, so let me add that here.

I saw three parts to this work:

1. The first one was the SCCP refactoring that already went in,
2. This patch that adds the function specialisation framework,
3. The further fine-tuning and cost-modelling to get this enabled by default.

In my opinion, the GCC results would justify getting 2 committed first, then work on 3. A few notes on this:

- The requirement to get 2 committed I think is that we get compile-times under control,
- We really would like to get this enabled by default. If we don't reach parity with GCC, this work has little value to us,  and there is little point in doing this work.

Like I said, that was my plan, but am interested to hear if you agree or you see things differently @fhahn, ChuanqiXu. Please let me know.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93838/new/

https://reviews.llvm.org/D93838