[PATCH] D153570: [SLP] Explicitly pass AccessTy to getGEPCost

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 29 09:32:39 PDT 2023

Previous message: [PATCH] D149888: [RISCV][SLP] Add tests for unprofitable SLP vectorization due to GEP. NFC
Next message: [PATCH] D153570: [SLP] Explicitly pass AccessTy to getGEPCost
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

reames added a comment.

At this point, I'm going defer the remaining review to Alexey.  What you've got looks entirely reasonable to me, but Alexey knows the SLP interactions much better.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153570

Previous message: [PATCH] D149888: [RISCV][SLP] Add tests for unprofitable SLP vectorization due to GEP. NFC
Next message: [PATCH] D153570: [SLP] Explicitly pass AccessTy to getGEPCost
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list