diff options
| author | Tobias Burnus <tobias@codesourcery.com> | 2021-01-28 14:07:28 +0000 |
|---|---|---|
| committer | Hafiz Abid Qadeer <abidh@codesourcery.com> | 2021-01-28 14:54:43 +0000 |
| commit | 70ea15b88953e56681b997373fb11c97eeb05c4e (patch) | |
| tree | 9fc3629ccd6cf14cc574030b5ceb25b7a7765984 /llvm/lib/Support/APFloat.cpp | |
| parent | c818fa6729e02820c656431126d724d643001b70 (diff) | |
| download | llvm-70ea15b88953e56681b997373fb11c97eeb05c4e.zip llvm-70ea15b88953e56681b997373fb11c97eeb05c4e.tar.gz llvm-70ea15b88953e56681b997373fb11c97eeb05c4e.tar.bz2 | |
[MC][ELF] Fix accepting abbreviated form with sh_flags and sh_entsize
Followup to D92052 as I missed an issue as shown via GCC bug https://gcc.gnu.org/PR97827, namely: (e.g.) ".rodata." implies ELF::SHF_ALLOC.
Crossref:
- D73999 / commit 75af9da755721123e62b45cd0bc0c5e688a9722a
added for LLVM 11 a check that sh_flags and sh_entsize (and sh_type)
changes are an error, in line with GNU assembler.
- D92052 / commit 1deff4009e0ae661b03682901bf6932297ce7ea1
permitted the abbreviated form which many assemblers accept and
GCC generates: while the first .section contains the flags and entsize,
subsequent sections simply contain the name without repeating entsize or
flags.
However, the latter patch missed in the check that some flags are automatically set, e.g. '.rodata." implies ELF::SHF_ALLOC.
Related https://bugs.llvm.org/show_bug.cgi?id=48201
Reviewed By: jhenderson
Differential Revision: https://reviews.llvm.org/D94072
Diffstat (limited to 'llvm/lib/Support/APFloat.cpp')
0 files changed, 0 insertions, 0 deletions