This is an archive of the discontinued LLVM Phabricator instance.

Define a feature for __float128 support in the PPC back end
ClosedPublic

Authored by nemanjai on Dec 1 2015, 9:02 AM.

Details

Summary

In preparation for supporting IEEE Quad precision floating point, this patch simply defines a feature to specify the target supports this. For now, nothing is done with the target feature, we just don't want warnings from the Clang FE when a user specifies -mfloat128.

Diff Detail

Repository
rL LLVM

Event Timeline

nemanjai updated this revision to Diff 41520.Dec 1 2015, 9:02 AM
nemanjai retitled this revision from to Define a feature for __float128 support in the PPC back end.
nemanjai updated this object.
nemanjai added reviewers: hfinkel, kbarton, wschmidt.
nemanjai set the repository for this revision to rL LLVM.
nemanjai added a subscriber: llvm-commits.
hfinkel accepted this revision.Dec 10 2015, 1:29 PM
hfinkel edited edge metadata.

I assume you're planning to follow-up this work with actually calling-convention work to make sure we pass float128s in the right registers, etc.? If so, then go ahead.

This revision is now accepted and ready to land.Dec 10 2015, 1:29 PM
nemanjai closed this revision.Dec 15 2015, 4:22 AM

Yes, the calling convention work will follow. This feature is simply so that I can add the front end patch and provide a test case (so we don't get warnings about the unsupported target feature).

Committed revision 255642.