Open
Conversation
Begin working on revising the address space proposal. This mostly reorders the text and adds notes for work to be done. Signed-off-by: Ben Woodard <woodard@redhat.com>
* Change the name of the operator * Remove implicit pointer stuff * Remove CFA/CFI stuff deferred until the CFA/CFI proposal. * Notable rewriting. Signed-off-by: Ben Woodard <woodard@redhat.com>
Too many changes to describe fully. Signed-off-by: Ben Woodard <woodard@redhat.com>
Cary's email and Tony's response both made some important points which need to be discussed in the group as a whole. Signed-off-by: Ben Woodard <woodard@redhat.com>
Fixed a few grammar and spelling mistakes. Changed the way I presented tables in md Signed-off-by: Ben Woodard <woodard@redhat.com>
ghost
reviewed
Dec 14, 2025
Fix several bugs that Baris pointed out. Signed-off-by: Ben Woodard <woodard@redhat.com>
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
ccoutant
reviewed
Dec 17, 2025
t-tye
reviewed
Dec 17, 2025
Address comments and questions that were discussed in the Dec 17 GPU meeting as well as comments that Tony entered in the PR. Signed-off-by: Ben Woodard <woodard@redhat.com>
I missed a few later breaking comments in the PR. Added those changes. Signed-off-by: Ben Woodard <woodard@redhat.com>
t-tye
reviewed
Dec 19, 2025
Signed-off-by: Ben Woodard <woodard@redhat.com>
t-tye
suggested changes
Jan 10, 2026
Contributor
t-tye
left a comment
t-tye
left a comment
There was a problem hiding this comment.
Read through and suggested a collection of changes, but overall I really like this:-)
Tony did a careful read through and picked up some minor errors. I fixed most of them. Only two are left, I believe that both of those require discussion. Signed-off-by: Ben Woodard <woodard@redhat.com>
Fix action item before meeting. Add some notes to a problematic paragraph. Signed-off-by: Ben Woodard <woodard@redhat.com>
ghost
reviewed
Jan 14, 2026
Baris pointed out several problems on the PR. This attempts to address most of them. Signed-off-by: Ben Woodard <woodard@redhat.com>
ghost
reviewed
Jan 16, 2026
1. This fixes a couple of minor bugs that Baris found. 2. It also rearranges where I describe the problems with xderef 3. However, the biggest change is rewriting the problematic paragraph describing DW_AT_address_space/ Signed-off-by: Ben Woodard <woodard@redhat.com>
palves
reviewed
Jan 28, 2026
Baris provided additional feedback and I applied many of his suggested changes. Signed-off-by: Ben Woodard <woodard@redhat.com>
As discussed in the meeting, the algorithmic description about how the size changes are handled when A doesn't match the size of the address space is replaced with simpler text. Signed-off-by: Ben Woodard <woodard@redhat.com>
ghost
reviewed
Feb 9, 2026
Applied all of Baris's suggestions from his final review. Fixed the verb agreement with undo vs. undoes in the introductory text. Adjusted the description of DW_OP_mem to address Tony's concern. Signed-off-by: Ben Woodard <woodard@redhat.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
With the latest changes. I feel like it is ready to review. Turning it into a PR also allows people to comment on the patch.