-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docker: use pre-built mlir and cva6 toolchain #234
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I overall like this, but I'm unsure about the current versioning strategy and I don't feel like we should build gcc ourselves (do we need to?)
this is a draft @JosseVanDelm the commented code is because the container was running out of memory trying to build the system, because of many tcdm connections. not the case anymore since #254 |
32a9d45
to
94fd4fd
Compare
94fd4fd
to
dc118f5
Compare
dc118f5
to
6ae3c2c
Compare
Now open for review! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much @jorendumoulin !!!
I like this, but can we pin the versions in the llvm-build repo here as well maybe?
@@ -46,6 +45,17 @@ RUN apt-get update && apt-get install -y lsb-release wget software-properties-co | |||
# Enable break-system-packages for non-venv package installs | |||
python3 -m pip config set global.break-system-packages true | |||
|
|||
# Get prebuilt mlir | |||
# Set the URL for the download | |||
ENV MLIR_URL=https://github.com/KULeuven-MICAS/llvm-build/releases/latest/download/mlir.tar.gz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we pin the version here instead as well?
ENV MLIR_URL=https://github.com/KULeuven-MICAS/llvm-build/releases/ <version> /download/mlir.tar.gz
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK! Nice. So we can use CVA6 with this? 😄
@rgantonio from what I understand that is the current goal yes. |
When doing some initial attempts on compiling the sw, some weird errors are raised (at linking stage). The simpler software, e.g., bootrom, seems to have no problems. Now @Konste11ation is investigating it, and please do not merge after we solved all the problems. |
Maybe another solution is to establish a new runtime and compilation flow for our system... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
HeMAiA test passed! Thanks @Konste11ation !
this also adds the streamer-gemm and kul-cluster system, until we converge on the single system for all tests