Page MenuHomePhabricator

[NFC][LoopVectorize] Change trip counts for some tests to guarantee a scalar tail
AcceptedPublic

Authored by david-arm on Thu, Mar 16, 4:57 AM.

Details

Summary

Quite a few vectoriser tests were using a trip count of 1024,
which meant:

  1. For fixed-length VFs we would never actually tail-fold, e.g. see Transforms/LoopVectorize/RISCV/uniform-load-store.ll. This is because we can prove at compile-time there will never be a scalar tail.
  2. As of D146199 the same optimisation mentioned above will also apply to scalable VFs too.

I've changed all such trip counts to be 1025 instead.

Diff Detail

Event Timeline

david-arm created this revision.Thu, Mar 16, 4:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Mar 16, 4:57 AM
david-arm requested review of this revision.Thu, Mar 16, 4:57 AM
reames accepted this revision.Thu, Mar 16, 8:14 AM

LGTM

This revision is now accepted and ready to land.Thu, Mar 16, 8:14 AM
fhahn accepted this revision.Thu, Mar 16, 8:28 AM

LGTM, thanks for splitting this off!