<div dir="ltr">Thanks for fast fix, Benjamin.</div><div class="gmail_extra"><br><div class="gmail_quote">2018-05-17 0:09 GMT+02:00 Friedman, Eli <span dir="ltr"><<a href="mailto:efriedma@codeaurora.org" target="_blank">efriedma@codeaurora.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 5/16/2018 2:54 PM, Benjamin Kramer via llvm-commits wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
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>
</blockquote>
<br></span>
I looked at the patch multiple times, and never spotted this issue. Granted, maybe I should have looked a little more carefully at that part of the patch (I was more focused on the actual transform).<span class="HOEnZb"><font color="#888888"><br>
<br>
-Eli<br>
<br>
-- <br>
Employee of Qualcomm Innovation Center, Inc.<br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project<br>
<br>
</font></span></blockquote></div><br></div>