This is an archive of the discontinued LLVM Phabricator instance.

[AVX-512] Support FCOPYSIGN form v16f32 and v8f64
ClosedPublic

Authored by craig.topper on Nov 16 2016, 11:17 PM.

Details

Summary

This extends FCOPYSIGN support to 512-bit vectors.

I've also added tests to show what the 128-bit and 256-bit cases look like with broadcast loads.

Diff Detail

Repository
rL LLVM

Event Timeline

craig.topper retitled this revision from to [AVX-512] Support FCOPYSIGN form v16f32 and v8f64.
craig.topper updated this object.
craig.topper added reviewers: spatel, delena, zvi, RKSimon.
craig.topper added a subscriber: llvm-commits.
spatel accepted this revision.Nov 17 2016, 6:44 AM
spatel edited edge metadata.

LGTM.

lib/Target/X86/X86ISelLowering.cpp
14858–14861 ↗(On Diff #78326)

We see this kind of pattern a lot. I wonder if there's some way to generalize and unify these checks in a helper function, so we don't have to keep repeating it?

This revision is now accepted and ready to land.Nov 17 2016, 6:44 AM
This revision was automatically updated to reflect the committed changes.