[PATCH] D146924: [clang] Add support for dollar sign in ud_suffix of numeric constants

Mon Mar 27 05:54:08 PDT 2023

cor3ntin added a comment.

I think it's worth pursuing and investigating as long as

- We requires some prefix before the dollar ( like `_`), and `42$ is never valid
- `-Wdollar-in-identifier-extension` is emitted
- We have a better understanding of the current behavior for the euro sign - logically the euro sign should be split in its own token too at the moment so *maybe* there is another bug there!

Thanks for working on this btw.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146924