Skip to content

[SPEC] Define arch/os portability flag for riscv64-linux-gnu #158

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

Merged
merged 1 commit into from
Oct 26, 2024

Conversation

lukel97
Copy link
Contributor

@lukel97 lukel97 commented Sep 6, 2024

Without -DSPEC_MANUAL_CONFIG (or another dummy arch/os flag specified), 500.perlbench_r will fail to build on riscv64-linux-gnu. -DSPEC_LP64 and -DSPEC_LINUX are also already defined via the generic handlers above. I'm not sure if we need to define more flags, but this enough to get things building.

Without -DSPEC_MANUAL_CONFIG (or another dummy arch/os flag specified), 500.perlbench_r will fail to build on riscv64-linux-gnu. -DSPEC_LP64 and -DSPEC_LINUX are also already defined via the generic handlers above. I'm not sure if we need to define more flags, but this enough to get things building.
Copy link

@asb asb left a comment

Choose a reason for hiding this comment

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

It doesn't compile out of the box without this, LGTM!

@lukel97 lukel97 merged commit d026e3d into llvm:main Oct 26, 2024
1 check passed
SixWeining added a commit that referenced this pull request Nov 27, 2024
Similar to #158. Without this change, 500.perlbench_r will fail to
build on LoongArch.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants