[PATCH] D41382: [AArch64] Asm: Fix parsing of register aliases that have a name starting with 'z'
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 19 13:53:18 PST 2017
fhahn added reviewers: echristo, olista01.
fhahn added a comment.
LGTM. The problem this solves is that `MatchRegisterName` parses GPRs, NEON and SVE registers, which leads to surprising behaviour, i.e. `matchRegisterNameAlias` with RegKind == Scalar would return > 0 for any (valid) Neon & SVE register as well. From the comments and the behaviour, it seems like `tryParseRegister` should only parse scalar registers, which this patch implements. I think it's worth adding a comment making that clear.
Adding Eric, in case he has some more thoughts!
https://reviews.llvm.org/D41382
More information about the llvm-commits
mailing list