Page MenuHomePhabricator

[mlir][spirv] Define spv.ImageQuerySize operation

Authored by Weiwei-2021 on May 6 2021, 3:56 PM.



Support OpImageQuerySize in spirv dialect

co-authered-by: Alan Liu <>

Diff Detail

Event Timeline

Weiwei-2021 created this revision.May 6 2021, 3:56 PM
Weiwei-2021 requested review of this revision.May 6 2021, 3:56 PM
antiagainst accepted this revision.May 12 2021, 6:06 AM

Cool; overall LGTM. Just a few nits. Please fix them and then land.


Use switch statement here?


Use switch statement here?


"expected the result to have " << componentNumber << " components, but found " << resultComponentNumber << " components"

This revision is now accepted and ready to land.May 12 2021, 6:06 AM

Thank you @antiagainst for reviewing! I will change based on your comments and then land.

Weiwei-2021 marked 3 inline comments as done.

Upload the latest version and land.

This revision was landed with ongoing or failed builds.May 13 2021, 10:21 AM
This revision was automatically updated to reflect the committed changes.