[PATCH] D120571: [CGP, AArch64] Replace zexts with shuffle that can be lowered using tbl.

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added inline comments.
Herald added a project: All.


================
Comment at: llvm/test/CodeGen/AArch64/vselect-ext.ll:193
 define void @extension_in_loop_v16i8_to_v16i32(i8* %src, i32* %dst) {
 ; CHECK-LABEL: extension_in_loop_v16i8_to_v16i32:
 ; CHECK:       ; %bb.0: ; %entry
----------------
I haven't looked too carefully at this, so this is more of a drive by comment, but I think I see more code/instructions so was wondering if we shouldn't be doing this for code-size, and if this is always a win (e.g. smaller/in-order aarch64 implementations).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D120571/new/

https://reviews.llvm.org/D120571