[PATCH] D37206: [ItaniumCXXABI] Always use linkonce_odr linkage for RTTI data on MinGW
Martin Storsjö via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 30 14:16:19 PDT 2017
mstorsjo added inline comments.
Comment at: lib/CodeGen/ItaniumCXXABI.cpp:2994
+ // MinGW always uses LinkOnceODRLinkage for type info.
+ if (RD->isDynamicClass() && !IsMinGW)
> Thanks for having a look at this!
> When giving this a second thought now, should we perhaps skip the check for `Environment == GNU` altogether? Or should `xxx-windows-itanium` be different by restricting this behaviour strictly to GNU (does it matter)?
> And given the existing test case that you added with the previous form of this code, and the new test I'm adding here, are there any other odd corner cases I should check?
To rephrase; previously, the comment talked about mingw specifically, but the code only checked for the dllimport attribute. So perhaps I should drop the check for GNU and just check for windows?
More information about the cfe-commits