[PATCH] D146418: Support for OpenMP 5.0 sec 2.12.7 - Declare Target initializer expressions

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 30 06:14:38 PDT 2023


ABataev added inline comments.


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:23136
+          TargetVarDecl->hasInit() && TargetVarDecl->hasGlobalStorage()) {
+        if (auto *Ex = TargetVarDecl->getInit()) 
+	  Visit(Ex);
----------------
Better to expand `auto` here


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:23137
+        if (auto *Ex = TargetVarDecl->getInit()) 
+	  Visit(Ex);
+      }
----------------
Formatting?


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

https://reviews.llvm.org/D146418



More information about the cfe-commits mailing list