docs: small improvements to build instructions

Signed-off-by: Ingo Müller <[email protected]>
substrait-io · Nov 29, 2024 · 936f519 · 936f519
1 parent 0502182
commit 936f519
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.github/workflows/build_and_test.yml b/.github/workflows/build_and_test.yml
@@ -72,7 +72,7 @@ jobs:
           -DCMAKE_C_COMPILER:FILEPATH=clang \
           -DCMAKE_CXX_COMPILER:FILEPATH=clang++ \
           -DLLVM_ENABLE_PROJECTS="mlir;clang;clang-tools-extra" \
-          -DLLVM_TARGETS_TO_BUILD="X86" \
+          -DLLVM_TARGETS_TO_BUILD="Native" \
           -DLLVM_ENABLE_ASSERTIONS=ON \
           -DLLVM_INCLUDE_TESTS=OFF \
           -DLLVM_INCLUDE_UTILS=ON \

diff --git a/README.md b/README.md
@@ -201,7 +201,7 @@ by CMake:
 * [`ninja`](<https://ninja-build.org/>)
 * [LLVM prerequisites](https://llvm.org/docs/GettingStarted.html#software) and a
   [C/C++ toolchain](https://llvm.org/docs/GettingStarted.html#host-c-toolchain-both-compiler-and-standard-library)
-* Protobuf (compiler, runtime, and headers)
+* Protobuf >= 3.12 (compiler, runtime, and headers)
 
 ### Define Paths
 
@@ -272,7 +272,7 @@ cmake \
   -DCMAKE_EXPORT_COMPILE_COMMANDS=TRUE \
   -DCMAKE_BUILD_TYPE=Debug \
   -DLLVM_ENABLE_PROJECTS="mlir;clang;clang-tools-extra" \
-  -DLLVM_TARGETS_TO_BUILD="X86" \
+  -DLLVM_TARGETS_TO_BUILD="Native" \
   -DLLVM_ENABLE_ASSERTIONS=ON \
   -DLLVM_INCLUDE_TESTS=OFF \
   -DLLVM_INCLUDE_UTILS=ON \