Skip to content

Commit ea3c6b2

Browse files
committed
Update release notes for PR #5041
1 parent 88cebb0 commit ea3c6b2

2 files changed

Lines changed: 6 additions & 5 deletions

File tree

.release-notes/fix-tool-arch.md

Lines changed: 0 additions & 5 deletions
This file was deleted.

.release-notes/next-release.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,3 +12,9 @@ The standard library is now always searched first, before any `--path` entries.
1212

1313
If you were relying on `--path` to override a standard library package, that will no longer work.
1414

15+
## Fix illegal instruction crashes in pony-lint, pony-lsp, and pony-doc
16+
17+
The tool build commands didn't pass `--cpu` to ponyc, so ponyc targeted the build machine's specific CPU via `LLVMGetHostCPUName()`. Release builds happen on CI machines that often have newer CPUs than user machines. When a user ran pony-lint, pony-lsp, or pony-doc on an older CPU, the binary could contain instructions their CPU doesn't support, resulting in a SIGILL crash.
18+
19+
The C/C++ side of the build already respected `PONY_ARCH` (e.g., `arch=x86-64` targets the baseline x86-64 instruction set). The Pony code generation for the tools just wasn't wired up to use it. Now it is.
20+

0 commit comments

Comments
 (0)