Stefan =?utf-8?q?Gränitz?= <stefan.graen...@gmail.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/84...@github.com>
================
@@ -14,7 +14,7 @@ struct A { int a; A(int a) : a(a) {} virtual ~A(); };
// PartialTranslationUnit.
inline A::~A() { printf("~A(%d)\n", a); }
-// Create one instance with new and delete it.
+// Create one instance with new and delete it. We crash here now:
A *a1 = new A(1);
----------------
hahnjo wrote:
Do we even have initial PTUs in the default case? Also the minimal reproducer
shows a more general version where the `virtual` destructor is actually defined
inline (c861d32d7c2791bdc058d9d9fbaecc1c2f07b8c7 addresses the case where it is
out-of-line, which is special due to key `virtual` functions). So if that
breaks entirely (which is critical for us), I'm personally not ok with just
`XFAIL`ing it to land another change...
https://github.com/llvm/llvm-project/pull/84758
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
