[PATCH] D77290: [OpenMP] Specialize OpenMP calls after template instantiation

Mike Rice via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 6 18:01:37 PDT 2020


mikerice added inline comments.


================
Comment at: clang/lib/Sema/SemaExprCXX.cpp:8260
     }
-    return Result;
+
+    if (!SemaRef.getLangOpts().OpenMP || !Result.isUsable() ||
----------------
I was expecting the code would be added to RebuildCallExpr in TreeTransform.h.  This seems to just override one of the classes derived from TreeTransform.  

I think I'd try adding it to TreeTransform then check all the cases where a subclass overrides RebuildCallExpr and try to determine if something needs to be done there.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77290/new/

https://reviews.llvm.org/D77290





More information about the cfe-commits mailing list