Opened 3 months ago

Closed 2 weeks ago

Last modified 2 weeks ago

#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:

  1. Install Xcode 16.0 beta 3 (16A5202i) from Apple;
  2. Install port clang-16, clang-17, clang-18;
  3. Run port build highway, and got Crash Report from clang.
  4. Change compiler, rebuild highway, crash again.

Workaround:

  1. Use port clang-15, or
  2. 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 in reply to:  description ; Changed 3 months ago by jmroot (Joshua Root)

Replying to thelastlin:

  1. Install port clang-16, clang-17, clang-18;

From binaries, or build from source with the beta toolchain?

comment:2 in reply to:  1 Changed 3 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 Changed 3 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 3 weeks ago by i0ntempest

Cc: i0ntempest added

comment:5 Changed 3 weeks ago by aslakr (Aslak Raanes)

Cc: aslakr added

comment:6 Changed 2 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 in reply to:  3 Changed 2 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 2 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 2 weeks ago by jmroot (Joshua Root)

Owner: set to jmroot
Resolution: fixed
Status: newclosed

In 9c921e4504b3c6b9c357828e8c739789edaf27a7/macports-ports (master):

highway: disable SVE to avoid ICE

Closes: #70345

comment:10 Changed 2 weeks ago by ryandesign (Ryan Carsten Schmidt)

Cc: lukaso added
Summary: ICE while building google/highway with Apple Clang >= 16 and LLVM Clang >= 16highway @1.2.0: Compiler crash with Apple Clang >= 16 and LLVM Clang >= 16

Has duplicate #70879.

Note: See TracTickets for help on using tickets.