-
Notifications
You must be signed in to change notification settings - Fork 247
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build failures under Xcode 15 #1146
Comments
Dear @timsutton ; thanks your your report. According to https://developer.arm.com/architectures/instruction-sets/intrinsics/#q=vrnd32x_f64 , that function is part of A64 ; but LLVM/clang didn't add it until version 18: llvm/llvm-project@dbeb3d0 I can confirm the build errors by adding the new macos arm64 GitHub runners to our CI: https://github.com/simd-everywhere/simde/actions/runs/8307298324 At #1148 I'm testing a fix ; can you also confirm it on your side? Can you carry a patch, or do you need a new release? |
@mr-c Thanks for the quick reply! Cherry-picking the code fixes in #1148 seem to work for me locally, Carrying a patch is no problem. If you end up landing a single commit as a patch that makes it even more straightforward for including it as part of the formula definition, then on subsequent releases we'll remove it. |
@timsutton Are you actually running the SIMDe tests, or just compiling them? I get test failures on arm64 macos: https://github.com/simd-everywhere/simde/actions/runs/8307482025/job/22736747964?pr=1148 (looks like #1099 again) |
@mr-c I think very much likely just compiling. If you're interested to see specifically what is done for the formula in Homebrew, you can see that here: https://github.com/Porkepix/homebrew-core/blob/bump-simde-0.8.0/Formula/s/simde.rb#L33-L48 Since the compile/install isn't explicitly running |
A "fun" discovery: On the m1, appleclang has different cc -dM -E -fopenmp-simd -march=native - < /dev/null
cc -dM -E -fopenmp-simd -mcpu=apple-m1 - < /dev/null
The situation is improved for Apple clang version 15.0.0 (clang-1500.1.0.2.5), where the cc -dM -E -fopenmp-simd -march=native - < /dev/null
cc -dM -E -fopenmp-simd -mcpu=apple-m1 - < /dev/null
|
Build failures fixed in #1148 ; I'll make a new issue to re-implement the FCVTZS/FCVTMS/FCVTPS/FCVTNS family intrinsics that I had to disable |
You rock! Thanks for that followup as well :) |
👋 Hi, I work on maintaining the Homebrew core package repository. In attempting to bump SIMD Everywhere to 0.8.0, we ran into a couple basic failures during its post-build tests. This is a basic
test.c
file that looks like this:we would build that sample like this, and then execute it:
But a couple errors crop up, for example:
While it's possible to sidestep the implicit function declarion by setting
-Wno-error=implicit-function-declaration
, the incompatible result type error is there.For what it's worth, this is on macOS Sonoma 14.4, Xcode 15.3, on an M3 Max laptop. Thanks for your time!
The text was updated successfully, but these errors were encountered: