[PATCH] D83601: [ValueTracking] fix bug in maxnum case of cannotBeOrderedLessThanZeroImpl (PR46627)

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 13 10:38:54 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/Analysis/ValueTracking.cpp:3383
+      return (isPositiveNum(V0) && CannotBeNegativeZero(V1, TLI, Depth + 1)) ||
+             (isPositiveNum(V1) && CannotBeNegativeZero(V0, TLI, Depth + 1));
+    }
----------------
Can you just pass false for "SignBitOnly", instead of doing a bunch of extra calls?


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

https://reviews.llvm.org/D83601





More information about the llvm-commits mailing list