I don't have LLVM commit privileges, so can someone please submit this change on my behalf?<div><br></div><div>Thanks,</div><div>Steve<br><br><div class="gmail_quote">On Tue, May 3, 2011 at 7:28 PM, Bruno Cardoso Lopes <span dir="ltr"><<a href="mailto:bruno.cardoso@gmail.com">bruno.cardoso@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 class="im">On Tue, May 3, 2011 at 10:50 PM, Stephen Hines <<a href="mailto:srhines@google.com">srhines@google.com</a>> wrote:<br>

> I should be a bit more clear with my answer. The<br>
> file test/CodeGen/ARM/pack.ll does contain tests for pkhbt and pkhtb, but<br>
> they do not test the actual encoding of the instruction bits. The readable<br>
> assembly code is correct, but it is the encoding where the problem shows up.<br>
> If there are actual encoding tests for the classic code generation path on<br>
> ARM, I would be more than happy to update them. The MC version does<br>
> correctly generate the proper encoding (as seen in<br>
> test/MC/ARM/simple-encoding.ll).<br>
<br>
</div>I misread the "non-MC path" on the subject :)<br>
<font color="#888888"><br>
<br>
--<br>
</font><div><div></div><div class="h5">Bruno Cardoso Lopes<br>
<a href="http://www.brunocardoso.cc" target="_blank">http://www.brunocardoso.cc</a><br>
</div></div></blockquote></div><br></div>