This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Some saturation instructions not DSP-only
ClosedPublic

Authored by bogden on Aug 1 2016, 5:46 AM.

Details

Summary

Commit 276701 requires that targets have the DSP extensions to use
certain saturating instructions. This requires some corrections.

For ARM ISA the instructions in question are available in all v6*
architectures.

For Thumb2, the instructions in question are available from v6T2.
SSAT and USAT are part of the base architecture while SSAT16 and
USAT16 require the DSP extensions.

Diff Detail

Event Timeline

bogden updated this revision to Diff 66307.Aug 1 2016, 5:46 AM
bogden retitled this revision from to [ARM] Some saturation instructions not DSP-only.
bogden updated this object.
bogden added a reviewer: rengolin.
bogden added a subscriber: llvm-commits.
rengolin accepted this revision.Aug 1 2016, 6:08 AM
rengolin edited edge metadata.

Hi Bernie,

This is much clearer, thanks!

--renato

This revision is now accepted and ready to land.Aug 1 2016, 6:08 AM
bogden closed this revision.Aug 2 2016, 3:11 AM
This revision was automatically updated to reflect the committed changes.

Backported this to release 3.9 (together with the original patch).

Thanks!