This is an archive of the discontinued LLVM Phabricator instance.

Add missing builtins to the PPC back end for ABI compliance (vol. 1)
ClosedPublic

Authored by nemanjai on Jun 23 2015, 12:57 AM.

Details

Summary

This is the back end portion of patch
http://reviews.llvm.org/D10637
It just adds the code gen and intrinsic functions necessary to support that patch to the back end.

Diff Detail

Repository
rL LLVM

Event Timeline

nemanjai updated this revision to Diff 28207.Jun 23 2015, 12:57 AM
nemanjai retitled this revision from to Add missing builtins to the PPC back end for ABI compliance (vol. 1).
nemanjai updated this object.
nemanjai edited the test plan for this revision. (Show Details)
nemanjai set the repository for this revision to rL LLVM.
nemanjai added a subscriber: Unknown Object (MLST).
wschmidt accepted this revision.Jun 23 2015, 1:23 PM
wschmidt edited edge metadata.

LGTM. I guess it's ok to add a single new test with this name and put all the new ones in there. We have a lot of existing tests with the naming convention vec_*.ll, but I'm ok with having one new test for the ones you're adding rather than splitting it up into a bunch of little ones.

This revision is now accepted and ready to land.Jun 23 2015, 1:23 PM
nemanjai closed this revision.Jun 26 2015, 12:27 PM

Committed revision 240820.