[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