[PATCH] D97110: [InstrProfiling] Use nobits as __llvm_prf_cnts section type in ELF

[Xinliang David Li via llvm-commits]

Kazu has seen this error

error: SHT_NOBITS section '__llvm_prf_cnts' cannot have non-zero
initializers

This is with lld.

David

On Tue, Feb 23, 2021 at 1:38 PM Petr Hosek via Phabricator <
reviews at reviews.llvm.org> wrote:

> phosek added a comment.
>
> In D97110#2583026 <https://reviews.llvm.org/D97110#2583026>, @Conanap
> wrote:
>
> > Hello,
> >
> > Power PC is also experiencing the same error,
> >
> >   Malformed instrumentation profile data
> >    error: No profiles could be merged.
> >
> > I built a branch of `main` with this change reverted and the error goes
> away. Would it be possible to have this reverted for now until a proper
> solution has been created? Thanks!
>
> I've managed to reproduce this locally in a Docker container. It's a bug
> in the older binutils bfd ld, the `__start_` and `__stop_` have incorrect
> addresses, it's working correctly in gold and also in newer binutils
> versions. I'll send a change to conditionally enable this feature with
> `-fbinutils-version=`, I'm just trying to identify what version of binutils
> this was fixed in.
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D97110/new/
>
> https://reviews.llvm.org/D97110
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210223/75ddc43a/attachment.html>