[PATCH] D51288: [MinGW] [X86] Add stubs for references to data variables that might end up imported from a dll

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added inline comments.


================
Comment at: lib/Target/X86/X86Subtarget.cpp:146
+      !GV->hasDLLImportStorageClass() && GV->hasExternalLinkage() &&
+      GV->isDeclaration() && isa<GlobalVariable>(GV))
+    return X86II::MO_COFFSTUB;
----------------
I guess this condition also should be changed similarly, replacing `GV->hasExternalLinkage() && GV->isDeclaration()` with `GV->isDeclarationForLinker()`? It doesn't seem to change any of the existing testcases here in LLVM though.


https://reviews.llvm.org/D51288