Author: phosek Date: Thu May 16 23:07:37 2019 New Revision: 360991 URL: http://llvm.org/viewvc/llvm-project?rev=360991&view=rev Log: [Analysis] Only run plugins tests if plugins are actually enabled
When plugins aren't enabled, don't try to run plugins tests. Don't enable plugins unconditionally based on the platform, instead check if LLVM shared library is actually being built which may not be the case for every host configuration, even if the host itself supports plugins. This addresses test failures introduced by r360891/D59464. Differential Revision: https://reviews.llvm.org/D62050 Modified: cfe/trunk/test/Analysis/checker-plugins.c cfe/trunk/test/CMakeLists.txt cfe/trunk/test/lit.cfg.py cfe/trunk/test/lit.site.cfg.py.in Modified: cfe/trunk/test/Analysis/checker-plugins.c URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Analysis/checker-plugins.c?rev=360991&r1=360990&r2=360991&view=diff ============================================================================== --- cfe/trunk/test/Analysis/checker-plugins.c (original) +++ cfe/trunk/test/Analysis/checker-plugins.c Thu May 16 23:07:37 2019 @@ -1,9 +1,9 @@ +// REQUIRES: plugins + // RUN: %clang_analyze_cc1 -verify %s \ // RUN: -load %llvmshlibdir/SampleAnalyzerPlugin%pluginext \ // RUN: -analyzer-checker='example.MainCallChecker' -// REQUIRES: plugins - // Test that the MainCallChecker example analyzer plugin loads and runs. int main(); Modified: cfe/trunk/test/CMakeLists.txt URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CMakeLists.txt?rev=360991&r1=360990&r2=360991&view=diff ============================================================================== --- cfe/trunk/test/CMakeLists.txt (original) +++ cfe/trunk/test/CMakeLists.txt Thu May 16 23:07:37 2019 @@ -24,7 +24,8 @@ llvm_canonicalize_cmake_booleans( CLANG_ENABLE_STATIC_ANALYZER ENABLE_BACKTRACES HAVE_LIBZ - LLVM_ENABLE_PER_TARGET_RUNTIME_DIR) + LLVM_ENABLE_PER_TARGET_RUNTIME_DIR + LLVM_ENABLE_PLUGINS) configure_lit_site_cfg( ${CMAKE_CURRENT_SOURCE_DIR}/lit.site.cfg.py.in Modified: cfe/trunk/test/lit.cfg.py URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/lit.cfg.py?rev=360991&r1=360990&r2=360991&view=diff ============================================================================== --- cfe/trunk/test/lit.cfg.py (original) +++ cfe/trunk/test/lit.cfg.py Thu May 16 23:07:37 2019 @@ -85,13 +85,7 @@ config.substitutions.append( os.path.join(config.clang_tools_dir, 'hmaptool')))) # Plugins (loadable modules) -# TODO: This should be supplied by Makefile or autoconf. -if sys.platform in ['win32', 'cygwin']: - has_plugins = config.enable_shared -else: - has_plugins = True - -if has_plugins and config.llvm_plugin_ext: +if config.has_plugins and config.llvm_plugin_ext: config.available_features.add('plugins') # Set available features we allow tests to conditionalize on. Modified: cfe/trunk/test/lit.site.cfg.py.in URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/lit.site.cfg.py.in?rev=360991&r1=360990&r2=360991&view=diff ============================================================================== --- cfe/trunk/test/lit.site.cfg.py.in (original) +++ cfe/trunk/test/lit.site.cfg.py.in Thu May 16 23:07:37 2019 @@ -27,6 +27,7 @@ config.enable_backtrace = @ENABLE_BACKTR config.host_arch = "@HOST_ARCH@" config.python_executable = "@PYTHON_EXECUTABLE@" config.use_z3_solver = lit_config.params.get('USE_Z3_SOLVER', "@USE_Z3_SOLVER@") +config.has_plugins = @LLVM_ENABLE_PLUGINS@ # Support substitution of the tools and libs dirs with user parameters. This is # used when we can't determine the tool dir at configuration time. _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
