Page MenuHomePhabricator

[ARM][CostModel] Improve getCastInstrCost
ClosedPublic

Authored by samparker on Thu, May 7, 3:24 AM.

Details

Summary
  • Specifically check for sext/zext users which have 'long' form NEON instructions.
  • Add more entries to the table for sext/zexts so that we can report more accurately the number of vmovls required for NEON.
  • Pass the instruction to the base implementation.

Diff Detail

Event Timeline

samparker created this revision.Thu, May 7, 3:24 AM
dmgreen added inline comments.Thu, May 7, 4:54 AM
llvm/test/Transforms/LoopVectorize/ARM/add-cast-vect.ll
1 ↗(On Diff #262586)

Should these be in test/Analysis/CostModel/ARM?

samparker updated this revision to Diff 262621.Thu, May 7, 5:25 AM

Cheers, now moved the tests.

dmgreen accepted this revision.Tue, May 12, 2:08 AM

Thanks for the updates. Looks like a good one to me.

This revision is now accepted and ready to land.Tue, May 12, 2:08 AM
This revision was automatically updated to reflect the committed changes.