[PATCH] D51340: Add /Zc:DllexportInlines option to clang-cl

Takuto Ikuta via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 10 02:32:18 PDT 2018


takuto.ikuta added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:5244
+                   false))
+    CmdArgs.push_back("-fvisibility-inlines-hidden");
+
----------------
hans wrote:
> Huh, does this actually affect whether functions get dllexported or not?
Sorry, what you want to ask?

This will used to not add dllexport attr in L5690 of SemaDeclCXX.cpp.



https://reviews.llvm.org/D51340





More information about the cfe-commits mailing list