This is an archive of the discontinued LLVM Phabricator instance.

ValueTracking: Handle constrained_sqrt in computeKnownFPClass
ClosedPublic

Authored by arsenm on Apr 13 2023, 8:09 AM.

Details

Summary

With this, the body of CannotBeNegativeZero can be dropped.
Dropped the rounding mode restriction the current implementation has,
The standard text is pretty definitive about the sign of sqrt.

Diff Detail

Event Timeline

arsenm created this revision.Apr 13 2023, 8:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 13 2023, 8:09 AM
arsenm requested review of this revision.Apr 13 2023, 8:09 AM
Herald added a reviewer: sstefan1. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: wdng. · View Herald Transcript
foad accepted this revision.Apr 13 2023, 9:05 AM
This revision is now accepted and ready to land.Apr 13 2023, 9:05 AM
jcranmer-intel accepted this revision.Apr 14 2023, 1:02 PM
foad accepted this revision.May 17 2023, 1:43 AM