This is an archive of the discontinued LLVM Phabricator instance.

[X86] Remove unnecessary MMX declarations from Intrin.h
ClosedPublic

Authored by RKSimon on Aug 23 2015, 10:16 AM.

Details

Summary

As discussed in PR23648 - the intrinsics _m_from_int, _m_to_int and _m_prefetch are defined in mmintrin.h and prfchwintrin.h so we don't need to in Intrin.h

Added tests for _m_from_int and _m_to_int

D11338 already added a test for _m_prefetch

I'll add _m_from_float and _m_to_float in a future (3dNow) patch which will remove the last of the 'missing' declarations from Intrin.h

Diff Detail

Repository
rL LLVM

Event Timeline

RKSimon updated this revision to Diff 32926.Aug 23 2015, 10:16 AM
RKSimon retitled this revision from to [X86] Remove unnecessary MMX declarations from Intrin.h.
RKSimon updated this object.
RKSimon added reviewers: echristo, silvas, craig.topper.
RKSimon set the repository for this revision to rL LLVM.
RKSimon added a subscriber: cfe-commits.
echristo accepted this revision.Aug 24 2015, 10:49 PM
echristo edited edge metadata.
This revision is now accepted and ready to land.Aug 24 2015, 10:49 PM
This revision was automatically updated to reflect the committed changes.