[PATCH] D140677: [AArch64][DAG] `canCombineShuffleToExtendVectorInreg()`: allow illegal types before legalization

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 2 10:44:50 PST 2023


fhahn added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/fp-conversion-to-tbl.ll:473
 ; CHECK-NEXT:    cmp x8, #1000
-; CHECK-NEXT:    tbl.16b v3, { v2 }, v0
-; CHECK-NEXT:    tbl.16b v2, { v2 }, v1
-; CHECK-NEXT:    ucvtf.4s v3, v3
+; CHECK-NEXT:    dup.2s v2, v1[1]
+; CHECK-NEXT:    tbl.16b v1, { v1 }, v0
----------------
I think this is not quite as clear cut as D140676, but I think this is still a regression over the original code where the constant-pool loads are outside the loop.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140677



More information about the llvm-commits mailing list