Expertcoderz wrote:

@Sirraide It seems that the SemaCXX test is failing with the new constant 
interpreter: the errors are emitted as expected when encountering a 
`break`/`continue` in a constant expression but for some reason it fails to 
emit the *"this use of statement expressions is not supported..."* note 
alongside the error.

I've rebuilt and run the test on an older commit to make sure it's not this PR 
that introduced the bug. This also doesn't occur when passing 
`-fno-experimental-new-constant-interpreter` so I guess there's a bug specific 
to it?

Would it be better to leave the test failing or just avoid testing with the new 
constant interpreter until a fix has been decided? I doubt that the required 
fix would fall under the scope of this PR.

https://github.com/llvm/llvm-project/pull/228655
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to