[llvm] r194253 - [ARM] In ARMAsmParser, MatchCoprocessorOperandName() permitted p10 and p11 as operands for coprocessor instructions, resulting in encodings that clash with FP/NEON instruction encodings

Joerg Sonnenberger joerg at britannica.bec.de
Sat Nov 30 14:03:41 PST 2013


On Fri, Nov 08, 2013 at 09:16:31AM -0000, Artyom Skrobov wrote:
> Author: askrobov
> Date: Fri Nov  8 03:16:31 2013
> New Revision: 194253
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=194253&view=rev
> Log:
> [ARM] In ARMAsmParser, MatchCoprocessorOperandName() permitted p10 and
> p11 as operands for coprocessor instructions, resulting in encodings
> that clash with FP/NEON instruction encodings

This commit breaks valid assembler sources that use MCR/MRC explicitly
in place of the (newer) NP/NEON forms. I don't see the point of this
change and therefore would like to see it reverted on trunk and 3.4.

Joerg



More information about the llvm-commits mailing list