Index: lld/trunk/test/wasm/undefined-data.ll =================================================================== --- lld/trunk/test/wasm/undefined-data.ll +++ lld/trunk/test/wasm/undefined-data.ll @@ -13,4 +13,4 @@ } ; UNDEF: undefined symbol: data_external -; BADRELOC: undefined-data.ll.tmp.o: relocation of type R_WASM_MEMORY_ADDR_* against undefined data symbol: data_external +; BADRELOC: undefined-data.ll.tmp.o: relocation R_WASM_MEMORY_ADDR_LEB cannot be used againt symbol data_external; recompile with -fPIC Index: lld/trunk/wasm/InputChunks.h =================================================================== --- lld/trunk/wasm/InputChunks.h +++ lld/trunk/wasm/InputChunks.h @@ -218,6 +218,8 @@ } // namespace wasm std::string toString(const wasm::InputChunk *); +StringRef reloctTypeToString(uint8_t RelocType); + } // namespace lld #endif // LLD_WASM_INPUT_CHUNKS_H Index: lld/trunk/wasm/InputChunks.cpp =================================================================== --- lld/trunk/wasm/InputChunks.cpp +++ lld/trunk/wasm/InputChunks.cpp @@ -22,7 +22,7 @@ using namespace lld; using namespace lld::wasm; -static StringRef reloctTypeToString(uint8_t RelocType) { +StringRef lld::reloctTypeToString(uint8_t RelocType) { switch (RelocType) { #define WASM_RELOC(NAME, REL) \ case REL: \ Index: lld/trunk/wasm/Writer.cpp =================================================================== --- lld/trunk/wasm/Writer.cpp +++ lld/trunk/wasm/Writer.cpp @@ -1141,18 +1141,6 @@ File->TypeMap[Reloc.Index] = registerType(Types[Reloc.Index]); File->TypeIsUsed[Reloc.Index] = true; break; - case R_WASM_MEMORY_ADDR_SLEB: - case R_WASM_MEMORY_ADDR_I32: - case R_WASM_MEMORY_ADDR_LEB: { - DataSymbol *DataSym = File->getDataSymbol(Reloc.Index); - if (!Config->Relocatable && !isa(DataSym) && - !DataSym->isWeak()) - error(File->getName() + - ": relocation of type R_WASM_MEMORY_ADDR_* " - "against undefined data symbol: " + - DataSym->getName()); - break; - } case R_WASM_GLOBAL_INDEX_LEB: { auto* Sym = File->getSymbols()[Reloc.Index]; if (!isa(Sym) && !Sym->isInGOT()) { @@ -1161,6 +1149,23 @@ } } } + + if (!Config->Relocatable) { + switch (Reloc.Type) { + // These relocations types appear the code section. + // They should never appear in code compiled with -fPIC. + case R_WASM_TABLE_INDEX_SLEB: + case R_WASM_MEMORY_ADDR_I32: + case R_WASM_MEMORY_ADDR_LEB: { + auto *Sym = File->getSymbols()[Reloc.Index]; + if (Sym->isUndefined() && !Sym->isWeak()) + error(File->getName() + ": relocation " + + reloctTypeToString(Reloc.Type) + + " cannot be used againt symbol " + Sym->getName() + + "; recompile with -fPIC"); + } + } + } } }