================ @@ -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