[PATCH] D158540: Improve error message for constexpr constructors of virtual base classes

Wed Aug 30 07:04:08 PDT 2023

aaron.ballman added a comment.

In D158540#4628265 <https://reviews.llvm.org/D158540#4628265>, @NoumanAmir657 wrote:

> No, I don't have code examples that showcase the importance of the note. As you said the class context would be obvious whenever we run into this error.
> The test files also don't show where the note would be helpful.

The crux of https://github.com/llvm/llvm-project/issues/64843 is about the error diagnostic, and that logic hasn't been changed in this patch -- are there other changes that are missing from the patch? The text of the tests shows that the error diagnostic behavior should have changed as well, but I'm not seeing the functional changes to make that happen.

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

https://reviews.llvm.org/D158540