[PATCH] D116952: [ConstantFolding] Flush folded denormals to zero when using fastmath

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 20 05:22:48 PST 2022


lenary added inline comments.


================
Comment at: llvm/lib/Analysis/ConstantFolding.cpp:1007
+      /// If folding produces a floating point denormal, check whether
+      /// it should be forced to zero.
+      if (CFP->getValueAPF().isDenormal()) {
----------------
You should probably also explain that the `return nullptr` means if you don't have a TTI, you are explicitly preventing any constant folding of operations that produce denormals, rather than continuing to fold to the denormal value.


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

https://reviews.llvm.org/D116952



More information about the llvm-commits mailing list