[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