This is an archive of the discontinued LLVM Phabricator instance.

[Flang] [PowerPC} Implement the floating-point conversion PowerPC intrinsics
ClosedPublic

Authored by pscoro on Mar 1 2023, 10:02 AM.

Details

Summary

All the fc* floating point conversion PowerPC intrinsics are simply lowered to their LLVM IR intrinsic counterparts and do not require any additional error checking.

Diff Detail

Event Timeline

pscoro created this revision.Mar 1 2023, 10:02 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptMar 1 2023, 10:02 AM
pscoro requested review of this revision.Mar 1 2023, 10:02 AM
pscoro edited the summary of this revision. (Show Details)Mar 1 2023, 10:08 AM
pscoro retitled this revision from [Flang] [PowerPC} Implement the fc* PowerPC intrinsics to [Flang] [PowerPC} Implement the floating-point conversion PowerPC intrinsics.Mar 1 2023, 10:08 AM
klausler accepted this revision.Mar 1 2023, 10:12 AM

LGTM although I can't comment on whether these are accurate interfaces for the specific PPC instructions or not.

This revision is now accepted and ready to land.Mar 1 2023, 10:12 AM
jeanPerier accepted this revision.Mar 3 2023, 12:52 AM
pscoro updated this revision to Diff 502133.Mar 3 2023, 7:59 AM

fcfi is actually just another name for fcfid and both should lower to the same llvm.ppc.fcfid intrinsic. llvm.ppc.fcfi does not exist.
This change and the fcf* intrinsics are now properly tested.