mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-07-22 09:29:31 +00:00
c39f5dd0e2
Fixes PR19582. Previously, when an asm assignment (.set or =) was created, we would look up the section immediately in MCSymbol::setVariableValue. This caused symbols to receive the wrong section if the RHS of the assignment had not been seen yet. This had a knock-on effect in the object file emitters, causing them to emit extra symbols, or to give symbols the wrong visibility or the wrong section. For example, in the following asm: .data .Llocal: .text leaq .Llocal1(%rip), %rdi .Llocal1 = .Llocal2 .Llocal2 = .Llocal the first assignment would give .Llocal1 a null section, which would never get fixed up by the second assignment. This would cause the ELF object file emitter to consider .Llocal1 to be an undefined symbol and give it external linkage, even though .Llocal1 should not have been emitted at all in the object file. Or in the following asm: alias_to_local = Ltmp0 Ltmp0: the Mach-O object file emitter would give the alias_to_local symbol a n_type of N_SECT and a n_sect of 0. This is invalid under the Mach-O specification, which requires N_SECT symbols to receive a non-zero section number if the symbol is defined in a section in the object file. https://developer.apple.com/library/mac/documentation/DeveloperTools/Conceptual/MachORuntime/#//apple_ref/c/tag/nlist After this change we do not look up the section when the assignment is created, but instead look it up on demand and store it in Section, which is treated as a cache if the symbol is a variable symbol. This change also fixes a bug in MCExpr::FindAssociatedSection. Previously, if we saw a subtraction, we would return the first referenced section, even in cases where we should have been returning the absolute pseudo-section. Now we always return the absolute pseudo-section for expressions that subtract two section-derived expressions. This isn't always correct (e.g. if one of the sections ends up being laid out at an absolute address), but it's probably the best we can do without more context. This allows us to remove code in two places where we appear to have been working around this bug, in MachObjectWriter::markAbsoluteVariableSymbols and in X86AsmPrinter::EmitStartOfAsmFile. Re-applies r233595 (aka D8586), which was reverted in r233898. Differential Revision: http://reviews.llvm.org/D8798 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@233995 91177308-0d34-0410-b5e6-96231b3b80d8 |
||
|---|---|---|
| .. | ||
| MCDisassembler | ||
| MCParser | ||
| CMakeLists.txt | ||
| ConstantPools.cpp | ||
| ELFObjectWriter.cpp | ||
| LLVMBuild.txt | ||
| MachObjectWriter.cpp | ||
| Makefile | ||
| MCAsmBackend.cpp | ||
| MCAsmInfo.cpp | ||
| MCAsmInfoCOFF.cpp | ||
| MCAsmInfoDarwin.cpp | ||
| MCAsmInfoELF.cpp | ||
| MCAsmStreamer.cpp | ||
| MCAssembler.cpp | ||
| MCCodeEmitter.cpp | ||
| MCCodeGenInfo.cpp | ||
| MCContext.cpp | ||
| MCDwarf.cpp | ||
| MCELF.cpp | ||
| MCELFObjectTargetWriter.cpp | ||
| MCELFStreamer.cpp | ||
| MCExpr.cpp | ||
| MCInst.cpp | ||
| MCInstPrinter.cpp | ||
| MCInstrAnalysis.cpp | ||
| MCLabel.cpp | ||
| MCLinkerOptimizationHint.cpp | ||
| MCMachObjectTargetWriter.cpp | ||
| MCMachOStreamer.cpp | ||
| MCNullStreamer.cpp | ||
| MCObjectFileInfo.cpp | ||
| MCObjectStreamer.cpp | ||
| MCObjectWriter.cpp | ||
| MCRegisterInfo.cpp | ||
| MCSection.cpp | ||
| MCSectionCOFF.cpp | ||
| MCSectionELF.cpp | ||
| MCSectionMachO.cpp | ||
| MCStreamer.cpp | ||
| MCSubtargetInfo.cpp | ||
| MCSymbol.cpp | ||
| MCSymbolizer.cpp | ||
| MCTargetOptions.cpp | ||
| MCValue.cpp | ||
| MCWin64EH.cpp | ||
| MCWinEH.cpp | ||
| StringTableBuilder.cpp | ||
| SubtargetFeature.cpp | ||
| WinCOFFObjectWriter.cpp | ||
| WinCOFFStreamer.cpp | ||
| YAML.cpp | ||