Skip to content

Commit 064d55b

Browse files
committed
readme: Update LLVM/CIRCT build instructions
Don't install LLVM/CIRCT to a local `install` directory, as this tends to break things in subtle ways. Will need a fix for this eventually, but not today.
1 parent 6052a23 commit 064d55b

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

README.md

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ And then follow these steps:
2424
pushd circt/llvm/build
2525
cmake ../llvm \
2626
-DCMAKE_BUILD_TYPE=Release \
27-
-DCMAKE_INSTALL_PREFIX=../install \
2827
-DLLVM_BUILD_EXAMPLES=OFF \
2928
-DLLVM_ENABLE_ASSERTIONS=ON \
3029
-DLLVM_ENABLE_BINDINGS=OFF \
@@ -33,7 +32,7 @@ And then follow these steps:
3332
-DLLVM_INSTALL_UTILS=ON \
3433
-DLLVM_OPTIMIZED_TABLEGEN=ON \
3534
-DLLVM_TARGETS_TO_BUILD="host"
36-
cmake --build . --target install
35+
cmake --build .
3736
popd
3837

3938
#### Build CIRCT
@@ -42,21 +41,20 @@ And then follow these steps:
4241
pushd circt/build
4342
cmake .. \
4443
-DCMAKE_BUILD_TYPE=Release \
45-
-DCMAKE_INSTALL_PREFIX=../install \
46-
-DMLIR_DIR=$PWD/../llvm/install/lib/cmake/mlir \
47-
-DLLVM_DIR=$PWD/../llvm/install/lib/cmake/llvm \
44+
-DMLIR_DIR=$PWD/../llvm/build/lib/cmake/mlir \
45+
-DLLVM_DIR=$PWD/../llvm/build/lib/cmake/llvm \
4846
-DLLVM_ENABLE_ASSERTIONS=ON
49-
cmake --build . --target install
47+
cmake --build .
5048
popd
5149

5250
#### Build Moore
5351

5452
Set the following environment variables to indicate where your LLVM and CIRCT build is:
5553

5654
export CIRCT_SYS_CIRCT_DIR=$PWD/circt
57-
export CIRCT_SYS_CIRCT_BUILD_DIR=$PWD/circt/install
55+
export CIRCT_SYS_CIRCT_BUILD_DIR=$PWD/circt/build
5856
export CIRCT_SYS_LLVM_DIR=$PWD/circt/llvm
59-
export CIRCT_SYS_LLVM_BUILD_DIR=$PWD/circt/llvm/install
57+
export CIRCT_SYS_LLVM_BUILD_DIR=$PWD/circt/llvm/build
6058

6159
Use cargo to install Moore:
6260

0 commit comments

Comments
 (0)