[PATCH] D120571: [CGP, AArch64] Replace zexts with shuffle that can be lowered using tbl.
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 10 09:56:39 PDT 2023
efriedma added a comment.
Herald added a subscriber: StephenFan.
Regression reported at https://github.com/llvm/llvm-project/issues/62620 . I think the issue is that this transform isn't aware of widening instructions; if the <16 x i32> output is used in a way that can be optimized to use 16-bit inputs, the six ushll/ushll2 instructions are actually lowered to just two ushll/ushll2 instructions, so transforming that into for tbl isn't profitable.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D120571/new/
https://reviews.llvm.org/D120571
More information about the llvm-commits
mailing list