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">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>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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 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>