diff --git a/lld/ELF/Target.cpp b/lld/ELF/Target.cpp --- a/lld/ELF/Target.cpp +++ b/lld/ELF/Target.cpp @@ -103,7 +103,8 @@ ? (Out::bufferStart + isec->getParent()->offset + isec->outSecOff) : isec->data().data(); if (isecLoc == nullptr) { - assert(isa(isec) && "No data but not synthetic?"); + assert((isa(isec) || (isec->type & SHT_NOBITS)) && + "No data but not synthetic and not SHT_NOBITS?"); continue; } if (isecLoc <= loc && loc < isecLoc + isec->getSize()) diff --git a/lld/test/ELF/mips-jalr-non-functions.s b/lld/test/ELF/mips-jalr-non-functions.s --- a/lld/test/ELF/mips-jalr-non-functions.s +++ b/lld/test/ELF/mips-jalr-non-functions.s @@ -6,7 +6,12 @@ ## relocations to avoid generating binaries that crash when executed. # RUN: llvm-mc -filetype=obj -triple=mips64-unknown-linux %s -o %t.o -# RUN: ld.lld -shared %t.o -o %t.so 2>&1 | FileCheck %s -check-prefix WARNING-MESSAGE +## Link in another object file with a .bss as a regression test: +## Previously getErrPlace() would assert when skipping over .bss sections. +## By adding another file with a .bss section before the actual %t.o we can +## reproduce this case. +# RUN: llvm-mc -filetype=obj -triple=mips64-unknown-linux %S/Inputs/common.s -o %t-common.o +# RUN: ld.lld -shared %t-common.o %t.o -o %t.so 2>&1 | FileCheck %s -check-prefix WARNING-MESSAGE # RUN: llvm-objdump --no-show-raw-insn --no-leading-addr -d %t.so | FileCheck %s .set noreorder