[PATCH] D60118: [AArch64][AsmParser] Fix .arch_extension directive parsing

[Cullen Rhodes via Phabricator via llvm-commits]

c-rhodes added a comment.

> From a quick grep in the test directory, looks like we don't have many tests for the .arch_extension directive. I.e., I see only one, for the simd directive. I think that confirms my suspicion that we tend to forget to add tests/support for the .arch_extension directive when we upstream new architecture support. So, sorry to be a pain here, but while you're at it, my request would be to add tests for some other architecture extensions that we are missing. For example, `rasv8_4` might be an interesting one because it has an underscore in it. Looks like the refactoring in D54633 <https://reviews.llvm.org/D54633> introduced new extensions that we could test.

Hi Sjoerd, yes the tests are lacking for the .arch_extension directive, thanks for pointing that patch out I'll add a few tests for features with interesting characters in the name that are not currently covered.


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

https://reviews.llvm.org/D60118