[llvm-dev] Enabling EarlyCSE w/ MemorySSA by default
Zhendong Su via llvm-dev
llvm-dev at lists.llvm.org
Sat Jun 10 15:38:45 PDT 2017
Sure, we are happy to start some testing to see what we may help find.
On Sat, Jun 10, 2017 at 3:36 PM, Daniel Berlin <dberlin at dberlin.org> wrote:
> If possible, it definitely helps to fuzz it.
> On Sat, Jun 10, 2017 at 2:45 PM, Davide Italiano via llvm-dev <
> llvm-dev at lists.llvm.org> wrote:
>> On Sat, Jun 10, 2017 at 8:21 AM, Geoff Berry via llvm-dev
>> <llvm-dev at lists.llvm.org> wrote:
>> > Hi All,
>> > I'd like to switch the function simplification run of EarlyCSE to use
>> > MemorySSA by default. This change has been in tree for a while and I've
>> > done benchmark and self-hosting testing on both aarch64 and x86, but I
>> > wanted to give other folks with different tests/targets a chance to try
>> > out before I flip the switch. I've added an option
>> > (or '-enable-npm-gvn-hoist' if you're using the new pass manager) that
>> > can be used to test this change out.
>> Not that I don't trust your self-hosting test strategy, but other
>> passes (e.g. GVNHoist or NewGVN) showed us that's not quite enough to
>> declare victory. I'll be much more confident if we can do at least a
>> round of fuzzing.
>> LLVM Developers mailing list
>> llvm-dev at lists.llvm.org
Zhendong Su, Professor
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-dev