Skip to content
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

Provide a detailed message for invalid arch in target triple #21921

Merged
merged 2 commits into from
Nov 6, 2024
Merged

Provide a detailed message for invalid arch in target triple #21921

merged 2 commits into from
Nov 6, 2024

Conversation

danielchasehooper
Copy link
Contributor

@danielchasehooper danielchasehooper commented Nov 5, 2024 •
edited
Loading

adds arch error message. Matches format of invalid cpu and object format messages.

example:

$ zig build-exe -target baz-macos main.zig
info: available architectures:
 amdgcn
 arc
 arm
 armeb
 thumb
 thumbeb
 aarch64
 aarch64_be
 avr
 bpfel
 bpfeb
 csky
 hexagon
 kalimba
 lanai
 loongarch32
 loongarch64
 m68k
 mips
 mipsel
 mips64
 mips64el
 msp430
 nvptx
 nvptx64
 powerpc
 powerpcle
 powerpc64
 powerpc64le
 propeller1
 propeller2
 riscv32
 riscv64
 s390x
 sparc
 sparc64
 spirv
 spirv32
 spirv64
 spu_2
 ve
 wasm32
 wasm64
 x86
 x86_64
 xcore
 xtensa
 native

error: unknown architecture: 'baz'

@andrewrk
Copy link
Member

andrewrk commented Nov 6, 2024

Nice, thanks!

@andrewrk andrewrk enabled auto-merge (squash) November 6, 2024 02:32
@andrewrk andrewrk merged commit ed04acf into ziglang:master Nov 6, 2024
10 checks passed
@danielchasehooper danielchasehooper deleted the target_diagnostics branch November 6, 2024 03:55
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
@danielchasehooper @andrewrk