[PATCH] D106140: [LV] Change interface of getReductionPatternCost to return Optional

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 19 00:29:04 PDT 2021


sdesmalen accepted this revision.
sdesmalen added a comment.
This revision is now accepted and ready to land.

The patch looks fine to me, although this might be an incentive to add a new UnknownCost state to `InstructionCost`? Is this something you've considered?


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

https://reviews.llvm.org/D106140



More information about the llvm-commits mailing list