cchen marked an inline comment as done.
cchen added inline comments.
================
Comment at: clang/test/OpenMP/target_defaultmap_messages.cpp:49
+
+ #pragma omp target defaultmap(tofrom:scalar) defaultmap(to:scalar) //
expected-error {{at most one defaultmap clause for each variable-category can
appear on the directive}}
+ foo();
----------------
ABataev wrote:
> cchen wrote:
> > ABataev wrote:
> > > Same test case must exist for OpenMP 4.5
> > This test is only for testing the new rule in OpenMP 5.0 which disallow
> > duplicate variable-category in defaultmap. For OpenMP 4.5, only one
> > defaultmap is allowed, and I think I didn't remove the test for that by
> > this one.
> We don't have such test for OpenMP 4.5 and it would be good to have it
I'm glad to add the test (duplicate defaultmap) for OpenMP 4.5 defaultmap. But
just to clarify, are you also saying that I should remove the `ifdef OMP5` and
add the error message for OpenMP 4.5?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69204/new/
https://reviews.llvm.org/D69204
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
