<div dir="ltr">Hi Amaury,<div><br></div><div style>I'm not proficient with the disassembler, but by the look of your three patches, it seems you're just filtering out bad encodings. Do they ever get generated from LLVM? Or is this just a response to MC Hammer?</div>
<div style><br></div><div style>If they ever get generated, would be good to know why and also fix the origin, but if it's just safe guards, than I'm happy with all patches, providing you double checked all encodings (I have to confess I did the first, but not the others ;).</div>
<div style><br></div><div style>cheers,</div><div style>--renato</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 7 June 2013 15:47, Amaury de la Vieuville <span dir="ltr"><<a href="mailto:Amaury.DeLaVieuville@arm.com" target="_blank">Amaury.DeLaVieuville@arm.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
This patch fixes the decoding of CPS in some cases where it is ambiguous<br>
with QADD.<br>
CPS is ambiguous with several other instructions, their decoders check<br>
for pred==0xf but do not check the rest of the encoding, we have to<br>
check for this in decodeCPSInstruction().<br>
This is usually done at the call site but I think in this case - as<br>
there are many of them - it is better to do that directly in the decode<br>
method of the instruction.<br>
<br>
Thanks,<br>
Amaury <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>