[PATCH] D50593: ConstantMerge: merge common initial sequences

JF Bastien via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 10 10:55:40 PDT 2018


jfb planned changes to this revision.
jfb added a comment.

In https://reviews.llvm.org/D50593#1228463, @hiraditya wrote:

> If you have the code-size numbers, please provide that then I can accept the patch.


I want to get back to this after addressing a few related things. 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. I'm working on other ways to achieve the size wins I want, and then I'll fiddle with LTO to see how much can be done.


Repository:
  rL LLVM

https://reviews.llvm.org/D50593





More information about the llvm-commits mailing list