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

[Alexey Bataev via Phabricator via cfe-commits]

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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits