<div dir="ltr"><div><div><div>Hi Tom, Owen,<br><br></div>I decided to use 3.7 branch instead of future 3.6.1 release in my project. I need a set of fixes to be applied to LLVM and it is all-or-nothing situation in my case. It is not guaranteed that all of them will be included in 3.6.1 and proposing patches for 3.6.1 is quite time-consuming. So you can ignore this nomination.<br><br></div>Thanks,<br></div>Paweł<br></div><br><div class="gmail_quote">On Mon, Apr 20, 2015 at 5:30 PM Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Mar 18, 2015 at 12:56:59PM +0000, Paweł Bylica wrote:<br>
> --------------------------------------------------------------------<br>
> r231984:<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=231984&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=231984&view=rev</a><br>
> Log:<br>
> Handle big index in getelementptr instruction<br>
><br>
> CodeGen incorrectly ignores (assert from APInt) constant index bigger<br>
> than 2^64 in getelementptr instruction. This is a test and fix for that.<br>
><br>
> Patch by Paweł Bylica!<br>
><br>
> Reviewed By: rnk<br>
><br>
> Subscribers: majnemer, rnk, mcrosier, resistor, llvm-commits<br>
><br>
> Differential Revision: <a href="http://reviews.llvm.org/D8219" target="_blank">http://reviews.llvm.org/D8219</a><br>
><br>
> Added:<br>
>     llvm/trunk/test/CodeGen/X86/getelementptr.ll<br>
> Modified:<br>
>     llvm/trunk/lib/CodeGen/SelectionDAG/FastISel.cpp<br>
>     llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp<br>
><br>
<br>
Hi Owen,<br>
<br>
Is this patch OK for the 3.6 branch?<br>
<br>
Thanks,<br>
Tom<br>
</blockquote></div>