This is an archive of the discontinued LLVM Phabricator instance.

clang: Use new frexp intrinsic for builtins and add f16 version
ClosedPublic

Authored by arsenm on May 2 2023, 7:17 PM.

Diff Detail

Event Timeline

arsenm created this revision.May 2 2023, 7:17 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 2 2023, 7:17 PM
arsenm requested review of this revision.May 2 2023, 7:17 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 2 2023, 7:17 PM
arsenm updated this revision to Diff 518947.May 2 2023, 7:34 PM

should we have tests for llvm.frexp.* for LLVM codegen?

should we have tests for llvm.frexp.* for LLVM codegen?

Those are added in the parent that adds the intrinsic, this is just the clang side

yaxunl accepted this revision.Jun 23 2023, 3:22 PM

LGTM. Thanks.

This revision is now accepted and ready to land.Jun 23 2023, 3:22 PM