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

I don't buy the 'noticeable impact on assertion build' here.  The very next 
line is loading the same variable, so even the smallest amount of inliner 
action makes this at best a non-taken 'jne' type instruction.  

While this MIGHT have a slight impact, it is near-zero, and of the order of 'i 
used a pair of if/else instead of a switch'.

https://github.com/llvm/llvm-project/pull/84184
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to