[PATCH] D97835: [AArch64][GlobalISel] Add combine for extract_vector_elt(build_vector, cst)
Amara Emerson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 3 11:32:17 PST 2021
aemerson updated this revision to Diff 327865.
aemerson added a comment.
New patch.
With the oneuse check in place, it now doesn't handle one of the cases I was interested in, where we see one extract for each element of the BV. I'll create a new patch to combine that away, probably starting from a build_vector.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97835/new/
https://reviews.llvm.org/D97835
Files:
llvm/include/llvm/CodeGen/GlobalISel/CombinerHelper.h
llvm/include/llvm/Target/GlobalISel/Combine.td
llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp
llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp
llvm/test/CodeGen/AArch64/GlobalISel/combine-extract-vec-elt.mir
llvm/test/CodeGen/AArch64/GlobalISel/legalizer-info-validation.mir
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D97835.327865.patch
Type: text/x-patch
Size: 16451 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210303/bd514dae/attachment.bin>
More information about the llvm-commits
mailing list