<div dir="ltr">Hi Evan, this is about your r159230, <a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20120625/145470.html">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20120625/145470.html</a> . The canonicalization was added, then removed by you. In post-commit review, Eli asked you to pick one of the two forms to be canonical.<br>

<div><br></div><div>I have no problem with David landing this patch as is -- it looks like goodness to me, particularly the "C - zext(X)" transform -- but I don't want to mess you up. Could you please chime in?</div>

<div><br></div><div>Nick</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 12 July 2013 16:03, David Majnemer <span dir="ltr"><<a href="mailto:david.majnemer@gmail.com" target="_blank">david.majnemer@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>The attached patch essentially reverts r159230.</div><div><br></div><div>The reasons why are because:</div>

<div>1. It creates an asymmetry in the IR by hurting our ability to canonicalize things like C - zext(X) and C + zext(!X) into the same thing.</div>
<div>2. Adding it back does not break any tests and without it we still generate the inferior code mentioned in the commit log.</div><div><br></div><div>It seems like we should keep the canonicalization and fix (the apparently still broken problem) in the lowering stage.</div>


<div>The commit mentioned that some targets don't have a sext i1. This shouldn't be a concern at the IR level, should it?</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>-- </div><div>David Majnemer</div>

</font></span></div>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>