Skip to content

Conversation

glaubitz
Copy link
Contributor

This fixes the rustc build on x32 for which struct sizes differ.

This fixes the rustc build on x32 for which struct sizes differ.
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 21, 2025
@ChayimFriedman2
Copy link
Contributor

I'm confused. Is there an x86-64 platform which is 32 bits?

@lnicola
Copy link
Member

lnicola commented Jul 21, 2025

Thanks, but merging it in rust-lang/rust would have been fine, too. This will normally reach that repo next Monday or Tuesday, is that too late?

@glaubitz
Copy link
Contributor Author

I'm confused. Is there an x86-64 platform which is 32 bits?

Yes, the Rust target is x86_64-unknown-linux-gnux32. Debian calls it just "x32".

@glaubitz
Copy link
Contributor Author

Thanks, but merging it in rust-lang/rust would have been fine, too. This will normally reach that repo next Monday or Tuesday, is that too late?

I tried to send it there first, but I was sent here. See the referenced PR above.

@lnicola
Copy link
Member

lnicola commented Jul 21, 2025

I know, but that's just a bot asking nicely :-).

@ShoyuVanilla
Copy link
Member

x86_64-unknown-linux-gnux32 | ✓ | 64-bit Linux (x32 ABI) (kernel 4.15, glibc 2.27) -- | -- | --

Oh.. It's 64 bit but due to x32 ABI, it has 32 bits pointer width 🙃

@glaubitz
Copy link
Contributor Author

I know, but that's just a bot asking nicely :-).

Yes, the bot was asking nicely. But a real human closed my PR without merged it. ;-)

@lnicola lnicola added this pull request to the merge queue Jul 21, 2025
Merged via the queue into rust-lang:master with commit 0d4f56d Jul 21, 2025
15 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 21, 2025
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.

5 participants