Page MenuHomePhabricator

[AVX512] Remove and autoupgrade many of the broadcast intrinsics
ClosedPublic

Authored by craig.topper on Aug 10 2017, 10:58 PM.

Details

Summary

This autoupgrades most of the broadcast intrinsics. They've been unused in clang for some time.

This leaves the 32x2 intrinsics because they are still used in clang.

Diff Detail

Repository
rL LLVM

Event Timeline

craig.topper created this revision.Aug 10 2017, 10:58 PM
craig.topper added a reviewer: igorb.
RKSimon accepted this revision.Aug 11 2017, 4:05 AM

LGTM

Do you think we should make SUBV_BROADCAST a memory intrinsic? Maybe BROADCAST as well, although that has reg implementations and splitting it into two may be awkward.

This revision is now accepted and ready to land.Aug 11 2017, 4:05 AM

I think I remember having that thought before. It's been a while. I actually found most of this patch in a "git stash" from quite a while ago.

This revision was automatically updated to reflect the committed changes.