[PATCH] D109294: [Inline] Introduce Constant::hasOneLiveUse, use it instead of hasOneUse in inline cost model (PR51667)

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added inline comments.


================
Comment at: llvm/include/llvm/IR/Constant.h:217
+  /// Constant::removeDeadConstantUsers, but doesn't remove dead constants.
+  bool hasNLiveUsesOrMore(unsigned N) const;
+
----------------
mtrofin wrote:
> is this used anywhere? If not, no need to introduce it. it'd also keep the implementation of constantHasLiveUses simpler.
+1


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109294