[llvm] [CodeGen] Use temp symbol for MBBs (PR #95031)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 13 01:28:33 PDT 2024


aengelke wrote:

> Shall we consider this blocked by the resolution to BPF https://github.com/llvm/llvm-project/pull/95103 ?

I don't think so. This maintains the existing functionality of separate label/global prefixes. Removing this distinction and cleaning up the code can be done later once all BPF also uses the same local/global prefix.

The only BPF-related change is that basic block labels are no longer in the symbol table if they are not referenced by inline assembly. That should be fine, because @4ast said in #95103:

> >  basic block labels would no longer be added to the symbol table
>
> that's a regression. they didn't meant to be in symbol table.

So this PR reduces this regression (although it does not fix it entirely, that's what #95103 originally was).

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


More information about the llvm-commits mailing list