From e191eb4c3c3f3144503a8a117d760de5ddcc7e89 Mon Sep 17 00:00:00 2001 From: mlevesquedion Date: Thu, 8 Feb 2024 17:18:17 -0800 Subject: [PATCH] Set Python bindings to OFF by default (#2002) Users building the project by following the README may not need the Python bindings, so we shouldn't recommend building them by default. #2001 --- README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 94120b1a92..5295b8d766 100644 --- a/README.md +++ b/README.md @@ -68,7 +68,7 @@ Here's how to build the StableHLO repo on Linux or macOS: 5. Configure and build MLIR: ```sh - MLIR_ENABLE_BINDINGS_PYTHON=ON build_tools/build_mlir.sh ${PWD}/llvm-project/ ${PWD}/llvm-build + MLIR_ENABLE_BINDINGS_PYTHON=OFF build_tools/build_mlir.sh ${PWD}/llvm-project/ ${PWD}/llvm-build ``` This will take a considerable amount of time. For example, on a MacBook Pro @@ -86,7 +86,7 @@ Here's how to build the StableHLO repo on Linux or macOS: -DLLVM_ENABLE_LLD="$LLVM_ENABLE_LLD" \ -DCMAKE_BUILD_TYPE=Release \ -DLLVM_ENABLE_ASSERTIONS=ON \ - -DSTABLEHLO_ENABLE_BINDINGS_PYTHON=ON \ + -DSTABLEHLO_ENABLE_BINDINGS_PYTHON=OFF \ -DMLIR_DIR=${PWD}/../llvm-build/lib/cmake/mlir cmake --build .