================
@@ -318,6 +318,9 @@ namespace {
if (Diags.hasUnrecoverableErrorOccurred())
return;
+ if (RD->shouldEmitInExternalSource())
----------------
efriedma-quic wrote:
The way I see it, Sema should have the exact right answer for whether the
vtable is required. If we need to second-guess the choice later, that means
Sema didn't compute the right thing in the first place, which would be a bug in
Sema.
https://github.com/llvm/llvm-project/pull/75912
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
