This is an archive of the discontinued LLVM Phabricator instance.

[ARM][CostModel] Improve getCastInstrCost
ClosedPublic

Authored by samparker on May 7 2020, 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

Build Status
Buildable 56039

Event Timeline

samparker created this revision.May 7 2020, 3:24 AM
dmgreen added inline comments.May 7 2020, 4:54 AM
llvm/test/Transforms/LoopVectorize/ARM/add-cast-vect.ll
1

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

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

Cheers, now moved the tests.

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

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

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