<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 9, 2016 at 4:00 AM, Sean Silva via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">silvas added a comment.<br>
<br>
Also, uhm, this needs tests. Maybe you can get away with just adding some more RUN lines to some LegacyGVN tests?<br>
<br>
<br></blockquote><div><br></div><div>All the old gvn tests are run with newgvn, and x-failed appropriately on the branch.</div><div>Davide, did you miss adding those changes to this patch?</div><div><br></div></div></div></div>