[PATCH] D97382: NFC: Migrate PartialInlining to work on InstructionCost

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm accepted this revision.
paulwalker-arm added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Transforms/IPO/PartialInlining.cpp:1361
+
+  // Don't try partial inlining if any of the costs are Invalid.
+  auto OutliningCosts = computeOutliningCosts(Cloner);
----------------
This comment doesn't make sense anymore given the most recent change.  Probably easier to just remove it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97382