[PATCH] D39089: [AArch64][SVE] Asm: Add SVE (Z) Register definitions and parsing support

[Renato Golin via Phabricator via llvm-commits]

rengolin added inline comments.


================
Comment at: lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:1953
+
+static bool isSVEVectorRegister(std::string Name) {
+  return Name[0] == 'z';
----------------
Isn't this creating a new temporary string just for the comparison? 

You can pass by reference, or depending on how the future usage will be, use a StringRef instead (this can be done later).


https://reviews.llvm.org/D39089