[llvm] f71e02f - [CMake] Disable LLVM_INCLUDE_GO_TESTS by default

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 5 20:03:57 PST 2022


Author: Fangrui Song
Date: 2022-01-05T20:03:52-08:00
New Revision: f71e02fa46f8ff2e37dc2fee02a63227b823b292

URL: https://github.com/llvm/llvm-project/commit/f71e02fa46f8ff2e37dc2fee02a63227b823b292
DIFF: https://github.com/llvm/llvm-project/commit/f71e02fa46f8ff2e37dc2fee02a63227b823b292.diff

LOG: [CMake] Disable LLVM_INCLUDE_GO_TESTS by default

llvm/test/Bindings/Go is quite flaky in the past few months and nobody fixes it.

See

* https://lists.llvm.org/pipermail/llvm-dev/2021-December/154353.html "Suggestions on debugging pre-merge test failure that looks irrelevant."
* https://github.com/llvm/llvm-project/issues/53017

Reviewed By: aeubanks

Differential Revision: https://reviews.llvm.org/D116698

Added: 
    

Modified: 
    llvm/CMakeLists.txt

Removed: 
    


################################################################################
diff  --git a/llvm/CMakeLists.txt b/llvm/CMakeLists.txt
index 672183f62da0..af32d35521a3 100644
--- a/llvm/CMakeLists.txt
+++ b/llvm/CMakeLists.txt
@@ -593,7 +593,7 @@ endif(LLVM_BUILD_EXAMPLES)
 option(LLVM_BUILD_TESTS
   "Build LLVM unit tests. If OFF, just generate build targets." OFF)
 option(LLVM_INCLUDE_TESTS "Generate build targets for the LLVM unit tests." ON)
-option(LLVM_INCLUDE_GO_TESTS "Include the Go bindings tests in test build targets." ON)
+option(LLVM_INCLUDE_GO_TESTS "Include the Go bindings tests in test build targets." OFF)
 
 option(LLVM_BUILD_BENCHMARKS "Add LLVM benchmark targets to the list of default
 targets. If OFF, benchmarks still could be built using Benchmarks target." OFF)


        


More information about the llvm-commits mailing list