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

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Err, wait, obviously they're not equivalent.  Not

In D106053#2882183 <https://reviews.llvm.org/D106053#2882183>, @efriedma wrote:

> llc compiles both of these to a single cvttps2dq; alive2 says these are not equivalent.

Err, wait, they're obviously not equivalent... not sure what I was thinking.  Would need a different testcase.

Anyway, the general issue issue I'm getting at is that poison in vectors is supposed to apply separately to each element of the vector.  Here, it's leaking across elements.


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

https://reviews.llvm.org/D106053