[llvm] [AlwaysInline] Avoid unnecessary BFI fetches (PR #117750)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 11 06:23:48 PST 2024


nikic wrote:

Thanks for the report. Also reproduces with `-passes="function(require<block-freq>,loop(loop-unroll-full)),always-inline"`.

The problem here is that we can't really fetch analyses during inlining at all, because AlwaysInliner doesn't invalidate existing analyses incrementally (i.e. for each modified function).

But then, now that I look closed, if we actually inline anything, then we're not preserving any analysis at all, so there is no need to update BFI in the first place.

https://github.com/llvm/llvm-project/pull/117750


More information about the llvm-commits mailing list