[PATCH] D140002: [C++20] [Modules] Merging of lambda types in deserialization

[Chuanqi Xu via Phabricator via cfe-commits]

ChuanqiXu planned changes to this revision.
ChuanqiXu added a comment.

In D140002#4000252 <https://reviews.llvm.org/D140002#4000252>, @vsapsai wrote:

> In D140002#4000233 <https://reviews.llvm.org/D140002#4000233>, @ChuanqiXu wrote:
>
>>> Test case MergeLambdas3.cppm is unstable. Sometimes it passes but usually it fails.
>>
>> Weird. What is the error message if it fails?
>
>
>
>   error: 'error' diagnostics seen but not expected: 
>     File /path/to/llvm/build/clang/relwithdebinfo/tools/clang/test/Modules/Output/MergeLambdas3.cppm.tmp/Use2.cpp Line 4: static assertion failed due to requirement '__is_same((lambda at /path/to/llvm/build/clang/relwithdebinfo/tools/clang/test/Modules/Output/MergeLambdas3.cppm.tmp/lambda.h:4:13), (lambda at /path/to/llvm/build/clang/relwithdebinfo/tools/clang/test/Modules/Output/MergeLambdas3.cppm.tmp/lambda.h:1:12))'
>
> Also looks like this test fails in pre-commit checks too.

I reproduced it after I rebase onto main. Let me check what happened..


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140002