[PATCH] D148230: ValueTracking: fadd +0 cannot return -0

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 5 02:40:35 PDT 2023


foad accepted this revision.
foad added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Analysis/ValueTracking.cpp:3945
+  // If we know there are no denormals, nothing can be flushed to zero.
+  if (isKnownNeverSubnormal())
+    return true;
----------------
Nit: might as well push this into the PositiveZero/default case.


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

https://reviews.llvm.org/D148230



More information about the llvm-commits mailing list