diff options
author | Sanjay Patel <spatel@rotateright.com> | 2021-01-05 13:21:49 -0500 |
---|---|---|
committer | Sanjay Patel <spatel@rotateright.com> | 2021-01-05 13:23:07 -0500 |
commit | 3b8b2c7da2efb88d9f13e911e383af430ab463ef (patch) | |
tree | 76f39818b28c40495af61464b0b28eb0493124a0 /clang/unittests/Frontend/CompilerInvocationTest.cpp | |
parent | c707716c049cd46bd89da102cf9444462487b490 (diff) | |
download | llvm-3b8b2c7da2efb88d9f13e911e383af430ab463ef.zip llvm-3b8b2c7da2efb88d9f13e911e383af430ab463ef.tar.gz llvm-3b8b2c7da2efb88d9f13e911e383af430ab463ef.tar.bz2 |
[SLP] delete unused pairwise reduction option
SLP tries to model 2 forms of vector reductions: pairwise and splitting.
From the cost model code comments, those are defined using an example as:
/// Pairwise:
/// (v0, v1, v2, v3)
/// ((v0+v1), (v2+v3), undef, undef)
/// Split:
/// (v0, v1, v2, v3)
/// ((v0+v2), (v1+v3), undef, undef)
I don't know the full history of this functionality, but it was partly
added back in D29402. There are apparently no users at this point (no
regression tests change). X86 might have managed to work-around the need
for this through cost model and codegen improvements.
Removing this code makes it easier to continue the work that was started
in D87416 / D88193. The alternative -- if there is some target that is
silently using this option -- is to move this logic into LoopUtils. We
have related/duplicate functionality there via llvm::createTargetReduction().
Differential Revision: https://reviews.llvm.org/D93860
Diffstat (limited to 'clang/unittests/Frontend/CompilerInvocationTest.cpp')
0 files changed, 0 insertions, 0 deletions