ABataev 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();
----------------
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


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

Reply via email to