[PATCH v2 6/7] configure: Add --disable-demangler

2023-02-08 Thread Ilya Leoshkevich via Elfutils-devel
__cxa_demangle is normally implemented in the C++ runtime library, instrumenting which for MSan is a hassle. Add a knob for disbling it. Signed-off-by: Ilya Leoshkevich --- configure.ac | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 7

[PATCH v2 7/7] configure: Add --enable-sanitize-memory

2023-02-08 Thread Ilya Leoshkevich via Elfutils-devel
Add support for clang Memory Sanitizer [1], which detects the usage of uninitialized values. While elfutils itself is already checked with valgrind, checking code that depends on elfutils requires elfutils to be built with MSan. MSan is not linked into shared libraries, and is linked into executab

[PATCH v2 4/7] x86_64_return_value_location: Support lvalue and rvalue references

2023-02-08 Thread Ilya Leoshkevich via Elfutils-devel
On the low level, they are the same as pointers. Signed-off-by: Ilya Leoshkevich --- backends/x86_64_retval.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/backends/x86_64_retval.c b/backends/x86_64_retval.c index f9114cb1..e668eacc 100644 --- a/backends/x86_64_retval.c +++ b/backends/x8

[PATCH v2 5/7] configure: Use -fno-addrsig if possible

2023-02-08 Thread Ilya Leoshkevich via Elfutils-devel
By default, clang produces .llvm_addrsig sections [1]. The GNU toolchain does not know how to handle them yet [2], so just ask clang not to generate them for the time being. [1] https://llvm.org/docs/Extensions.html#sht-llvm-addrsig-section-address-significance-table [2] https://gcc.gnu.org/bugzi

[PATCH v2 0/7] Add Memory Sanitizer support

2023-02-08 Thread Ilya Leoshkevich via Elfutils-devel
Hi, I've made the updates suggested so far and rebased on top of the latest master. Please take a look. v1: https://sourceware.org/pipermail/elfutils-devel/2023q1/005831.html v1 -> v2: * Drop the unnecessary and the integrated patches. * Add a comment to the xdefault_pattern patch. * Add extern t

[PATCH v2 1/7] libasm: Fix xdefault_pattern initialization

2023-02-08 Thread Ilya Leoshkevich via Elfutils-devel
clang complains: asm_newscn.c:48:22: error: field 'pattern' with variable sized type 'struct FillPattern' not at the end of a struct or class is a GNU extension [-Werror,-Wgnu-variable-sized-type-not-at-end] struct FillPattern pattern; ^ Fix by using a union i

[PATCH v2 3/7] readelf: Fix set but not used parameter

2023-02-08 Thread Ilya Leoshkevich via Elfutils-devel
clang complains: readelf.c:12205:72: error: parameter 'desc' set but not used [-Werror,-Wunused-but-set-parameter] handle_bit_registers (const Ebl_Register_Location *regloc, const void *desc, ^ Mark Wielaard says:

[PATCH v2 2/7] printversion: Fix unused variable

2023-02-08 Thread Ilya Leoshkevich via Elfutils-devel
clang complains: debuginfod.cxx:354:1: error: unused variable 'apba__' [-Werror,-Wunused-const-variable] ARGP_PROGRAM_BUG_ADDRESS_DEF = PACKAGE_BUGREPORT; ^ ../lib/printversion.h:47:21: note: expanded from macro 'ARGP_PROGRAM_BUG_ADDRESS_DEF' const char *const apba__ __asm

Re: ☠ Buildbot (Sourceware): elfutils - failed test (failure) (master)

2023-02-08 Thread Mark Wielaard
Hi Ilya, On Wed, 2023-02-08 at 17:57 +, builder--- via Elfutils-devel wrote: > A new failure has been detected on builder elfutils-gentoo-sparc while > building elfutils. > > Full details are available at: > https://builder.sourceware.org/buildbot/#builders/225/builds/12 > > Build state

Re: [PATCH RFC 07/11] addr2line: Do not test demangling in run-addr2line-i-test.sh

2023-02-08 Thread Mark Wielaard
Hi Ilya, On Mon, 2023-02-06 at 23:25 +0100, Ilya Leoshkevich via Elfutils-devel wrote: > There is run-addr2line-i-demangle-test.sh for that. Well spotted. Pushed as is. Thanks, Mark > Signed-off-by: Ilya Leoshkevich > --- > tests/run-addr2line-i-test.sh | 14 +++--- > 1 file changed,

Re: [PATCH RFC 10/11] configure: Add --disable-demangle

2023-02-08 Thread Mark Wielaard
Hi Ilya, On Mon, 2023-02-06 at 23:25 +0100, Ilya Leoshkevich via Elfutils-devel wrote: > __cxa_demangle is normally implemented in the C++ runtime library, > instrumenting which for MSan is a hassle. Add a knob for disabling it. > > Signed-off-by: Ilya Leoshkevich > --- > configure.ac | 8 +

☺ Buildbot (Sourceware): elfutils - build successful (master)

2023-02-08 Thread builder--- via Elfutils-devel
A restored build has been detected on builder elfutils-gentoo-sparc while building elfutils. Full details are available at: https://builder.sourceware.org/buildbot/#builders/225/builds/13 Build state: build successful Revision: 6cb1547e812e9c2eeca698b38d8ab2785b7a7abf Worker: gentoo-sparc Bu

☠ Buildbot (Sourceware): elfutils - failed test (failure) (master)

2023-02-08 Thread builder--- via Elfutils-devel
A new failure has been detected on builder elfutils-gentoo-sparc while building elfutils. Full details are available at: https://builder.sourceware.org/buildbot/#builders/225/builds/12 Build state: failed test (failure) Revision: 85c82b18475dfe1cbbc4250cfbec7844d233100b Worker: gentoo-sparc

Re: [PATCH RFC 06/11] Initialize reglocs for VMCOREINFO

2023-02-08 Thread Mark Wielaard
Hi Ilya, On Mon, 2023-02-06 at 23:25 +0100, Ilya Leoshkevich via Elfutils-devel wrote: > MSan complains: > > Uninitialized value was created by an allocation of 'reglocs' in the > stack frame >#0 0x562d35c686f0 in handle_core_note elfutils/src/readelf.c:12674:3 >#const Ebl_Re

Re: [PATCH RFC 05/11] readelf: Fix set but not used variable

2023-02-08 Thread Mark Wielaard
Hi Ilya, On Mon, 2023-02-06 at 23:25 +0100, Ilya Leoshkevich via Elfutils-devel wrote: > clang complains: > > readelf.c:10250:10: error: variable 'nculist' set but not used > [-Werror,-Wunused-but-set-variable] > size_t nculist = 0; > ^ > > Fix by deleting it. yeah, this

Re: [PATCH RFC 04/11] readelf: Fix set but not used parameter

2023-02-08 Thread Mark Wielaard
Hi Ilya, On Mon, 2023-02-06 at 23:25 +0100, Ilya Leoshkevich via Elfutils-devel wrote: > clang complains: > > readelf.c:12205:72: error: parameter 'desc' set but not used > [-Werror,-Wunused-but-set-parameter] > handle_bit_registers (const Ebl_Register_Location *regloc, const void > *de

☺ Buildbot (Sourceware): elfutils - build successful (master)

2023-02-08 Thread builder--- via Elfutils-devel
A restored build has been detected on builder elfutils-gentoo-sparc while building elfutils. Full details are available at: https://builder.sourceware.org/buildbot/#builders/225/builds/11 Build state: build successful Revision: 53b596ef4018693403395d702045c15762af3da7 Worker: gentoo-sparc Bu

Re: ☠ Buildbot (Sourceware): elfutils - failed test (failure) (master)

2023-02-08 Thread Frank Ch. Eigler via Elfutils-devel
Hi - > > A new failure has been detected on builder elfutils-gentoo-sparc while > > building elfutils. > > > > Full details are available at: > > https://builder.sourceware.org/buildbot/#builders/225/builds/10 > > - test-suite.log: > > https://builder.sourceware.org/buildbot/#builder

Re: [PATCH RFC 03/11] printversion: Fix unused variable

2023-02-08 Thread Ilya Leoshkevich via Elfutils-devel
On Tue, 2023-02-07 at 21:44 +0100, Mark Wielaard wrote: > Hi Ilya (CC Frank), > > On Mon, Feb 06, 2023 at 11:25:05PM +0100, Ilya Leoshkevich via > Elfutils-devel wrote: > > clang complains: > > > >     debuginfod.cxx:354:1: error: unused variable 'apba__' [- > > Werror,-Wunused-const-variable] >