Skip to content

Conversation

cyyever
Copy link
Contributor

@cyyever cyyever commented Sep 27, 2025

According to the GCC documentation, the march armv8-a+fp16fml is invalid. A build failure was observed on PyTorch CI.
FYI
image

but
image

Copy link

netlify bot commented Sep 27, 2025

Deploy Preview for pytorch-fbgemm-docs ready!

Name Link
🔨 Latest commit f539119
🔍 Latest deploy log https://app.netlify.com/projects/pytorch-fbgemm-docs/deploys/68d7a5a463283c000818798f
😎 Deploy Preview https://deploy-preview-4942--pytorch-fbgemm-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@meta-cla meta-cla bot added the cla signed label Sep 27, 2025
Signed-off-by: Yuanyuan Chen <cyyever@outlook.com>
@facebook-github-bot
Copy link
Contributor

@q10 merged this pull request in a4c1ecd.

@cyyever cyyever deleted the arm_native branch September 30, 2025 06:06
facebook-github-bot pushed a commit that referenced this pull request Oct 1, 2025
Summary:
X-link: facebookresearch/FBGEMM#1977

Add a missing part in #4942 .
I also fix another aarch64 issue on PyTorch CI:
```
/var/lib/jenkins/workspace/third_party/fbgemm/include/fbgemm/./Utils.h:26:10: fatal error: arm_neon_sve_bridge.h: No such file or directory
   26 | #include <arm_neon_sve_bridge.h> // manual
      |          ^~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
```
[ARM documentation](https://developer.arm.com/documentation/110065/0100/Software-codec-optimization/Compiler-recommendations) says:

SVE and SVE2 feature paths in libaom, libvpx, and x265 require the intrinsics declared in arm_neon_sve_bridge.h. This header file is available in the following releases:

Clang 17 and later.
GCC 14 and later.

Pull Request resolved: #4948

Reviewed By: cthi

Differential Revision: D83578552

Pulled By: q10

fbshipit-source-id: 13b07cbb68ec7b233488a1958d86510b8828a52f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants