[PATCH] D67126: [GVN/FP] Considate logic for reasoning about equality vs equivalance for floats

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 3 14:02:28 PDT 2019


reames marked an inline comment as done.
reames added inline comments.


================
Comment at: lib/Transforms/Scalar/GVN.cpp:1429
+      // equivalence. 
+      // FIXME: We should do this optimization if 'no signed zeros' is
+      // applicable via an instruction-level fast-math-flag or some other
----------------
xbolva00 wrote:
> Cmp->getFastMathFlags().noSignedZeros() ?
a) I just moved the todo
b) See the comment about not considering myself particularly FP knowledgeable.  


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67126





More information about the llvm-commits mailing list