Re: [PATCH 0/1] Detecting lifetime-dse issues via Valgrind [PR66487]

2024-10-03 Thread Sam James
Alexander Monakov writes: > I would like to propose Valgrind integration previously sent as RFC for trunk. > > Arsen and Sam, since you commented on the RFC I wonder if you can have > a look at the proposed configure and documentation changes and let me > know if they look fine for you? For refer

Re: [PATCH 0/1] Detecting lifetime-dse issues via Valgrind [PR66487]

2023-12-08 Thread Alexander Monakov
On Fri, 8 Dec 2023, Jakub Jelinek wrote: > On Fri, Dec 08, 2023 at 06:43:19PM +0300, Alexander Monakov wrote: > > On Fri, 8 Dec 2023, Jakub Jelinek wrote: > > > In your version, did the new function go just to libgcc.a or to > > > libgcc_s.so.1? > > > > It participates in libgcc_s link, but it

Re: [PATCH 0/1] Detecting lifetime-dse issues via Valgrind [PR66487]

2023-12-08 Thread Jakub Jelinek
On Fri, Dec 08, 2023 at 06:43:19PM +0300, Alexander Monakov wrote: > On Fri, 8 Dec 2023, Jakub Jelinek wrote: > > In your version, did the new function go just to libgcc.a or to > > libgcc_s.so.1? > > It participates in libgcc_s link, but it's not listed in the version script, > so it's not export

Re: [PATCH 0/1] Detecting lifetime-dse issues via Valgrind [PR66487]

2023-12-08 Thread Alexander Monakov
On Fri, 8 Dec 2023, Jakub Jelinek wrote: > Does VALGRIND_MAKE_MEM_UNDEFINED macro ever change onarches once implemented > there? It seems Valgrind folks take binary compatibility seriously, so that sounds unlikely. > Wouldn't this be better done by emitting the sequence inline? > Even if it is

Re: [PATCH 0/1] Detecting lifetime-dse issues via Valgrind [PR66487]

2023-12-08 Thread Jakub Jelinek
On Fri, Dec 08, 2023 at 04:49:49PM +0300, Alexander Monakov wrote: > I would like to propose Valgrind integration previously sent as RFC for trunk. > > Arsen and Sam, since you commented on the RFC I wonder if you can have > a look at the proposed configure and documentation changes and let me > k

[PATCH 0/1] Detecting lifetime-dse issues via Valgrind [PR66487]

2023-12-08 Thread Alexander Monakov
I would like to propose Valgrind integration previously sent as RFC for trunk. Arsen and Sam, since you commented on the RFC I wonder if you can have a look at the proposed configure and documentation changes and let me know if they look fine for you? For reference, gccinstall.info will say: ‘--e