Re: [gentoo-dev] [PATCH] profiles: create USE=valgrind global USE flag

2023-05-27 Thread Sam James
Alexe Stefan writes: > So it's only useful for developers that already use valgrind. Why would it be useful to anybody else? It can be useful if you suspect you may want to use Valgrind with system packages at some point and you don't want then to recompile, sure. I'm not sure if I understand

Re: [gentoo-dev] [PATCH] profiles: create USE=valgrind global USE flag

2023-05-27 Thread Alexe Stefan
So it's only useful for developers that already use valgrind. sâm., 27 mai 2023, 17:13 Sam James a scris: > > Alexe Stefan writes: > > > In that case, is it worth it to enable USE=valgrind globally? > > I have, and I'd say others interested in using Valgrind should too. > > >

Re: [gentoo-dev] [PATCH] profiles: create USE=valgrind global USE flag

2023-05-27 Thread Sam James
Alexe Stefan writes: > In that case, is it worth it to enable USE=valgrind globally? I have, and I'd say others interested in using Valgrind should too. signature.asc Description: PGP signature

Re: [gentoo-dev] [PATCH] profiles: create USE=valgrind global USE flag

2023-05-27 Thread Alexe Stefan
In that case, is it worth it to enable USE=valgrind globally? vin., 26 mai 2023, 10:44 Sam James a scris: > > Sam James writes: > > > [[PGP Signed Part:Undecided]] > > > > Sam James writes: > > > >> Alexe Stefan writes: > >> > >>> Does enabling USE=valgrind impact runtime performance in any

Re: [gentoo-dev] [PATCH] profiles: create USE=valgrind global USE flag

2023-05-26 Thread Sam James
Sam James writes: > [[PGP Signed Part:Undecided]] > > Sam James writes: > >> Alexe Stefan writes: >> >>> Does enabling USE=valgrind impact runtime performance in any way? >> >> A very small amount because it adds a check at runtime for whether >> the application is running under Valgrind. The

Re: [gentoo-dev] [PATCH] profiles: create USE=valgrind global USE flag

2023-05-14 Thread Sam James
Sam James writes: > Alexe Stefan writes: > >> Does enabling USE=valgrind impact runtime performance in any way? > > A very small amount because it adds a check at runtime for whether > the application is running under Valgrind. The compiler may be able > to optimise this a bit if it can

Re: [gentoo-dev] [PATCH] profiles: create USE=valgrind global USE flag

2023-05-14 Thread Sam James
Alexe Stefan writes: > Does enabling USE=valgrind impact runtime performance in any way? A very small amount because it adds a check at runtime for whether the application is running under Valgrind. The compiler may be able to optimise this a bit if it can determine it's unlikely (if the

Re: [gentoo-dev] [PATCH] profiles: create USE=valgrind global USE flag

2023-05-14 Thread Alexe Stefan
Does enabling USE=valgrind impact runtime performance in any way? dum., 14 mai 2023, 19:46 Arsen Arsenović a scris: > > Sam James writes: > > > This always has the same meaning in packages - build in annotations to > help > > with e.g. custom memory allocators to reduce noise and improve >

Re: [gentoo-dev] [PATCH] profiles: create USE=valgrind global USE flag

2023-05-14 Thread Arsen Arsenović
Sam James writes: > This always has the same meaning in packages - build in annotations to help > with e.g. custom memory allocators to reduce noise and improve Valgrind's > accuracy. > > All invalid uses of this were already fixed (cases where it was used to > control > running the testsuite

Re: [gentoo-dev] [PATCH] profiles: create USE=valgrind global USE flag

2023-05-14 Thread David Seifert
On Sun, 2023-05-14 at 09:26 +0100, Sam James wrote: > This always has the same meaning in packages - build in annotations to > help > with e.g. custom memory allocators to reduce noise and improve > Valgrind's accuracy. > > All invalid uses of this were already fixed (cases where it was used > to

[gentoo-dev] [PATCH] profiles: create USE=valgrind global USE flag

2023-05-14 Thread Sam James
This always has the same meaning in packages - build in annotations to help with e.g. custom memory allocators to reduce noise and improve Valgrind's accuracy. All invalid uses of this were already fixed (cases where it was used to control running the testsuite under Valgrind which we don't want