Skip to content

Fix memory order inconsistency with array store #406

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
Merged
merged 1 commit into from
May 29, 2025
Merged

Fix memory order inconsistency with array store #406

merged 1 commit into from
May 29, 2025

Conversation

dmlloyd
Copy link
Member

@dmlloyd dmlloyd commented May 29, 2025

No description provided.

@dmlloyd dmlloyd added this to the 2.0.0 milestone May 29, 2025
@dmlloyd dmlloyd added the 2.x Issue applies to Gizmo 2.x label May 29, 2025
@dmlloyd dmlloyd moved this to In Progress in WG - Gizmo 2 May 29, 2025
@dmlloyd
Copy link
Member Author

dmlloyd commented May 29, 2025

Also add tests.

Ladicek
Ladicek approved these changes May 29, 2025
View reviewed changes
Copy link
Contributor

@Ladicek Ladicek left a comment

Choose a reason for hiding this comment

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

I noticed this, but wasn't sure if that's a limitation of arrays or just an omission. I didn't follow through on that, though. Thanks!

@Ladicek Ladicek merged commit 81b9345 into quarkusio:main May 29, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in WG - Gizmo 2 May 29, 2025
@dmlloyd dmlloyd deleted the order branch May 29, 2025 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ladicek Ladicek Ladicek approved these changes

Assignees
No one assigned
Labels
2.x Issue applies to Gizmo 2.x
Projects
WG - Gizmo 2
Status: Done
Milestone
2.0.0.Beta1
Development

Successfully merging this pull request may close these issues.
2 participants
@dmlloyd @Ladicek