[clang] Disable -fdollars-in-identifiers by default (PR #135407)
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 15 04:31:32 PDT 2025
AaronBallman wrote:
> > If we're going to do this, I think we need better diagnostics. Just straight disabling this is going to give very confusing diagnostics to anyone actually using dollar-signs in identifiers.
> > Some ideas:
> > ```
> > * We can give a warning if we see a "$" adjacent to an identifier without any whitespace separating it.
> >
> > * Outside the preprocessor, we can parse a "$" adjacent to an identifier as part of the identifier, with some sort of diagnostic, since it's guaranteed to be an error anyway.
> > ```
>
> My concern here is with regressing performance of the lexer; testing those conditions when lexing _any_ identifier seems like we'd be spending a lot of time trying to catch a very uncommon issue.
>
> I think this is reasonable as follow-up work, but I don't think it should hold up this PR because this is fixing a conformance issue. (FWIW, I think the behavior you get currently is something we could live with even if that follow-up work never happened.)
Are you okay with me punting on this work and landing the changes as-os @efriedma-quic ?
https://github.com/llvm/llvm-project/pull/135407
More information about the cfe-commits
mailing list