[PATCH] D137538: [SIFoldOperands] Move `isFoldableCopy` into a separate helper, NFC.

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

+1 for the idea.



================
Comment at: llvm/lib/Target/AMDGPU/SIFoldOperands.cpp:1819
+      if (TII->isFoldableCopy(MI)) {
+        Changed |= tryFoldFoldableCopy(MI, CurrentKnownM0Val);
         continue;
----------------
Personally I would prefer to make CurrentKnownM0Val a field of SIFoldOperands, rather than pass in a reference to a local variable. But perhaps that is overkill.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137538