[PATCH] D149374: [LICM] Hoist widenable condition when it's the only non-invariant operand of its user

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 27 19:25:58 PDT 2023


skatkov added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LICM.cpp:1243
+      // its user
+      for (const Use &U : CI->uses())
+        if (const Instruction *UI = dyn_cast<Instruction>(U.getUser()))
----------------
Why do you need to iterate over uses instead of users?


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

https://reviews.llvm.org/D149374



More information about the llvm-commits mailing list