<div dir="ltr">On 7 November 2013 10:25, Artyom Skrobov <span dir="ltr"><<a href="mailto:Artyom.Skrobov@arm.com" target="_blank">Artyom.Skrobov@arm.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">CP10/11 are reserved for FP/NEON operations, and are invalid as operands for<br>

the generic coprocessor instructions.<br></blockquote><div></div></div><br></div><div class="gmail_extra">It's funny that the ARM ARM states that p0-p15 are accepted, but on the pseudo-code it states that Coproc_Accepted() cannot be called for p10/p11 (will assert). There should be a warning on the instructions that use them...</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">The change looks good to me, though I'd change the word "copro" in the comment to (at least) "coproc".</div><div class="gmail_extra"><br>
</div><div class="gmail_extra">cheers,</div><div class="gmail_extra">-renato</div></div>