Page MenuHomePhabricator

AVX-512: Fixed a bug in Scatter scheduling
ClosedPublic

Authored by delena on Sep 10 2017, 6:39 AM.

Details

Reviewers
igorb
RKSimon
Summary

Fixed a bug in splitting Scatter operation in the Type Legalizer.
After splitting one Scatter operation into two, the order of the new instructions is well defined - Lo goes before Hi. Otherwise the semantic of Scatter (from LSB to MSB) is broken.
I'm chaining 2 nodes to prevent reordering.

Diff Detail

Repository
rL LLVM

Event Timeline

delena created this revision.Sep 10 2017, 6:39 AM
igorb accepted this revision.Sep 10 2017, 7:13 AM

LGTM

This revision is now accepted and ready to land.Sep 10 2017, 7:13 AM
RKSimon closed this revision.Sep 22 2017, 1:41 AM
RKSimon edited edge metadata.

This was committed at rL312894