<div dir="ltr">Fixed in r183238.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 4, 2013 at 6:46 AM, James Molloy <span dir="ltr"><<a href="mailto:james@jamesmolloy.co.uk" target="_blank">james@jamesmolloy.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Me too. Thanks!<div class="HOEnZb"><div class="h5"><div><br></div><div><br><div class="gmail_quote">On 4 June 2013 13:44, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</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">LGTM.</div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">
On 4 June 2013 12:02, 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">Hi James,<div><br></div><div>I've attached a patch that address your concerns regarding the testcase.</div>


<span><font color="#888888"><div><br></div><div>-- </div><div>David Majnemer</div><div><br></div>
</font></span></div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 4, 2013 at 3:32 AM, James Molloy <span dir="ltr"><<a href="mailto:james@jamesmolloy.co.uk" target="_blank">james@jamesmolloy.co.uk</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi David,<div><br></div><div>Thanks for working on this. I was initially concerned that the patch didn't address LDRH too, but I've noticed that uses a different addressing mode so isn't affected.</div>



<div><br>
</div><div>Minor comments: your testcase can be reduced further. It's complicated by the fact that you took it directly from Clang's output - you can change the global constructor into a simple function and remove the TBAA data - that leaves you with a simpler testcase which is more obvious to a maintainer (we're not testing global constructors in this test!)</div>




<div><br></div><div>Also, could you please rename your testcase to something more descriptive than a PR number? You can say inside the testcase that it addresses a PR, but the test name is what shows up when we break it so please make it more descriptive.</div>




<div><br></div><div>Cheers,</div><div><br></div><div>James<br><br><div class="gmail_quote"><div><div>On 4 June 2013 06:54, David Majnemer <span dir="ltr"><<a href="mailto:david.majnemer@gmail.com" target="_blank">david.majnemer@gmail.com</a>></span> wrote:<br>




</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr">Hi,<div><br></div><div>The ARM backend did not expect LDRBi12 to hold a constant pool operand.<br>



</div>
<div><br></div><div>The attached patch allows for LLVM to deal with the instruction similar to how it deals with LDRi12.</div>
<div><br></div><div>This addresses PR16215.</div><span><font color="#888888"><div><br></div><div>-- </div><div>David Majnemer</div></font></span></div>
<br></div></div>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">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>
</blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">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>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>