[PATCH] D18488: [OpenMP] Parsing and sema support for the from clause
Samuel Antao via cfe-commits
cfe-commits at lists.llvm.org
Wed May 25 13:32:03 PDT 2016
sfantao added a comment.
Hi Alexey,
Thanks for the review!
In http://reviews.llvm.org/D18488#438841, @ABataev wrote:
> Missed a test for printing and serialization/deserialization
Maybe it is `test/OpenMP/target_update_ast_print.cpp` what you were looking for? Let me know if not.
Thanks again,
Samuel
================
Comment at: test/OpenMP/nesting_of_regions.cpp:1832
@@ -1831,3 +1831,3 @@
for (int i = 0; i < 10; ++i) {
-#pragma omp target update to(a)
+#pragma omp target update from(a)
}
----------------
ABataev wrote:
> Again, this test must be updated only for new directives, not clauses
Ok, I'm not touching this test in this patch anymore.
http://reviews.llvm.org/D18488
More information about the cfe-commits
mailing list