[PATCH] D76248: Fix a bug in the inliner that causes subsequent double inlining

[David Li via Phabricator via llvm-commits]

davidxl added a comment.

ok -- I misunderstood (it to be a runtime problem) -- it is actually a stale reference related issue. In the failing case,  is the inlining order the following?

top->second 
top->first (original site)
top->first (from the clone of 'second')?

Even without ICE, this is still wrong unless first has no side effect.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76248