[PATCH] D82800: [OPENMP50] extend array section for stride (Parsing/Sema/AST)

[Alexey Bataev via Phabricator via cfe-commits]

ABataev added inline comments.


================
Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:7893-7897
+        llvm::errs() << "DEBUG in CGOpenMPRuntime.cpp\n";
+        OASE->getLowerBound()->dump();
+        OASE->getLength()->dump();
+        OASE->getStride()->dump();
+
----------------
Must be removed


================
Comment at: clang/lib/Sema/SemaExpr.cpp:4907
+  if (Stride) {
+    auto Res =
+        PerformOpenMPImplicitIntegerConversion(Stride->getExprLoc(), Stride);
----------------
`auto`->`ExprResult`


================
Comment at: clang/test/OpenMP/target_update_ast_print.cpp:66
 
+#pragma omp target update from(marr [1:1:1] [0:2:] [1:0])
+
----------------
It would be good to add some positive tests for other constructs, like `target`, `target data` etc., if allowed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82800