This is an archive of the discontinued LLVM Phabricator instance.

[SLP] Add -slp-min-reg-size command line option.
ClosedPublic

Authored by mzolotukhin on Sep 29 2015, 6:58 PM.

Details

Summary

MinVecRegSize is currently hardcoded to 128; this patch adds a cl::opt to allow
changing it. I tried not to change any existing behavior for the default case.

This option would be handy for writing smaller tests. Also, currently it's not
even possible to write a test for vectorization of i1/i2/i4 stores, as we only
look for at most 16 accesses, while the minimal total size should be 128bits.
It's possible to write such test using phi-nodes, but that looks like a bug (I
plan to follow up with a fix for it too).

Diff Detail

Repository
rL LLVM

Event Timeline

mzolotukhin retitled this revision from to [SLP] Add -slp-min-reg-size command line option..
mzolotukhin updated this object.
mzolotukhin added a subscriber: llvm-commits.
aschwaighofer edited edge metadata.Sep 29 2015, 7:17 PM

Makes sense.

LGTM.

This revision was automatically updated to reflect the committed changes.

I discovered that I forgot to commit this patch - committed now in r263089. Thanks for taking a look!

Michael