[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