[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