Skip to content

stage2 ARM: lower const slices#10868

Merged
kubkon merged 1 commit intoziglang:masterfrom
joachimschmidt557:stage2-arm
Feb 11, 2022
Merged

stage2 ARM: lower const slices#10868
kubkon merged 1 commit intoziglang:masterfrom
joachimschmidt557:stage2-arm

Conversation

@joachimschmidt557
Copy link
Copy Markdown
Contributor

Follow-up to e1a5353 (#10862) for ARM

This also fixes some stack offset calculation bugs

cc @kubkon

@kubkon
Copy link
Copy Markdown
Member

kubkon commented Feb 11, 2022

Is the CI failure related by any chance? It didn't happen before 🤔

@joachimschmidt557
Copy link
Copy Markdown
Contributor Author

Is the CI failure related by any chance? It didn't happen before 🤔

I currently don't see how it could be related, the error is in a std test for Mutex.

Follow-up to e1a5353 for ARM

This also fixes some stack offset calculation bugs
@kubkon
Copy link
Copy Markdown
Member

kubkon commented Feb 11, 2022

Hmm, it passes now which makes me think we might have some hidden race condition somewhere.

@joachimschmidt557
Copy link
Copy Markdown
Contributor Author

Hmm, it passes now which makes me think we might have some hidden race condition somewhere.

Should I open an issue for this? I still have the link to the "original" CI failure here: https://ci.ziglang.org/ziglang/zig/2278/1/4

@kubkon
Copy link
Copy Markdown
Member

kubkon commented Feb 11, 2022

Yes please!

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