This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Don't use VLMAX for start value splat in reduction lowering.
ClosedPublic

Authored by craig.topper on Dec 10 2021, 9:19 AM.

Details

Summary

The reduction instructions only reads the first element. The
execution time for a splat may take longer with a larger VL.
We should use the smallest VL we can.

Diff Detail

Event Timeline

craig.topper created this revision.Dec 10 2021, 9:19 AM
craig.topper requested review of this revision.Dec 10 2021, 9:20 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 10 2021, 9:20 AM
Herald added a subscriber: MaskRay. · View Herald Transcript
This revision is now accepted and ready to land.Dec 11 2021, 7:24 AM
frasercrmck accepted this revision.Dec 13 2021, 2:01 AM

LGTM too but clang-format needs appeasing. Thanks, Craig!

This revision was landed with ongoing or failed builds.Dec 13 2021, 9:06 AM
This revision was automatically updated to reflect the committed changes.