[PATCH] D98963: [LoopVectorize] Change the identity element for FAdd

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 25 06:23:01 PDT 2021


spatel added a comment.

Do we know why we're seeing the mixed zeros in the tests? Is it an artifact of the test (because the test includes a +0.0 to start with) that we don't see in practice?
Note that phi instructions can have FMF ( D67564 <https://reviews.llvm.org/D67564> ), so if we assume that was the right direction, and we are losing the flags somewhere in the opt pipeline, we should try to fix that.


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

https://reviews.llvm.org/D98963



More information about the llvm-commits mailing list