[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