[PATCH] D28626: RFC: Generalize inverted gc dependencies
Rafael Avila de Espindola via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 23 11:38:39 PST 2017
Evgeniy Stepanov via Phabricator <reviews at reviews.llvm.org> writes:
> eugenis added inline comments.
>
>
> ================
> Comment at: llvm/include/llvm/Support/ELF.h:752
> + // FIXME: Tentative value.
> + SHF_ASSOCIATED = 0x1000U,
> +
> ----------------
> this is conflicting with XCORE_SHF_DP_SECTION a few lines below, which is printed in assembly as "d" flag instead of "o"
And XCORE_SHF_CP_SECTION is conflicting with SHF_COMPRESSED :-(
The story seems to be:
* Ever since xcore was added in r58838 it has sections named like cp.foo
and dp.foo.
* The cp/dp become flags in r79077.
* When MC came we started writing them to disk.
This is plainly a bug. These flags have to be moved to SHF_MASKPROC and
until they do they cannot be expected to work with standard flags:
Thees sections cannot be compressed and cannot have inverted gc edges.
Robert, is the MC produced objects used or is another assembler used?
Cheers,
Rafael
More information about the llvm-commits
mailing list