This is an archive of the discontinued LLVM Phabricator instance.

clang/HIP: Use abs builtins instead of implementing them
ClosedPublic

Authored by arsenm on Aug 14 2023, 2:11 PM.

Details

Summary

InstCombine already put these back together so there's no visible
change in the -O1 test for the header.

Diff Detail

Event Timeline

arsenm created this revision.Aug 14 2023, 2:11 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 14 2023, 2:11 PM
arsenm requested review of this revision.Aug 14 2023, 2:11 PM
AlexVlx accepted this revision.Aug 15 2023, 8:14 AM

LGTM, thanks.

This revision is now accepted and ready to land.Aug 15 2023, 8:14 AM