[PATCH] D29843: [CodeGen] Treat auto-generated __dso_handle symbol as HiddenVisibility

Joerg Sonnenberger via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Feb 11 13:28:03 PST 2017


joerg requested changes to this revision.
joerg added inline comments.
This revision now requires changes to proceed.


================
Comment at: lib/CodeGen/CodeGenModule.cpp:2402
+  return GetOrCreateLLVMGlobal(Name, llvm::PointerType::getUnqual(Ty), nullptr,
+                               NotForDefinition, Hidden);
 }
----------------
Can't this use the VarDecl argument of GetOrCreateLLVMGlobal to set the visibility? I'd prefer to keep reundant arguments to a minimum.


https://reviews.llvm.org/D29843





More information about the cfe-commits mailing list