[PATCH] D73558: AMDGPU: Correct memory size for image intrinsics

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 2 11:02:27 PST 2020


nhaehnle accepted this revision.
nhaehnle added a comment.
This revision is now accepted and ready to land.

Thank you. Memory sizes on image instructions are kind of weird in the first place, because we simply cannot know the actual *memory* size at compile time, but this is a pragmatic thing to do for GlobalISel at the moment. LGTM.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73558/new/

https://reviews.llvm.org/D73558





More information about the llvm-commits mailing list