[PATCH] D109074: [Codegen][TLI][X86] SimplifyMultipleUseDemandedBits(): 0'th vec subreg widening is free, try to perform it earlier
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 1 10:42:54 PDT 2021
lebedev.ri created this revision.
lebedev.ri added reviewers: RKSimon, craig.topper, spatel.
lebedev.ri added a project: LLVM.
Herald added subscribers: pengfei, hiraditya.
lebedev.ri requested review of this revision.
I believe, the profitability reasoning here is correct?
subreg is already located within the 0'th subreg of wider reg,
so if we have suvector insertion at index 0 into undef,
then it's always free do to?
After this, D109065 <https://reviews.llvm.org/D109065> finally avoids the regression in D108382 <https://reviews.llvm.org/D108382>.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D109074
Files:
llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp
llvm/test/CodeGen/AArch64/vecreduce-and-legalization.ll
llvm/test/CodeGen/X86/avg.ll
llvm/test/CodeGen/X86/horizontal-sum.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D109074.369983.patch
Type: text/x-patch
Size: 26020 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210901/93794b2e/attachment.bin>
More information about the llvm-commits
mailing list