Could be slightly smarter in cases that are probably uninteresting.
Details
Diff Detail
Event Timeline
llvm/lib/Analysis/ValueTracking.cpp | ||
---|---|---|
4495 | This also includes -subnormal in DAZ=preserve-sign mode. |
llvm/lib/Analysis/ValueTracking.cpp | ||
---|---|---|
4495 | The comment could elaborate on the preserve sign handling, but I think this is correct as is? There is a pre-existing bug in cannotBeOrderedLessThanZeroImpl for denormal handling |
Rebase to use new stuff
llvm/lib/Analysis/ValueTracking.cpp | ||
---|---|---|
4491 | Oh right, this predates when I added cannotBeOrderedLessThanZero to KnownFPClass |
llvm/lib/Analysis/ValueTracking.cpp | ||
---|---|---|
4488–4489 | Yes, I need to go back to refine this. You are never allowed to introduce signaling nans. Non-constrained ops just don't guarantee quieting. The propagateNaN helper introduced later helps with this (although it doesn't quite apply here since there are additional nan conditions |
"known"