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.

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

Add location order to the definition of from-reads #2402

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

hernanponcedeleon
Copy link
Contributor

We are currently discussing a problem and a potential fix with the members of the Memory Model TSG. If the fix gets accepted, the specification also needs to be updated to reflect this change.

@CLAassistant
Copy link

CLAassistant commented Aug 1, 2024

CLA assistant check
All committers have signed the CLA.

@Robert-Qualcomm
Copy link

Added to agenda for the next MM TSG meeting on 2024-08-29

@jeffbolznv
Copy link
Contributor

The memory model TSG signed off on this today. The Vulkan WG should sign off on it next week.

@oddhack
Copy link
Contributor

oddhack commented Sep 26, 2024

The memory model TSG signed off on this today. The Vulkan WG should sign off on it next week.

I happened to notice this in passing, but for future reference, I sweep new issues and PRs/MRs each week and classify things for the maintenance call or for one of the TSGs. I don't revisit old issues that have previously been assigned to a TSG just to see if something changed, though. In that case either the TSG can just sign off if it's within their remit, or ping me (or bring it up on the teleconference) if the entire WG should also sign off. I will put this on the next maintenance agenda.

@oddhack oddhack added this to the Signed-off to Merge milestone Oct 9, 2024
@oddhack oddhack merged commit 2ff3b67 into KhronosGroup:main Oct 10, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants