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

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 11 09:30:20 PDT 2024


================
@@ -1162,6 +1162,11 @@ class Sema final {
   /// CurContext - This is the current declaration context of parsing.
   DeclContext *CurContext;
 
+  SemaOpenACC &OpenACC() {
----------------
AaronBallman wrote:

I think @sam-mccall is reading the coding guidelines the same way I read them, and I agree with his conclusion that I also prefer `openACC()` over `getOpenACC()`. I don't find `get` to add much value to readability in this case and I think deviating from the style guideline is appropriate.

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


More information about the cfe-commits mailing list