[PATCH] D97110: [InstrProfiling] Use nobits as __llvm_prf_cnts section type in ELF
Petr Hosek via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 23 13:38:44 PST 2021
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
More information about the llvm-commits
mailing list