Skip to content

Conversation

@alexrp
Copy link
Member

@alexrp alexrp commented Oct 6, 2024

This matches avr-gcc's ABI.

Note that this makes us diverge from Clang 19's AVR ABI. But we already did that in other ways for AVR, so I don't consider that a significant issue. (Right now, we're in a weird middle-ground where we don't fully match Clang or avr-gcc.) I'm working on getting Clang to match avr-gcc as well, which should hopefully make it into Clang 20.

Also worth noting:

zig/src/Type.zig

Lines 1639 to 1641 in d5c9d85

pub fn maxIntAlignment(target: std.Target, use_llvm: bool) u16 {
return switch (target.cpu.arch) {
.avr => 1,

@alexrp alexrp enabled auto-merge October 6, 2024 21:32
@alexrp alexrp merged commit aa8f39e into ziglang:master Oct 6, 2024
10 checks passed
@alexrp alexrp deleted the target-avr-align branch October 7, 2024 00:06
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.

1 participant