[PATCH] D41333: [ValueTracking] ignore FP signed-zero when detecting a casted-to-integer fmin/fmax pattern

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 18 13:05:29 PST 2017


hfinkel added a comment.

LGTM as well.

Thinking about it, there are some other cases that will have a similar effect (e.g., adding/subtracting to a non-zero constant, squaring the result). Also, if the min/max is with a constant such that zero is not a possible answer. Maybe we could add a comment about that too?


https://reviews.llvm.org/D41333





More information about the llvm-commits mailing list