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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
