<div dir="ltr">Hi Tim.<div><br></div><div style>As requested, I have split the patch into two and fixed that test.</div><div style><br>Regards,</div><div style>Mihai</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Tue, May 14, 2013 at 1:09 PM, Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Mihail,<br>
<br>
Although these are both changes to "fix VSTn disassembly", I think<br>
they should probably be committed separately.<br>
<br>
Both code changes look reasonable to me, but the test<br>
invalid-VST2b32_UPD-arm.txt is semantically incorrect. The<br>
disassembler will try to resynchronise the stream after the first<br>
failure, but it will only skip 1 byte. So the file given won't<br>
actually test what you think it does. The output of the llvm-mc<br>
command is:<br>
<stdin>:11:1: warning: invalid instruction encoding<br>
0xb3 0x09 0x03 0xf4<br>
^<br>
<stdin>:11:6: warning: invalid instruction encoding<br>
0xb3 0x09 0x03 0xf4<br>
     ^<br>
        stmeq   r3, {r0, r1, r10, r12, sp, lr, pc} ^<br>
        ldmeq   r3!, {r0, r1, r10, r12, sp, lr, pc}<br>
<stdin>:19:11: warning: invalid instruction encoding<br>
0xb3 0x08 0x03 0xf4<br>
          ^<br>
<stdin>:19:16: warning: invalid instruction encoding<br>
0xb3 0x08 0x03 0xf4<br>
               ^<br>
<br>
where those stmeq and ldmeq instructions come from disassembling<br>
unintended sequences of bytes and the failures aren't for what you<br>
think they should be.<br>
<br>
Tim.<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>
</blockquote></div><br></div>