Depends on D44961
Details
Diff Detail
- Build Status
Buildable 16510 Build 16510: arc lint + arc unit
Event Timeline
lld/ELF/Relocations.cpp | ||
---|---|---|
1360–1361 | With assignOffsets() this line may no longer be required. From memory it was used when a thunk was added in a later pass to a pre-existing ThunkSection and if there were no other thunk sections added we reported no addresses changed. It looks like assignOffsets() would catch this. | |
lld/ELF/Thunks.h | ||
55 | Perhaps getThunkTarget() or getThunkTargetSym() as this would reinforce the 1 target of the Thunk. | |
60 | I think that the extra symbols are to account for the presence of Arm/Thumb mapping symbols. These are aids for disassembly and aren't part of the interface. There will only ever be one "Thunk Symbol". Perhaps the comment on addSymbols() should say something like "All thunks must define a symbol, known as the thunk symbol, so that we can redirect relocations to it. The thunk may define additional symbols, but these are never targets for relocations. |
With assignOffsets() this line may no longer be required. From memory it was used when a thunk was added in a later pass to a pre-existing ThunkSection and if there were no other thunk sections added we reported no addresses changed. It looks like assignOffsets() would catch this.