Skip to content

Add RISC-V 64-bit architecture detection #4525

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
Cyan4973 merged 1 commit into from
Dec 2, 2025
Merged

Add RISC-V 64-bit architecture detection #4525

Cyan4973 merged 1 commit into from
Dec 2, 2025
+1 −0

Conversation

@Polaris-911
Copy link
Contributor

@Polaris-911 Polaris-911 commented Nov 3, 2025

  • Add compile-time check for RISC-V 64-bit via __riscv and __riscv_xlen == 64
  • Align with existing 64-bit architecture detection logic (x86_64/ARM64 etc.)
  • Ensure 64BIT is defined for RISC-V 64-bit builds
@meta-cla meta-cla bot added the CLA Signed label Nov 3, 2025
@Polaris-911
Copy link
Contributor Author

Polaris-911 commented Nov 27, 2025

Hi @Cyan4973 , thanks again for your time! Would you also be able to take a look at this PR when convenient?
@Cyan4973 Cyan4973 self-assigned this Dec 2, 2025
Cyan4973
Cyan4973 approved these changes Dec 2, 2025
View reviewed changes
Copy link
Contributor

@Cyan4973 Cyan4973 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me !
@Cyan4973 Cyan4973 merged commit 92505de into facebook:dev Dec 2, 2025
105 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed

2 participants

@Polaris-911 @Cyan4973