[PATCH] D92238: [SCEVExpander] Migrate costAndCollectOperands to use InstructionCost.

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

I think this is conflating use-cases.
I'm not sure the entire `InstructionCost` should be used as a variable to track full/remaining cost over a several instructions.
Perhaps there should be a[nother] abstraction for that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92238