[PATCH] D50593: ConstantMerge: merge common initial sequences

Aditya Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 30 10:15:10 PDT 2018


hiraditya added a comment.

> My main worry is that this is only useful with LTO because the linker should do this work otherwise, so at a minimum I need to change the code accordingly.

This is still useful even if it helps during LTO. As long as you have numbers to validate, it should be fine. We can add a flag to disable this by default.


Repository:
  rL LLVM

https://reviews.llvm.org/D50593





More information about the llvm-commits mailing list