Skip to content
Merged
Changes from 5 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .github/workflows/containers/libc/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,10 @@ RUN wget https://apt.llvm.org/llvm.sh && \
RUN apt-get install -y llvm-23-tools && \
ln -s -t /usr/bin $(dpkg -L llvm-23-tools | grep '^/usr/lib/llvm-23/bin/')

# llvm-link is required to build bitcode library for GPU targets
RUN if [ ! -e /usr/bin/llvm-link ]; then \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can drop the if statement and only keep the ln -s command, and also fold this into the previous RUN line.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, @boomanaiden154
Done.
Thanks very much.

ln -s /usr/lib/llvm-23/bin/llvm-link /usr/bin/llvm-link; \
fi

# Install gmp and mpfr for cross compilation
RUN wget https://ftp.gnu.org/gnu/gmp/gmp-6.3.0.tar.xz && \
Expand Down
Loading