ebevhan marked an inline comment as done.
ebevhan added inline comments.

================
Comment at: test/SemaOpenCL/address-spaces.cl:87
+
+  // FIXME: This doesn't seem right. This should be an error, not a warning.
+  __local int * __global * __private * lll;
----------------
Anastasia wrote:
> Are we sure it has to be an error? May be we can change this wording to 
> something like - unknown behavior that needs clarifying? Some similar text to 
> your comment in the code.
Well... If `__private int **` to `__generic int **` is an error but `__private 
int ***` to `__generic int **` isn't, that seems a bit odd to me...



CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58236/new/

https://reviews.llvm.org/D58236



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

Reply via email to