Opened 2 years ago
Closed 2 years ago
#19968 closed enhancement (fixed)
llvm-19.1.0
| Reported by: | Bruce Dubbs | Owned by: | Xi Ruoyao |
|---|---|---|---|
| Priority: | normal | Milestone: | 12.3 |
| Component: | BOOK | Version: | git |
| Severity: | trivial | Keywords: | |
| Cc: |
Description (last modified by )
New major version.
Change History (17)
comment:2 by , 2 years ago
| Summary: | llvm-18.1.8 → llvm-18.1.8 (Wait for next version) |
|---|
comment:3 by , 2 years ago
| Milestone: | 12.2 → 99-Waiting |
|---|
follow-up: 5 comment:4 by , 2 years ago
Why not just update to .8? It doesn't need any changes and you could close 2 tickets? libclc being the other.
follow-ups: 6 7 comment:5 by , 2 years ago
Replying to Joe Locash:
Why not just update to .8? It doesn't need any changes and you could close 2 tickets? libclc being the other.
Because the odds of a newer version before we go into package freeze around August 15 is relatively high. If so, we would need to do it twice. If not then we still only need to do it once.
comment:6 by , 2 years ago
Replying to Bruce Dubbs:
Replying to Joe Locash:
Why not just update to .8? It doesn't need any changes and you could close 2 tickets? libclc being the other.
Because the odds of a newer version before we go into package freeze around August 15 is relatively high. If so, we would need to do it twice. If not then we still only need to do it once.
I don't understand? The only change would be to packages.ent That is updated daily.
We really need to test build it, run a few tests, update package.ent, changelog, and at least the md5sum in the llvm chapter. To me that cost in time is more than the benefit is worth.
comment:7 by , 2 years ago
Replying to Bruce Dubbs:
Replying to Joe Locash:
Why not just update to .8? It doesn't need any changes and you could close 2 tickets? libclc being the other.
Because the odds of a newer version before we go into package freeze around August 15 is relatively high. If so, we would need to do it twice. If not then we still only need to do it once.
Hmm... The release date for llvm-19.1.0 is Sep 3.
comment:8 by , 2 years ago
| Milestone: | 99-Waiting → 12.3 |
|---|
comment:9 by , 2 years ago
| Milestone: | 12.3 → 99-Waiting |
|---|
comment:10 by , 2 years ago
| Milestone: | 99-Waiting → 12.3 |
|---|---|
| Summary: | llvm-18.1.8 (Wait for next version) → llvm-19.1.0 |
comment:11 by , 2 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:12 by , 2 years ago
| Description: | modified (diff) |
|---|
comment:13 by , 2 years ago
comment:14 by , 2 years ago
As expected, rustc needs to be adapted:
warning: rustc_llvm@0.0.0: llvm-wrapper/RustWrapper.cpp: In function ‘LLVMOpaqueValue* LLVMRustGetInstrProfMCDCCondBitmapIntrinsic(LLVMModuleRef)’: warning: rustc_llvm@0.0.0: llvm-wrapper/RustWrapper.cpp:1562:35: 错误:‘instrprof_mcdc_condbitmap_update’ is not a member of ‘llvm::Intrinsic’; did you mean ‘instrprof_mcdc_tvbitmap_update’? warning: rustc_llvm@0.0.0: 1562 | unwrap(M), llvm::Intrinsic::instrprof_mcdc_condbitmap_update)); warning: rustc_llvm@0.0.0: | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ warning: rustc_llvm@0.0.0: | instrprof_mcdc_tvbitmap_update error: failed to run custom build command for `rustc_llvm v0.0.0 (/home/xry111/sources/12.3/rustc-1.81.0-src/compiler/rustc_llvm)`
comment:15 by , 2 years ago
To be tested:
SPIRV-LLVM-Translator-19.1.0libclc-19.1.0mesa-24.2.3(NVK disabled)
Rustc-1.81.0librsvg-2.58.4spidermonkey-128.0snapshot-47.0.1ruby-3.3.5cbindgen-0.27.0firefox-128.2.0thunderbird-128.2.0- seamonkey
rust-bindgen-0.70.1mesa (NVK compile test)
comment:16 by , 2 years ago
New Clang does not like seamonkey:
1:03.84 /home/xry111/sources/12.3/seamonkey-2.53.18.2/js/src/threading/ExclusiveData.h:124:33: error: reference to non-static member function must be called 1:03.85 124 | : lock_(mozilla::Move(rhs.lock)) 1:03.85 | ~~~~^~~~ 1:04.19 1 error generated.
With 2.53.19 it's slightly different but still rejected:
1:29.35 /home/xry111/sources/12.3/seamonkey-2.53.19/js/src/threading/ExclusiveData.h:124:29: error: reference to non-static member function must be called 1:29.35 124 | : lock_(std::move(rhs.lock)) 1:29.35 | ~~~~^~~~
comment:17 by , 2 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
- r12.2-192-g3e711549ea Update to libclc-19.1.0
- r12.2-191-ge5d0a32824 Update to SPIRV-LLVM-Translator-19.1.0
- r12.2-190-gd68c966868 Update to LLVM-19.1.0
- r12.2-189-gce271d7314 seamonkey: Fix a bug highlighted by Clang 19
- r12.2-188-ged84c58003 rust: Adapt for LLVM 19

__recommend() + 1by__endian_factorWe don't install libc++ so we can skip the release or just do a stat change.