This is an archive of the discontinued LLVM Phabricator instance.

[mlir][vector] Remove default value in populateVectorMultiReductionLoweringPatterns
ClosedPublic

Authored by dcaballe on Dec 15 2021, 8:18 AM.

Details

Summary

Having a default value for the lowering strategy of the multi-reduction op has proven
to be unexpected by users. This patch is dropping the default value so that users have
to explicitly choose the lowering strategy to be applied.

Diff Detail

Event Timeline

dcaballe created this revision.Dec 15 2021, 8:18 AM
dcaballe requested review of this revision.Dec 15 2021, 8:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 15 2021, 8:18 AM
This revision is now accepted and ready to land.Dec 15 2021, 11:17 PM