[PATCH] D74042: [test] Disable the Passes/PluginsTest cases on windows with BUILD_SHARED_LIBS

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 5 16:20:04 PST 2020


rnk added inline comments.


================
Comment at: llvm/unittests/Passes/CMakeLists.txt:4-5
 
 # If plugins are disabled, this test will disable itself at runtime. Otherwise,
 # reconfiguring with plugins disabled will leave behind a stale executable.
 if (LLVM_ENABLE_PLUGINS)
----------------
It seems there is some prior reason this is done this way, so that reconfiguring a build directory from BUILD_SHARED_LIBS=ON to OFF and back rebuilds the executable. Otherwise lit test discovery will find it.

To this point, I think we should simply disable this whole directory on Windows. (`if (WIN32)`) One cannot switch from win32 to non-win32 in the same directory and expect things to work.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74042/new/

https://reviews.llvm.org/D74042





More information about the llvm-commits mailing list