This is an archive of the discontinued LLVM Phabricator instance.

[AVX512] Don't switch unmasked subvector insert/extract instructions when AVX512DQI is enabled.

Authored by craig.topper on Aug 13 2017, 11:03 PM.



There's no reason to switch instructions with and without DQI. It just creates extra isel patterns and test divergences.

There is however value in enabling the masked version of the instructions with DQI.

This required introducing some new multiclasses to enabling this splitting. If anyone has a better suggestion of how to do this I'm open to it.

Diff Detail


Event Timeline

craig.topper created this revision.Aug 13 2017, 11:03 PM
igorb accepted this revision.Aug 17 2017, 2:55 AM

Could you please make sure that tests cover the masked version of the instructions with DQI.

This revision is now accepted and ready to land.Aug 17 2017, 2:55 AM
This revision was automatically updated to reflect the committed changes.