[PATCH] D106053: [CodeGen] Remove pending AssertZext AssertSext in promoting FP_TO_INT

[Xiang Zhang via Phabricator via llvm-commits]

xiangzhangllvm added a comment.

@efriedma, Thanks you for explain, I am much clear about that.  (And sorry for my late respond, I got a long travel yesterday)

Or Maybe we can replace the AssertZext with AssertZextOrPoison in "May Be Poison" case ? 
If it is much easier to change all the existed optimizations.

Anyway, I suggest let us first revert 2a419a0b9957 <https://reviews.llvm.org/rG2a419a0b9957ebac9e11e4b43bc9fbe42a9207df> (= D106222 <https://reviews.llvm.org/D106222>) , because we have a job blocked by this problem and we already find it really has problem.


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

https://reviews.llvm.org/D106053