[PATCH] D132566: [SLP] Fix cost model w.r.t. operand properties

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 22 07:33:11 PDT 2022


RKSimon added a comment.

LGTM but naturally test coverage would be a plus. Have you had any luck with your multiply by (pow2 and non-pow2) constant example?


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

https://reviews.llvm.org/D132566



More information about the llvm-commits mailing list