[PATCH] D124232: [AMDGPU] Use d16 flag for image.sample instructions

Mariusz Sikora via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 27 03:42:46 PDT 2022


mariusz-sikora-at-amd marked an inline comment as done.
mariusz-sikora-at-amd added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUInstCombineIntrinsic.cpp:142
+
+  bool RemoveOldIntr = !OldIntr.isIdenticalTo(&InstToReplace);
+
----------------
foad wrote:
> Can you compare pointers instead of using isIdenticalTo? That would be simpler, if it works.
Yes it will work. Thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124232



More information about the llvm-commits mailing list