diff options
| author | Peter Collingbourne <peter@pcc.me.uk> | 2017-05-15 17:59:21 +0000 |
|---|---|---|
| committer | Peter Collingbourne <peter@pcc.me.uk> | 2017-05-15 17:59:21 +0000 |
| commit | 0a2678e9aaf301815c3a256e14486ce40c09032d (patch) | |
| tree | c31d2574c1fd6eaa51b3805bf648ca6a811caa26 /llvm/docs/CommandGuide/llvm-config.rst | |
| parent | c5bee3212b099b345e5f49ebab53d43874cd826e (diff) | |
| download | bcm5719-llvm-0a2678e9aaf301815c3a256e14486ce40c09032d.tar.gz bcm5719-llvm-0a2678e9aaf301815c3a256e14486ce40c09032d.zip | |
ELF: --gdb-index: Change findSection to return an InputSection.
We should only ever expect this function to return a regular
InputSection; I would not expect a function definition to be in a
MergeInputSection or EhInputSection. We were previously crashing
in writeTo if this function returned a section that was not an
InputSection because we do not set OutSec for such sections.
This can happen in practice if a function is defined in an empty
section which shares its offset-in-file with a MergeInputSection,
as in the provided test case.
A better fix for this bug would be to fix the
DWARFUnit::collectAddressRanges() interface to provide section
information (see D33183), but this at least fixes the crash.
Differential Revision: https://reviews.llvm.org/D33176
llvm-svn: 303089
Diffstat (limited to 'llvm/docs/CommandGuide/llvm-config.rst')
0 files changed, 0 insertions, 0 deletions
