Index: llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMELFStreamer.cpp =================================================================== --- llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMELFStreamer.cpp +++ llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMELFStreamer.cpp @@ -464,7 +464,7 @@ void emitUnwindRaw(int64_t Offset, const SmallVectorImpl &Opcodes); void ChangeSection(MCSection *Section, const MCExpr *Subsection) override { - LastMappingSymbols[getPreviousSection().first] = std::move(LastEMSInfo); + LastMappingSymbols[getCurrentSection().first] = std::move(LastEMSInfo); MCELFStreamer::ChangeSection(Section, Subsection); auto LastMappingSymbol = LastMappingSymbols.find(Section); if (LastMappingSymbol != LastMappingSymbols.end()) { Index: llvm/trunk/test/MC/ARM/multi-section-mapping.s =================================================================== --- llvm/trunk/test/MC/ARM/multi-section-mapping.s +++ llvm/trunk/test/MC/ARM/multi-section-mapping.s @@ -21,14 +21,31 @@ .arm add r0, r0, r0 +@ Similarly no $t if we change back .starts_thumb using .pushsection + .pushsection .starts_thumb + .thumb + adds r0, r0, r0 + +@ When we change back to .text using .popsection .thumb is still active, so we +@ should emit a $t + .popsection + add r0, r0, r0 + +@ .ident does a push then pop of the .comment section, so the .word should +@ cause $d to appear in the .text section + .ident "ident" + .word 0 + @ With all those constraints, we want: -@ + .text to have $a at 0 and no others +@ + .text to have $a at 0, $t at 8, $d at 12 @ + .wibble to have $a at 0 @ + .starts_thumb to have $t at 0 @ + .starts_data to have $d at 0 @ CHECK: 00000000 .text 00000000 $a @ CHECK-NEXT: 00000000 .wibble 00000000 $a +@ CHECK-NEXT: 0000000a .text 00000000 $d @ CHECK-NEXT: 00000000 .starts_thumb 00000000 $t +@ CHECK-NEXT: 00000008 .text 00000000 $t @ CHECK-NOT: ${{[adt]}}