[PATCH] D150347: [AMDGPU] Improve abs modifier usage

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 15 08:28:23 PDT 2023


foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp:5148
+    // Ensure we are not already sinking this operand.
+    if (any_of(Ops, [&](Use *U) { return U->get() == Op; }))
+      continue;
----------------
What does this do? Did you mean to check basic blocks here? In any case I am not sure this is required.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150347



More information about the llvm-commits mailing list