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

Luke Lau via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 29 10:34:35 PDT 2023


luke added a comment.

In D153570#4460419 <https://reviews.llvm.org/D153570#4460419>, @ABataev wrote:

> Just passing required argument looks good to me.

Thanks for the review: Just wanted to double check if you've approved this? I got an email that saying that you accepted the revision, but it still seems to be marked as needs review in Phabricator


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153570



More information about the llvm-commits mailing list