mstorsjo added inline comments.
================ Comment at: lib/CodeGen/ItaniumCXXABI.cpp:2766 + // Don't import the RTTI but emit it locally + if (CGM.getTriple().isWindowsGNUEnvironment() && IsDLLImport) + return false; ---------------- compnerd wrote: > Can't this be simplified to ignore the `IsDLLImport`? If it is export, it > should be emitted, if it is import, it should emit it locally. If it is > static, it should also emit it locally, no? Hmm, maybe - I tried to keep the condition as specific as possible for the one case that doesn't work right now. I think that might give more stray changes to tests though. https://reviews.llvm.org/D42641 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
