<div dir="ltr">+1<div>If possible, it definitely helps to fuzz it.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jun 10, 2017 at 2:45 PM, Davide Italiano via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat, Jun 10, 2017 at 8:21 AM, Geoff Berry via llvm-dev<br>
<<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br>
> Hi All,<br>
><br>
> I'd like to switch the function simplification run of EarlyCSE to use<br>
> MemorySSA by default.  This change has been in tree for a while and I've<br>
> done benchmark and self-hosting testing on both aarch64 and x86, but I<br>
> wanted to give other folks with different tests/targets a chance to try this<br>
> out before I flip the switch. I've added an option '-enable-earlycse-memssa'<br>
> (or '-enable-npm-gvn-hoist' if you're using the new pass manager) that you<br>
> can be used to test this change out.<br>
><br>
<br>
</span>Not that I don't trust your self-hosting test strategy, but other<br>
passes (e.g. GVNHoist or NewGVN) showed us that's not quite enough to<br>
declare victory. I'll be much more confident if we can do at least a<br>
round of fuzzing.<br>
<br>
Thanks,<br>
<br>
--<br>
Davide<br>
<div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
</div></div></blockquote></div><br></div>