[clang] [clang] Factor out OpenACC part of `Sema` (PR #84184)

via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 11 11:27:42 PDT 2024


cor3ntin 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.

Agreed!

https://github.com/llvm/llvm-project/pull/84184


More information about the cfe-commits mailing list