[PATCH] D62942: [PATCH 2/2] [AArch64][AsmParser] error on unexpected SVE predicate type suffix

Cullen Rhodes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 6 01:52:14 PDT 2019


c-rhodes added inline comments.


================
Comment at: lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:1085
     if (isSVEVectorReg<Class>() &&
-           (ElementWidth == 0 || Reg.ElementWidth == ElementWidth))
+           (Reg.ElementWidth == ElementWidth))
       return DiagnosticPredicateTy::Match;
----------------
SjoerdMeijer wrote:
> nit: this can now be on the same line?
Good spot, I'll fix before merging. thx!


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62942/new/

https://reviews.llvm.org/D62942





More information about the llvm-commits mailing list