[PATCH] D44193: [ELF] - Fix crash relative to SHF_LINK_ORDER sections.
Rafael Ávila de Espíndola via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 8 04:44:36 PST 2018
rafael added a comment.
George Rimar via Phabricator via llvm-commits
<llvm-commits at lists.llvm.org> writes:
> Index: ELF/OutputSections.cpp
> ===================================================================
>
> - ELF/OutputSections.cpp +++ ELF/OutputSections.cpp @@ -100,7 +100,8 @@ Flags = IS->Flags; } else { // Otherwise, check if new type or flags are compatible with existing ones.
> - if ((Flags & (SHF_ALLOC | SHF_TLS)) != (IS->Flags & (SHF_ALLOC | SHF_TLS))) + if ((Flags & (SHF_ALLOC | SHF_TLS | SHF_LINK_ORDER)) != + (IS->Flags & (SHF_ALLOC | SHF_TLS | SHF_LINK_ORDER)))
Please move SHF_ALLOC | SHF_TLS | SHF_LINK_ORDER to a mask variable.
LGTM with that.
Cheers,
Rafael
https://reviews.llvm.org/D44193
More information about the llvm-commits
mailing list