[PATCH] D154202: [MC/AsmLexer] Add '?' (Question) token

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 7 15:19:18 PDT 2023


MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

I agree that this is a way forward and enable disabling `?` in the middle of symbol names.

(Unfortunately D72680 <https://reviews.llvm.org/D72680> did not add `?` at symbol start tests, and I am unclear whether ml.exe does accept these symbols.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154202



More information about the llvm-commits mailing list