[lld] r357143 - [WebAssembly] Improve invalid relocation error message
Sam Clegg via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 27 19:02:08 PDT 2019
Author: sbc
Date: Wed Mar 27 19:02:07 2019
New Revision: 357143
URL: http://llvm.org/viewvc/llvm-project?rev=357143&view=rev
Log:
[WebAssembly] Improve invalid relocation error message
This message now matches the equivalent message in the ELF linker.
Differential Revision: https://reviews.llvm.org/D59860
Modified:
lld/trunk/test/wasm/undefined-data.ll
lld/trunk/wasm/InputChunks.cpp
lld/trunk/wasm/InputChunks.h
lld/trunk/wasm/Writer.cpp
Modified: lld/trunk/test/wasm/undefined-data.ll
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/test/wasm/undefined-data.ll?rev=357143&r1=357142&r2=357143&view=diff
==============================================================================
--- lld/trunk/test/wasm/undefined-data.ll (original)
+++ lld/trunk/test/wasm/undefined-data.ll Wed Mar 27 19:02:07 2019
@@ -13,4 +13,4 @@ entry:
}
; 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
Modified: lld/trunk/wasm/InputChunks.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/wasm/InputChunks.cpp?rev=357143&r1=357142&r2=357143&view=diff
==============================================================================
--- lld/trunk/wasm/InputChunks.cpp (original)
+++ lld/trunk/wasm/InputChunks.cpp Wed Mar 27 19:02:07 2019
@@ -22,7 +22,7 @@ using namespace llvm::support::endian;
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: \
Modified: lld/trunk/wasm/InputChunks.h
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/wasm/InputChunks.h?rev=357143&r1=357142&r2=357143&view=diff
==============================================================================
--- lld/trunk/wasm/InputChunks.h (original)
+++ lld/trunk/wasm/InputChunks.h Wed Mar 27 19:02:07 2019
@@ -218,6 +218,8 @@ protected:
} // namespace wasm
std::string toString(const wasm::InputChunk *);
+StringRef reloctTypeToString(uint8_t RelocType);
+
} // namespace lld
#endif // LLD_WASM_INPUT_CHUNKS_H
Modified: lld/trunk/wasm/Writer.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/wasm/Writer.cpp?rev=357143&r1=357142&r2=357143&view=diff
==============================================================================
--- lld/trunk/wasm/Writer.cpp (original)
+++ lld/trunk/wasm/Writer.cpp Wed Mar 27 19:02:07 2019
@@ -1141,18 +1141,6 @@ void Writer::processRelocations(InputChu
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<DefinedData>(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<GlobalSymbol>(Sym) && !Sym->isInGOT()) {
@@ -1161,6 +1149,23 @@ void Writer::processRelocations(InputChu
}
}
}
+
+ 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");
+ }
+ }
+ }
}
}
More information about the llvm-commits
mailing list