[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