<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, May 16, 2018 at 3:09 PM Friedman, Eli via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 5/16/2018 2:54 PM, Benjamin Kramer via llvm-commits wrote:<br>
> This triggered a really annoying miscompile (see r332531), and it's even<br>
> present in the test case. Please double check your tests, especially when<br>
> using update_test_checks.py.<br>
><br>
> Also you want to get these things reviewed by someone who's actually<br>
> familiar with the LLVM before they land.<br>
<br>
I looked at the patch multiple times, and never spotted this issue. <br>
Granted, maybe I should have looked a little more carefully at that part <br>
of the patch (I was more focused on the actual transform).<br></blockquote><div><br></div><div>Probably would have helped here if you'd ack'd the patch yourself. The only approval on the patch is an anonymous person with no llvm history (as far as I can tell).</div><div><br></div><div>-eric </div></div></div>