danix800 wrote:

> > Another PR would fix the crash when this one is landed.
> 
> Can you land the fix for the crash first? If the other PR gets reverted it 
> creates a lot of churn, so it would preferable to land that first. Assuming 
> it is not somehow dependent on this fix.

This new crash is a false alarm caused by the first incorrect fix. Sorry about 
that!

Could you also take a look at the new fix for the first crash?

Previously we think that lambda without trailing return couldn't
have return type defined inside its body, but for
```c
  [] { // no parentheses for parameters
    struct X {};
    return X();
  };
```
the proto has trailing return deduced to the type defined inside.

So I think we should remove the trailing return test for lambdas.

https://github.com/llvm/llvm-project/pull/101031
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to