[PATCH] D58233: Allow replacing intrinsic operands with variables

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

It looks like the change is essentially a no-op for constant hoisting itself?  Or maybe I'm misreading the code somehow?  But sure, I'm fine with generally hoisting the check out of canReplaceOperandWithVariable, for all three of the current callers.


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

https://reviews.llvm.org/D58233