[PATCH] D141251: Replace un-demanded values with undef in BDCE.

[Nuno Lopes via Phabricator via llvm-commits]

nlopes added a comment.

I must say that this patch is a bit annoying :)
We are trying to remove undef, and you're adding yet another use.
It's true we can't use poison here. The alternative in the no-undef world is 'freeze poison'. Or just leave the zero there.
Is this change important? If not, I would prefer to not do it, as we'll have to revert it when removing undef (hopefully later this year).
Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141251