Skip to content

Make ARM64EC build available with NEON intrinsics #1897

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.

Sign up for GitHub

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

Jump to bottom
Merged
jrouwe merged 2 commits into from
Jan 29, 2026
Merged

Make ARM64EC build available with NEON intrinsics #1897

jrouwe merged 2 commits into from
Jan 29, 2026
+18 −14

Conversation

@vinod-sv
Copy link
Contributor

@vinod-sv vinod-sv commented Jan 29, 2026 •
edited
Loading

  1. A batch file to make ARM64EC project
  2. Make use of NEON intrinsics for ARM64EC ABI.
  3. MSVC compiler causes internal compiler error in AABox4VsBox functions. It gets mitigated only if 'g' optimisation is disabled. A separate complaint is raised for Microsoft team to look into this compiler error. Until Microsoft resolves this compiler error, it is handled.

@vinod-sv
1. A batch file to make ARM64EC project
e1b092a 
2. Make use of NEON intrinsics for ARM64EC ABI.
3. MSVC compiler causes internal compiler error in AABox4VsBox functions. It gets mitigated only if 'g' optimisation is disabled. A separate complaint is raised for Microsoft team to look into this compiler error. Until Microsoft resolves this compiler error, it is handled.
@vinod-sv vinod-sv changed the title 1. A batch file to make ARM64EC project Make ARM64EC build available with NEON intrinsics Jan 29, 2026
@jrouwe
Copy link
Owner

jrouwe commented Jan 29, 2026 •
edited
Loading

  • The internal compiler error is due to link time code generation just like the other ARM versions. The #pragma is not needed.
  • I didn't find any documentation online about _ARM64EC_ and it also wasn't handled in the #if below it, so I removed it.
  • Your change was for an older version of Jolt, JPH_CPU_ADDRESS_BITS has been renamed to JPH_CPU_ARCH_BITS

Repository owner deleted a comment from CLAassistant Jan 29, 2026
@jrouwe
Copy link
Owner

jrouwe commented Jan 29, 2026

A separate complaint is raised for Microsoft team to look into this compiler error.

Can you link the issue here? (I couldn't find it and I don't remember if I ever filed a ticket for this)

@jrouwe jrouwe merged commit 757c6b7 into jrouwe:master Jan 29, 2026
71 of 72 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vinod-sv @jrouwe