[PATCH] D59710: [SLP] remove lower limit for forming reduction patterns
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 6 13:15:54 PST 2019
spatel marked 2 inline comments as done.
spatel added inline comments.
================
Comment at: llvm/include/llvm/Transforms/Vectorize/SLPVectorizer.h:120
slpvectorizer::BoUpSLP &R,
- TargetTransformInfo *TTI);
+ TargetTransformInfo *TTI, bool Try2WayRdx);
----------------
xbolva00 wrote:
> bool Try2WayRdx = false ?
I don't think it's a good thing in general to have bool args with defaults...it makes reading the code harder. But this is all a hack at this point anyway, so sure, let's reduce the diffs. :)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59710/new/
https://reviews.llvm.org/D59710
More information about the llvm-commits
mailing list