This is an archive of the discontinued LLVM Phabricator instance.

[TargetLibraryInfo] Correctly handle sqrt*_finite
ClosedPublic

Authored by probinson on Sep 24 2021, 7:21 AM.

Details

Summary

Other <math>_finite calls are marked as unavailable except on GNU/Linux;
it looks like the sqrt set was just overlooked.

Diff Detail

Event Timeline

probinson created this revision.Sep 24 2021, 7:21 AM
probinson requested review of this revision.Sep 24 2021, 7:21 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 24 2021, 7:21 AM
spatel accepted this revision.Sep 24 2021, 10:13 AM

LGTM - for reference, I moved this list of functions with 52149f03053a1590b9cb1f6 but sqrt wasn't there before that either.

This revision is now accepted and ready to land.Sep 24 2021, 10:13 AM
This revision was landed with ongoing or failed builds.Sep 24 2021, 11:57 AM
This revision was automatically updated to reflect the committed changes.