carlo.bertolli marked an inline comment as done.
carlo.bertolli added a comment.

> No positive tests for the construct


Do you mean in the regression test? I thought I covered all negative and 
positive cases, but please let me know if I missed anything.


================
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:8333
@@ -8332,1 +8332,3 @@
   "expected at least one 'to' clause or 'from' clause specified to '#pragma 
omp target update'">;
+def  err_omp_usedeviceptr_not_a_pointer : Error<
+  "expected pointer in 'use_device_ptr' clause">;
----------------
ABataev wrote:
> Shall we accept refs to pointers and arrays?
I think that people of the OpenMP committee agreed that we can accept 
references to pointers. There is no indication in the specifications about 
arrays, so I will leave them out for the time being.
I will add a mention of references here.


Repository:
  rL LLVM

http://reviews.llvm.org/D21904



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to