[PATCH] D41442: [AArch64][SVE] Asm: Add parsing of merging/zeroing suffix for SVE predicate vector operands

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 2 04:18:11 PST 2018


fhahn accepted this revision.
fhahn added reviewers: MatzeB, t.p.northover.
fhahn added subscribers: MatzeB, t.p.northover.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM, `/` should only be valid for SVE predicates. I think it should be also fine for the Apple Asm variant, but I am adding  @MatzeB and @t.p.northover as reviewers in case I am missing something.



================
Comment at: lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:2832
+  // Add zero/merge token.
+  const char *ZM = Pred == "z" ? "z" : "m";
+  Operands.push_back(
----------------
nit: No need for a variable here.


https://reviews.llvm.org/D41442





More information about the llvm-commits mailing list