[PATCH] D39792: [AArch64][SVE] Asm: More concise test format

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 10 03:16:15 PST 2017


rengolin accepted this revision.
rengolin added a comment.

In https://reviews.llvm.org/D39792#921624, @sdesmalen wrote:

> One of the negative tests actually exposed a bug where the same diagnostic is produced several times (it does this because it is unmatched for each register operand class, rather than bail out with a parse failure when z31.x is given). I have a fix for that, but will address it in a separate patch so not to complicate this patch.


Excellent! :)

I agree the fix is better at a different patch. LGTM, thanks!


https://reviews.llvm.org/D39792





More information about the llvm-commits mailing list