[PATCH] D140676: [AArch64] `LowerZERO_EXTEND_VECTOR_INREG()`: recursively apply `zip1` until done

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 26 17:06:03 PST 2022


lebedev.ri created this revision.
lebedev.ri added reviewers: fhahn, nilanjana_basu, dmgreen, david-arm, hassnaa-arm.
lebedev.ri added a project: LLVM.
Herald added subscribers: hiraditya, kristof.beyls.
Herald added a project: All.
lebedev.ri requested review of this revision.

While the one-step `zip1` lowering seemed obviously good,
here i feel like i should double-check: is 2 or more `zip1`
not worse than a `tbl`?
I'm guessing so, because it avoids constant pool load.

This comes up in a follow-up change to `combineShuffleToZeroExtendVectorInReg()`.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D140676

Files:
  llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
  llvm/test/CodeGen/AArch64/zext-to-tbl.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D140676.485321.patch
Type: text/x-patch
Size: 10031 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221227/cc1529e5/attachment.bin>


More information about the llvm-commits mailing list