This is an archive of the discontinued LLVM Phabricator instance.

[X86][AVX512] Add missing patterns for broadcast fallback in case load node has multiple uses (for v4i64 and v4f64).
ClosedPublic

Authored by aymanmus on Dec 11 2016, 5:09 AM.

Details

Summary

When the load node which the broadcast instruction broadcasts has multiple uses, it cannot be folded.
A fallback pattern is added to catch these cases and provide another solution.

Diff Detail

Event Timeline

aymanmus updated this revision to Diff 81018.Dec 11 2016, 5:09 AM
aymanmus retitled this revision from to [X86][AVX512] Add missing patterns for broadcast fallback in case load node has multiple uses (for v4i64 and v4f64)..
aymanmus updated this object.
aymanmus added reviewers: mkuper, zvi, spatel, RKSimon.
aymanmus added a subscriber: llvm-commits.
zvi accepted this revision.Dec 11 2016, 7:24 AM
zvi edited edge metadata.

LGTM. I think it would be better if the test cases were located in the same file as the other cases that cover the fallback patterns.

This revision is now accepted and ready to land.Dec 11 2016, 7:24 AM
aymanmus updated this revision to Diff 81024.Dec 11 2016, 11:47 AM
aymanmus edited edge metadata.
This revision was automatically updated to reflect the committed changes.