Skip to content

samplerlessReads: fix use_pitches test for devices with read_write images supported#2650

Open
karolherbst wants to merge 1 commit intoKhronosGroup:mainfrom
karolherbst:fix/samplerless_read_write_pitches
Open

samplerlessReads: fix use_pitches test for devices with read_write images supported#2650
karolherbst wants to merge 1 commit intoKhronosGroup:mainfrom
karolherbst:fix/samplerless_read_write_pitches

Conversation

@karolherbst
Copy link
Copy Markdown
Contributor

The test specified a image_row_pitch on the read_write image without a host_ptr or parent buffer.

The read_write image doesn't get inialized like the read_only does, so it's fine to just reset image_row_pitch back to 0 for its creation.

@karolherbst karolherbst force-pushed the fix/samplerless_read_write_pitches branch from f79ed23 to 1644dbd Compare March 28, 2026 16:15
…ages supported

The test specified a image_row_pitch and image_slice_pitch on the
read_write image without a host_ptr or parent buffer.

The read_write image doesn't get inialized like the read_only does, so
it's fine to just reset the pitches back to 0 for its creation.
@karolherbst karolherbst force-pushed the fix/samplerless_read_write_pitches branch from 1644dbd to cb49713 Compare March 28, 2026 16:38
@karolherbst
Copy link
Copy Markdown
Contributor Author

Important clarification: this fixes the test when ran with read_write use_pitches arguments.

@karolherbst
Copy link
Copy Markdown
Contributor Author

Also looks like that test_samplerless_reads use_pitches doesn't run read_write tests by default, so apparently this isn't critical for a conformance run anyway...

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.

1 participant