[PATCH] D11146: [ValueTracking] Add support for floating-point minnum and maxnum

James Molloy james.molloy at arm.com
Fri Jul 17 00:56:01 PDT 2015


jmolloy added inline comments.

================
Comment at: lib/Analysis/ValueTracking.cpp:3362
@@ +3361,3 @@
+  //   - maxnum/minnum (C99 fmaxf()/fminf()) return the non-NaN input.
+  //   - A simple C99 (a < b ? a : b) construction will return 'a' (as the
+  //     ordered comparison fails), which could be NaN or non-NaN.
----------------
Typo: This should of course read 'b', not 'a'; I've updated this locally.


Repository:
  rL LLVM

http://reviews.llvm.org/D11146







More information about the llvm-commits mailing list