[DAGCombiner] look through bitcasts when trying to narrow vector binops
authorSanjay Patel <spatel@rotateright.com>
Tue, 20 Nov 2018 22:26:35 +0000 (22:26 +0000)
committerSanjay Patel <spatel@rotateright.com>
Tue, 20 Nov 2018 22:26:35 +0000 (22:26 +0000)
commit357053f2899f0878e3203bd64db6c9449f1c2bbd
treeca4376a7e87316c13ba36b20e39e530b21312a37
parent5ffb47b3a19120e9718c1785c18c4c1db42cb0c1
[DAGCombiner] look through bitcasts when trying to narrow vector binops

This is another step in vector narrowing - a follow-up to D53784
(and hoping to eventually squash potential regressions seen in
D51553).

The x86 test diffs are wins, but the AArch64 diff is probably not.
That problem already exists independent of this patch (see PR39722), but it
went unnoticed in the previous patch because there were no regression tests
that showed the possibility.

The x86 diff in i64-mem-copy.ll is close. Given the frequency throttling
concerns with using wider vector ops, an extra extract to reduce vector
width is the right trade-off at this level of codegen.

Differential Revision: https://reviews.llvm.org/D54392

llvm-svn: 347356
llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
llvm/test/CodeGen/AArch64/arm64-ld1.ll
llvm/test/CodeGen/X86/avx-vperm2x128.ll
llvm/test/CodeGen/X86/avx1-logical-load-folding.ll
llvm/test/CodeGen/X86/i64-mem-copy.ll
llvm/test/CodeGen/X86/pr36199.ll
llvm/test/CodeGen/X86/sad.ll
llvm/test/CodeGen/X86/vector-reduce-mul.ll