[llvm] [Mips] Fix clang integrated assembler generates incorrect relocations… (PR #83115)

YunQiang Su via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 15 00:11:26 PDT 2024


wzssyqa wrote:

> > In fact we can update the generic asm scan for all ports, instead mips only. See `MCSymbol *MCContext::createSymbol(StringRef Name, bool AlwaysAddSuffix, bool CanBeUnnamed)` in `llvm/lib/MC/MCContext.cpp`
> 
> Can you explain it in more detail?

Integrated assembler scan the asm code in `MCParser/MCAsmParser.cpp`, and if it find a symbol, it will call `createSymbol` just now, so that it cannot know that this symbol is defined in the same asm code later.

That's the real problem, and this problem effects all architectures.

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


More information about the llvm-commits mailing list