[PATCH] D105298: [InstSimplify] do not propagate poison from select arm to icmp user

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 1 10:57:40 PDT 2021


efriedma added a comment.

I'm skeptical it's correct to explicitly check for PoisonValue here.  Does it matter if the value is poison, but not a poison literal?  Does it matter if a constant vector has a poison element?


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

https://reviews.llvm.org/D105298



More information about the llvm-commits mailing list