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

Fix expectations for PointerAccessChains/pointer_index_is_constant_1.cl #1298

Merged
merged 1 commit into from
Feb 1, 2024

Conversation

dneto0
Copy link
Collaborator

@dneto0 dneto0 commented Feb 1, 2024

The new code is correct. The constant indexing into an array of arrays of floats collapses into a constant index of an array of floats.

Ref #1292

The new code is correct.  The constant indexing into an array of arrays
of floats collapses into a constant index of an array of floats.

Ref google#1292
@dneto0
Copy link
Collaborator Author

dneto0 commented Feb 1, 2024

cc: @rjodinchr

Copy link
Collaborator

@alan-baker alan-baker left a comment

Choose a reason for hiding this comment

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

Looks ok. Arguably it should be better as a flat array assuming the underlying data type wasn't narrowed.

@alan-baker alan-baker merged commit 545a65b into google:main Feb 1, 2024
10 of 12 checks passed
@rjodinchr
Copy link
Collaborator

Thank you @dneto0.
I think I got to this one during the end, and I was a bit confused as to whether losing the arraystride was an issue or not.
In order to unblock TAP, I decided to just mark it as failing to come back to it later.
Thanks again for taking care of it.

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.

3 participants