[clang] [C++20] [Modules] [Itanium ABI] Generate the vtable in the module unit of dynamic classes (PR #75912)

[David Blaikie via cfe-commits]

================
@@ -1830,6 +1830,9 @@ void ItaniumCXXABI::emitVTableDefinitions(CodeGenVTables &CGVT,
   if (VTable->hasInitializer())
     return;
 
+  if (RD->shouldEmitInExternalSource())
+    return;
----------------
dwblaikie wrote:

Yes, and we do emit them in normal builds above -O0 if the vtable is homed elsewhere due to a key function for instance - they can be used to allow for devirtualization

https://github.com/llvm/llvm-project/pull/75912