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

2018-09-18 Thread Takuto Ikuta via Phabricator via cfe-commits
takuto.ikuta updated this revision to Diff 165966. takuto.ikuta edited the summary of this revision. takuto.ikuta added a comment. Current implementation cannot build chrome when pch is enabled. undefined symbol error happens during linking blink_modules.dll https://reviews.llvm.org/D51340 File

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

2018-09-12 Thread Takuto Ikuta via Phabricator via cfe-commits
takuto.ikuta added inline comments. Comment at: clang/lib/Driver/ToolChains/Clang.cpp:5244 + false)) +CmdArgs.push_back("-fvisibility-inlines-hidden"); + rnk wrote: > takuto.ikuta wrote: > > hans wrote: > > > takuto.ikuta wrote: > > > > hans

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

2018-09-11 Thread Reid Kleckner via Phabricator via cfe-commits
rnk added inline comments. Comment at: clang/lib/Driver/ToolChains/Clang.cpp:5244 + false)) +CmdArgs.push_back("-fvisibility-inlines-hidden"); + takuto.ikuta wrote: > hans wrote: > > takuto.ikuta wrote: > > > hans wrote: > > > > Huh, does th