> -----Original Message----- > From: Prathamesh Kulkarni > Sent: 31 January 2026 10:35 > To: 'Prathamesh Kulkarni' <[email protected]>; Tobias Burnus > <[email protected]>; Matthew Malcomson <[email protected]>; > [email protected] > Subject: RE: [PATCH 2/5] libgomp: Ensure memory sync after performing > tasks > > > > > -----Original Message----- > > From: Prathamesh Kulkarni <[email protected]> > > Sent: 20 January 2026 09:48 > > To: Tobias Burnus <[email protected]>; Matthew Malcomson > > <[email protected]>; [email protected]; Jakub Jelinek > > <[email protected]> > > Cc: Thomas Schwinge <[email protected]>; Andrew Stubbs > > <[email protected]>; Tom de Vries <[email protected]>; Sebastian Huber > > <[email protected]> > > Subject: RE: [PATCH 2/5] libgomp: Ensure memory sync after > performing > > tasks > > > > External email: Use caution opening links or attachments > > > > > > > -----Original Message----- > > > From: Tobias Burnus <[email protected]> > > > Sent: 20 January 2026 02:39 > > > To: Prathamesh Kulkarni <[email protected]>; Matthew > Malcomson > > > <[email protected]>; [email protected]; Jakub Jelinek > > > <[email protected]> > > > Cc: Thomas Schwinge <[email protected]>; Andrew Stubbs > > > <[email protected]>; Tom de Vries <[email protected]>; Sebastian > Huber > > > <[email protected]> > > > Subject: Re: [PATCH 2/5] libgomp: Ensure memory sync after > > performing > > > tasks > > > > > > External email: Use caution opening links or attachments > > > > > > > > > Hi Prathamesh, > > > > > > Prathamesh Kulkarni wrote: > > > > The attached patch similarly adds target pragmas to the test as > > > previous patch. > > > > Does it look OK ? > > > > > > Thanks for revising Matthew's patch. > > > > > > Yes, LGTM. – However, except for 'Daily bump.', it seems to be > > > uncommon to have a '.' at the end of the first line (summary) of > the > > > git commit; hence, consider removing it. > > Thanks, adjusted the commit summary and committed to trunk in: > > > https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=304d08fea9e68c5b48065328 > > 16b58d33a15a917d > Hi Tobias, > Sorry I messed up the newly added test (pr122356.c) while committing, > and Linaro CI reported the test failing: > https://linaro.atlassian.net/browse/GNU-1822 > > This happens because full_data is not initialized to 0 on second call > to test() on host, which results in call to abort. > The attached patch modifies pr122356.c (and similarly pr122314.c) to > initialize full_data to 0 at beginning of test(). > > With patch, running make check-target-libgomp > RUNTESTFLAGS="c.exp=pr122356.c" shows: > > === libgomp Summary === > # of expected passes 2 > > OK to commit ? Hi, ping: https://gcc.gnu.org/pipermail/gcc-patches/2026-January/707277.html
Thanks, Prathamesh > > Thanks, > Prathamesh > > > > Thanks, > > Prathamesh > > > > > > Tobias
