Re: [Patch] libgomp.texi: Document omp_pause_resource{,_all} and omp_target_memcpy*

2024-01-23 Thread Tobias Burnus
Hi Sandra, thanks for the comments and proposals! An updated version is enclosed. Unless you find more issues, I intent to commit it soon. Tobias PS: I think besides filling gaps, some editing wouldn't harm; if you feel bored ... https://gcc.gnu.org/onlinedocs/libgomp/ libgomp.texi: Documen

Re: [Patch] libgomp.texi: Document omp_pause_resource{,_all} and omp_target_memcpy* (was: [Patch] libgomp.texi: Document omp_pause_resource{,_all})

2024-01-14 Thread Sandra Loosemore
On 1/14/24 16:15, Tobias Burnus wrote: +@node omp_target_memcpy +@subsection @code{omp_target_memcpy} -- Copy data between devices +@table @asis +@item @emph{Description}: +This routine tests copies @var{length} of bytes of data from the device +identified by device number @var{src_device_num} t

[Patch] libgomp.texi: Document omp_pause_resource{,_all} and omp_target_memcpy* (was: [Patch] libgomp.texi: Document omp_pause_resource{,_all})

2024-01-14 Thread Tobias Burnus
Hi Sandra, hi all, Sandra Loosemore: On 1/14/24 07:26, Tobias Burnus wrote: I have some minor nits about typos and copy-editing. Thanks. That's the downside of doing editing while being sleepy on a train. Updated and extended version (documenting also omp_target_memcpy) is attached. Warning:

Re: [Patch] libgomp.texi: Document omp_pause_resource{,_all}

2024-01-14 Thread Sandra Loosemore
On 1/14/24 07:26, Tobias Burnus wrote: This documents two more OpenMP (5.0) routines, omp_pause_resource and omp_pause_resource_all. Comments, remarks, suggestions - to the patch or the documentation in general? I have some minor nits about typos and copy-editing. I assume the formatting of

[Patch] libgomp.texi: Document omp_pause_resource{,_all}

2024-01-14 Thread Tobias Burnus
This documents two more OpenMP (5.0) routines, omp_pause_resource and omp_pause_resource_all. Comments, remarks, suggestions - to the patch or the documentation in general? Tobias PS: When looking at it, I found an issue in the spec with regards to a new constant (post TR12, hence, not adde