This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Add some basic Min/Max costs
ClosedPublic

Authored by dmgreen on Feb 12 2021, 6:37 AM.

Details

Summary

This adds basic MVE costs for SMIN/SMAX/UMIN/UMAX, as well as MINNUM and MAXNUM representing fmin and fmax. It tightens up the costs, not using a ICmp+Select cost.

Diff Detail

Event Timeline

dmgreen created this revision.Feb 12 2021, 6:37 AM
dmgreen requested review of this revision.Feb 12 2021, 6:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 12 2021, 6:37 AM
SjoerdMeijer accepted this revision.Feb 15 2021, 2:49 AM

looks reasonable to me.

This revision is now accepted and ready to land.Feb 15 2021, 2:49 AM
This revision was automatically updated to reflect the committed changes.