ahatanak added a comment.

In https://reviews.llvm.org/D26108#591270, @jordan_rose wrote:

> Ah, my apologies, @ahatanak. I was testing against the Swift branch of Clang.


That's OK. I haven't tested it yet, but the patch itself looks fine to me.



================
Comment at: lib/Sema/SemaType.cpp:3988
       // Allow arrays of auto if we are a generic lambda parameter.
       // i.e. [](auto (&array)[5]) { return array[0]; }; OK
       if (AT && D.getContext() != Declarator::LambdaExprParameterContext) {
----------------
This isn't something that is related to the changes made in this patch, but it 
looks like we issue a -Wnullability-completeness warning when there is a 
generic lambda like this:

```
auto G2 = [](auto a){};
```

Is this expected?


Repository:
  rL LLVM

https://reviews.llvm.org/D26108



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

Reply via email to