-
Notifications
You must be signed in to change notification settings - Fork 12.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Assertion allLookupResultsAreTheSame failure on code with C++ Modules #122328
Comments
@llvm/issue-subscribers-clang-modules Author: Mikail Bagishov (MikailBag)
Code (https://gist.github.com/MikailBag/75ff72a9e3cd7e4ef9dbdeb5523c4ca9)
I can't provide a single preprocessed source file, since code uses modules.
This code is likely invalid (each file contains independently reduced copy of std and stdexec so some ODR violations were bound to occur; in particular __make_sexpr_t struct has different definitions in TUs; I get the following error
There is another issue with exactly same assertion message: #15198. But I don't think this one is duplicate: as far as I can see, this code makes no attempt to declare additional overload for the lambda call operator, in particular no |
Code (https://gist.github.com/MikailBag/75ff72a9e3cd7e4ef9dbdeb5523c4ca9)
I can't provide a single preprocessed source file, since code uses modules.
This code is likely invalid (each file contains independently reduced copy of std and stdexec so some ODR violations were bound to occur; in particular __make_sexpr_t struct has different definitions in TUs;
Xclang -fno-skip-odr-check-in-gmf
gives errors as well).I get the following error
There is another issue with exactly same assertion message: #15198. But I don't think this one is duplicate: as far as I can see, this code makes no attempt to declare additional overload for the lambda call operator, in particular no
friend
-s are involved.The text was updated successfully, but these errors were encountered: