Skip to content

Conversation

@arsenm
Copy link
Contributor

@arsenm arsenm commented Feb 25, 2025

Comparing the case where each dimension is used alone, the only codegen difference
is a missed addressing mode fold for the constant offset in the old version due to
an ancient bug.

…_size

Comparing the case where each dimension is used alone, the only codegen difference
is a missed addressing mode fold for the constant offset in the old version due to
an ancient bug.
@arsenm arsenm added the libclc libclc OpenCL library label Feb 25, 2025 — with Graphite App
Copy link
Contributor Author

arsenm commented Feb 25, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@arsenm arsenm requested a review from frasercrmck February 25, 2025 10:52
@arsenm arsenm marked this pull request as ready for review February 25, 2025 10:52
@arsenm
Copy link
Contributor Author

arsenm commented Feb 25, 2025

Actually the old codegen looks totally busted, loading from a constant offset from null

@arsenm arsenm merged commit b57e63b into main Feb 25, 2025
13 of 14 checks passed
@arsenm arsenm deleted the users/arsenm/libclc/amdgcn-stop-using-r600-asm-intrinsic-declarations-get-global-size branch February 25, 2025 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

libclc libclc OpenCL library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants