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

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 7 08:08:16 PDT 2021


mtrofin added a comment.

In D109294#3047100 <https://reviews.llvm.org/D109294#3047100>, @erikdesjardins wrote:

> I don't have commit access--please commit this on my behalf (`Erik Desjardins <erikdesjardinspublic at gmail.com>`)
>
> Looks like the check failure is due to it failing to update the status on phabricator, the tests passed (and they pass locally)

ack, I'll do it (wanted to wait for AM my time in case there are actual build breaks)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109294



More information about the llvm-commits mailing list