[llvm] [DebugInfo] Emit skeleton to avoid mismatching inlining flags (PR #153568)
Qiu Chaofan via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 18 01:29:35 PDT 2025
ecnelises wrote:
> I see that [e731a26](https://github.com/llvm/llvm-project/commit/e731a2678c7cf81b1d3817489a52b519cb14f85e) seems to have handled this to avoid creating a lot of tiny CUs for all the imported CUs in each backend compile in this scenario.
Thanks! Yes, e731a2678c7cf81b1d3817489a52b519cb14f85e already addressed the issue of split-dwarf-omit-empty.ll added by 418120556398c01550d42500d56e6d328290185b.
> Not sure I follow - looking at the text of the change it does look like a revert. Is there some difference in the code this patch is adding compared to the code that was removed back in the original?
Technically this is a revert. I did not try an old enough LLVM to exactly contain the commit and test whether it has the ICE at that time. Anyway since the original issue is resolved by a later commit (e731a2678c7cf81b1d3817489a52b519cb14f85e), and reverting this fixes another ICE, this can also be regarded as a fix, I think which is what @Sockke means. :-)
https://github.com/llvm/llvm-project/pull/153568
More information about the llvm-commits
mailing list