<div dir="ltr">Seems reasonable, pretty sure the ARM port has the same bug though so could you fix it in both places? Maybe pull it out into more general code somewhere?<div><br></div><div>-eric</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Wed, Nov 6, 2013 at 4:23 PM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This patch changes fastisel to only fold constant add operations into GEPs when the type sizes match.  Without it, the operands of the add are implicitly promoted, changing the conditions of an overflow and the result of the GEP.<br>

<br>
<rdar://problem/15292280><br>
<br>
<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>