#70345 closed defect (fixed)
highway @1.2.0: Compiler crash with Apple Clang >= 16 and LLVM Clang >= 16
| Reported by: | thelastlin (Xavier Hsinyuan) | Owned by: | jmroot (Joshua Root) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 2.9.3 |
| Keywords: | haspatch sonoma sequoia | Cc: | i0ntempest, aslakr (Aslak Raanes), Ygar-Vanaxe-Tsy-xtledron (YVT), Schamschula (Marius Schamschula), jmroot (Joshua Root), lukaso (Lukas Oberhuber) |
| Port: | highway |
Description
Reproduce steps:
- Install Xcode 16.0 beta 3 (16A5202i) from Apple;
- Install port clang-16, clang-17, clang-18;
- Run
port build highway, and got Crash Report from clang. - Change compiler, rebuild highway, crash again.
Workaround:
- Use port clang-15, or
- Disable dynamic dispatch (upstream PR) entirely or just for clang-16 and later before Apple (or LLVM) fixes the ICE.
Change History (10)
comment:1 follow-up: 2 Changed 4 months ago by jmroot (Joshua Root)
comment:2 Changed 4 months ago by thelastlin (Xavier Hsinyuan)
Replying to jmroot:
From binaries, or build from source with the beta toolchain?
Build from source with the beta toolchain.
Notice that this issue can be reproduced on clang-17(amd64) from LLVM's Debian/Ubuntu Packages Repo, with preprocessed source and associated run script generated by crash reproducer.
comment:3 follow-up: 7 Changed 7 weeks ago by i0ntempest
There's a patch here: https://github.com/google/highway/issues/2317 that disables SVE compilation and fixes this. SVE isn't supported on ASi for now anyways.
Maybe we can take that patch before they have a new version?
comment:4 Changed 7 weeks ago by i0ntempest
| Cc: | i0ntempest added |
|---|
comment:5 Changed 7 weeks ago by aslakr (Aslak Raanes)
| Cc: | aslakr added |
|---|
comment:6 Changed 7 weeks ago by ryandesign (Ryan Carsten Schmidt)
| Cc: | Ygar-Vanaxe-Tsy-xtledron Schamschula jmroot added |
|---|---|
| Keywords: | haspatch sonoma sequoia added |
Has duplicate #70777.
comment:7 Changed 7 weeks ago by jmroot (Joshua Root)
Replying to i0ntempest:
Maybe we can take that patch before they have a new version?
I was hoping for at least a reviewed and CI-tested PR upstream so there would be a little more assurance that the change is correct, but there's been nothing so far. I guess we can try the patch on spec.
comment:8 Changed 7 weeks ago by jmroot (Joshua Root)
The PR linked from the ticket description was already merged and is in 1.2.0, BTW.
comment:9 Changed 7 weeks ago by jmroot (Joshua Root)
| Owner: | set to jmroot |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
comment:10 Changed 7 weeks ago by ryandesign (Ryan Carsten Schmidt)
| Cc: | lukaso added |
|---|---|
| Summary: | ICE while building google/highway with Apple Clang >= 16 and LLVM Clang >= 16 → highway @1.2.0: Compiler crash with Apple Clang >= 16 and LLVM Clang >= 16 |
Has duplicate #70879.
Replying to thelastlin:
From binaries, or build from source with the beta toolchain?