[Patch] libgomp: Add tests for omp_atv_serialized and deprecate omp_atv_sequential.

Jakub Jelinek jakub@redhat.com
Mon Oct 11 09:49:40 GMT 2021


On Mon, Oct 11, 2021 at 11:40:54AM +0200, Marcel Vollweiler wrote:
> libgomp: Add tests for omp_atv_serialized and deprecate omp_atv_sequential.
> 
> The variable omp_atv_sequential was replaced by omp_atv_serialized in OpenMP
> 5.1. This was already implemented by Jakub (C/C++, commit ea82325afec) and
> Tobias (Fortran, commit fff15bad1ab).
> 
> This patch adds two tests to check if omp_atv_serialized is available (one test
> for C/C++ and one for Fortran). Besides that omp_atv_sequential is marked as
> deprecated in C/C++ and Fortran for OpenMP 5.1.
> 
> libgomp/ChangeLog:
> 
> 	* allocator.c (omp_init_allocator): Replace omp_atv_sequential with
> 	omp_atv_serialized.
> 	* omp.h.in: Add deprecated flag for omp_atv_sequential.
> 	* omp_lib.f90.in: Add deprecated flag for omp_atv_sequential.
> 	* testsuite/libgomp.c-c++-common/alloc-10.c: New test.
> 	* testsuite/libgomp.fortran/alloc-12.f90: New test.

LGTM, except one nit.

> --- /dev/null
> +++ b/libgomp/testsuite/libgomp.c-c++-common/alloc-10.c
> +}
> \ No newline at end of file

Please make sure the file ends with a newline before committing.

	Jakub



More information about the Gcc-patches mailing list