[PATCH] D44193: [ELF] - Fix crash relative to SHF_LINK_ORDER sections.

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 7 10:12:59 PST 2018


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


More information about the llvm-commits mailing list