This is an archive of the discontinued LLVM Phabricator instance.

[X86] Combine masked store and truncate into masked truncating stores.
ClosedPublic

Authored by craig.topper on Jan 24 2019, 9:29 PM.

Details

Summary

We also need to combine to masked truncating with saturation stores, but I'm leaving that for a future patch.

This does regress some tests that used truncate wtih saturation followed by a masked store. Those now use a truncating store and use min/max to saturate.

Diff Detail

Repository
rL LLVM

Event Timeline

craig.topper created this revision.Jan 24 2019, 9:29 PM
RKSimon accepted this revision.Jan 25 2019, 12:05 AM

LGTM with one minor

lib/Target/X86/X86ISelLowering.cpp
37669 ↗(On Diff #183479)

remove this - its a shadow duplicate now

This revision is now accepted and ready to land.Jan 25 2019, 12:05 AM
This revision was automatically updated to reflect the committed changes.