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

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 2 03:39:19 PDT 2018


hans accepted this revision.
hans added a comment.
This revision is now accepted and ready to land.

Assuming the test still passes when you put the EXPORTINLINE filecheck prefix back, looks good to me!



================
Comment at: clang/test/CodeGenCXX/dllexport-no-dllexport-inlines.cpp:9
+// RUN:     -emit-llvm -O1 -o - |                                       \
+// RUN:     FileCheck --check-prefix=CHECK %s
+
----------------
Did the "EXPORTINLINE" check-prefix get lost here?


https://reviews.llvm.org/D51340





More information about the cfe-commits mailing list