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

Juneyoung Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 19 04:09:22 PDT 2021


aqjune 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));
+
----------------
If FI's operand is constant (e.g., undef), getting dominance relation might not work well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106233



More information about the llvm-commits mailing list