Anastasia added inline comments.
================
Comment at: clang/test/SemaOpenCLCXX/references.cl:29
+void foo();
+void test(void (&par)()) {
+ void (&loc)();
----------------
Anastasia wrote:
> oops, I thought this was covered in my patch. I will see if there is a quick
> fix and if not I will create another PR.
Ok, I have addressed that. I realized that my original patch was not doing what
I said in the description as it was still allowing the references to functions
in type aliases or template arguments. While I think such functionality might
be useful we should first start from the language definition side of it. For
now let's just go with the conservative approach and disallow this.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95442/new/
https://reviews.llvm.org/D95442
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
