Skip to content

Commit

Permalink
Remove LLVM_ABI_BREAKING_CHECKS=FORCE_OFF.
Browse files Browse the repository at this point in the history
The issue raised in #4212 (comment) and temporary fixed in #4512 has now been fixed upstream.

The non-deterministic seed inside the hashing function
(included in this llvm commit: llvm/llvm-project@ce80c80) was disabled for clang <= 11 for non-pic builds.

This was removed again in xgupta/llvm-project@5255b81 and we can enable ABI-breaking checks again.

We build LLVM with PIC. If anyone ever builds with non-PIC, they will get a compilation error and need to disable LLVM_ABI_BREAKING_CHECKS manually.
  • Loading branch information
karupayun authored and chsigg committed Oct 7, 2024
1 parent ab07e54 commit 78050f1
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 4 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/llvm-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,6 @@ jobs:
-DLLVM_INSTALL_UTILS=ON
-DLLVM_TARGETS_TO_BUILD="host;NVPTX;AMDGPU"
-DLLVM_ENABLE_TERMINFO=OFF
-DLLVM_ABI_BREAKING_CHECKS=FORCE_OFF
llvm-project/llvm
ninja -C llvm-project/build check-mlir install
Expand All @@ -131,7 +130,6 @@ jobs:
-DLLVM_INSTALL_UTILS=ON
-DLLVM_TARGETS_TO_BUILD="host;NVPTX;AMDGPU"
-DLLVM_ENABLE_TERMINFO=OFF
-DLLVM_ABI_BREAKING_CHECKS=FORCE_OFF
llvm-project/llvm
ninja -C llvm-project/build check-mlir install
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/llvm-build/almalinux.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ RUN cmake -GNinja -Bbuild \
-DLLVM_ENABLE_PROJECTS=mlir \
-DLLVM_ENABLE_TERMINFO=OFF \
-DLLVM_INSTALL_UTILS=ON \
-DLLVM_ABI_BREAKING_CHECKS=FORCE_OFF \
-DLLVM_TARGETS_TO_BUILD="host;NVPTX;AMDGPU" \
/source/llvm-project/llvm

Expand Down
2 changes: 1 addition & 1 deletion cmake/llvm-hash.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
61f8a7f618901797ee8663389a29722f29216a96
5255b819c1b117e6048f354c035f5d36f3afad2b

0 comments on commit 78050f1

Please sign in to comment.