[PATCH] D39088: [AArch64][SVE] Asm: Replace 'IsVector' by 'RegKind' in AArch64AsmParser (NFC)

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

A few minor nits, but this looks good to me now. @rengolin what do you think?



================
Comment at: lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:2589
   Operands.push_back(
-      AArch64Operand::CreateReg(Reg, true, S, getLoc(), getContext()));
+      AArch64Operand::CreateReg(Reg, RegisterKind, S, getLoc(), getContext()));
+
----------------
fhahn wrote:
> Any reason for not passing `RegKind::NeonVector` directly here? It seems like RegisterKind is not used anywhere else around here.
Do we need the RegisterKind variable? Doesn't look like it is used somewhere else.


================
Comment at: lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:1909
       return 0;
+
     // set RegNum if the match is the right kind of register
----------------
nit: unrelated whitespace change


================
Comment at: lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:1914
   }
+
   return RegNum;
----------------
nit: unrelated whitespace change.


https://reviews.llvm.org/D39088