[PATCH] Don't use /implib when linking LLVM executables
Reid Kleckner
rnk at google.com
Tue Mar 24 09:17:56 PDT 2015
Is this still needed, or are we leaving liblldb alone and moving on with life?
================
Comment at: cmake/modules/HandleLLVMOptions.cmake:463
@@ -462,3 +462,3 @@
if(CYGWIN OR MINGW)
# Prune --out-implib from executables. It doesn't make sense even
----------------
I'd add a comment here saying that if we ever want to support loadable plugins on Windows, we should leave these flags alone.
Right now we don't support loadable plugins on Windows at all, so taking out these flags is fine.
http://reviews.llvm.org/D8423
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list