[clang] [clang] Factor out OpenACC part of `Sema` (PR #84184)
Erich Keane via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 11 09:30:44 PDT 2024
erichkeane wrote:
> This direction looks good to me, mostly I just have nits.
>
> The one major change I'd consider before landing is to find a way to avoid the verbosity regression cor3ntin mentions for `Diag()` and friends. Notational regressions within Sema itself may be as important as those of its clients, as it's so much code.
I'd be in favor of just using a 'base' class for these and adding a few of the 'common' ones (`Diag`, `ASTContext`, `LangOpts`), and adding others if we found the need. `SourceManager` isn't really used enough in these sorts of things to be valuable, nor is PP or the FP Features. The OpenCL features might be worth adding, but only to the OpenCL breakout.
https://github.com/llvm/llvm-project/pull/84184
More information about the cfe-commits
mailing list