[PATCH] D106233: [InstCombine] Add freezeAllUsesOfArgument to visitFreeze

[Hyeongyu Kim via Phabricator via llvm-commits]

hyeongyukim marked an inline comment as done.
hyeongyukim added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstructionCombining.cpp:3593
+bool InstCombinerImpl::freezeDominatedUses(FreezeInst &FI) {
+  Value *Op = dyn_cast<Value>(FI.getOperand(0));
+
----------------
aqjune wrote:
> If FI's operand is constant (e.g., undef), getting dominance relation might not work well.
Is there any other consideration(except constant) when obtaining the dominance relation?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106233