This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Use BaseT::getMemoryOpCost for getMemoryOpCost
ClosedPublic

Authored by dmgreen on Jun 24 2020, 5:45 AM.

Details

Summary

This alters getMemoryOpCost to use the Base TargetTransformInfo version that includes some additional checks for whether extending loads are legal. This will generally have the effect of making <2 x ..> and some <4 x ..> loads/stores more expensive, which in turn should help favour larger vector factors.

Notably it alters the cost of a <4 x half>, which with the current codegen will be expensive if it is not extended.

Diff Detail

Event Timeline

dmgreen created this revision.Jun 24 2020, 5:45 AM
samparker accepted this revision.Jun 29 2020, 7:30 AM
This revision is now accepted and ready to land.Jun 29 2020, 7:30 AM
This revision was automatically updated to reflect the committed changes.