[PATCH] D84792: [InstSimplify/NewGVN] Add option to control the use of undef.

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Is the goal here that it's legal to replace the output value of SimplifyInstruction with the input value, in addition to the normal replacement of the input value with the output value?  Or do we specifically care about literal UndefValue constants somehow, as opposed to values which are undef?  If we had a PoisonValue constant, would we also need to exclude it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84792