This is an archive of the discontinued LLVM Phabricator instance.

[X86] Remove unnecessary 3DNow declarations from Intrin.h
AbandonedPublic

Authored by RKSimon on Aug 25 2015, 2:52 PM.

Details

Summary

Follow up to D12272

This adds the missing 3dnow intrinsics _m_to_float / _m_from_float and removes their unnecesary declarations from Intrin.h. These are really just wrappers to existing MMX instructions (they are moved to/from the MMX register as i32 as 3dnow doesn't provide float specific instructions).

Final part to resolve PR23648

Diff Detail

Repository
rL LLVM

Event Timeline

RKSimon updated this revision to Diff 33130.Aug 25 2015, 2:52 PM
RKSimon retitled this revision from to [X86] Remove unnecessary 3DNow declarations from Intrin.h.
RKSimon updated this object.
RKSimon set the repository for this revision to rL LLVM.
RKSimon added a subscriber: cfe-commits.
mkuper resigned from this revision.Jun 6 2016, 4:49 PM
mkuper removed a reviewer: mkuper.
silvas resigned from this revision.Jul 8 2016, 11:42 PM
silvas removed a reviewer: silvas.
RKSimon abandoned this revision.EditedJan 15 2017, 10:55 AM

Abandoning this - unless we have a way to know when to automatically insert emms/femms then its not safe to do this.