Re: [PATCH] readelf: allow build with LLVM/clang

2021-07-15 Thread Mark Wielaard
Hi Timm, On Thu, Jul 15, 2021 at 07:46:15AM +0200, Timm Baeder via Elfutils-devel wrote: > FWIW, there is a different version of this patch at > https://sourceware.org/pipermail/elfutils-devel/2021q1/003674.html > that doesn't have those problems as far as I remember. Indeed, that looks more clea

Re: [PATCH] readelf: allow build with LLVM/clang

2021-07-14 Thread Timm Baeder via Elfutils-devel
On 14/07/2021 19:40, Mark Wielaard wrote: AC_CACHE_CHECK([whether gcc supports __attribute__((visibility()))], ac_cv_visibility, [dnl diff --git a/src/readelf.c b/src/readelf.c index 161d7e65..3d6f263e 100644 --- a/src/readelf.c +++ b/src/readelf.c @@ -8763,13 +8763,17 @@ print_debug_li

Re: [PATCH] readelf: allow build with LLVM/clang

2021-07-14 Thread Mark Wielaard
Hi Tomasz, On Mon, Jul 12, 2021 at 11:59:41AM +0200, Tomasz Paweł Gajc via Elfutils-devel wrote: > convert last nested function to allow build with > LLVM/clang. Original patch comes from > https://github.com/OpenMandrivaAssociation/elfutils/blob/master/elfutils-0.185-clang.patch > Tested with O

[PATCH] readelf: allow build with LLVM/clang

2021-07-12 Thread Tomasz Paweł Gajc via Elfutils-devel
convert last nested function to allow build with LLVM/clang. Original patch comes from https://github.com/OpenMandrivaAssociation/elfutils/blob/master/elfutils-0.185-clang.patch Tested with OpenMandriva Lx cooker and LLVM/clang-12.0.1 Signed-off-by: Tomasz Paweł Gajc --- configure.ac | 2 +-