Fznamznon wrote:
> Oh gosh, I'm an idiot, i meant true
That makes it a little bit challenging to preserve old warnings for older
versions of the language, without additional flags and options, in some cases
like:
```
struct A {
~A();
};
struct B : A { };
struct C {
B a;
constexpr ~C() {}
};
```
So, when considering `~C`, it can be checked in
`CheckConstexprDestructorSubobjects` that `B` has constexpr destructor and by
the new rules it will, but there is no recursion there so `A` won't be checked
and warning is lost. I'm not sure I want to add recursion to
`CheckConstexprDestructorSubobjects`. Not sure about flags to CXXRecordDecl too.
https://github.com/llvm/llvm-project/pull/77753
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
