Re: [PATCH][regression][OpenMP][PR120286] Double free detected when using openmp

Jerry Delisle <[email protected]>
Newsgroups gmane.comp.gcc.patches,gmane.comp.gcc.fortran
Message-ID <CAJJq7z0RcechL+ksNVMu_Lgf5P3RGjHBGAhRRj2fi==tCda=WQ@mail.gmail.com>
```
Sorry about that Tobias, I only try to move things along. Is there any
specific way to invoke the testsuite other that make check-fortran?

Jerry --
```

On Tue, Mar 17, 2026, 8:11 AM Tobias Burnus <[email protected]> wrote:

> Hi all,
>
> first, I have to admit that I didn't see a review of this patch – but
> also no indication that Jerry planned to push that patch by Christopher,
> which I find confusing.
>
> In any case, there is an issue with where the testcase is placed:
>
> * OpenMP compile tests should be placed to
> gcc/testsuite/gfortran.dg/gomp/ - which automatically also adds the
> -fopenmp flag
>
> * OpenMP link and runtime tests – and all files that include omp_lib.h
> or 'use omp_lib' need to go to libgomp/testsuite/libgomp.fortran/ as
> neither the module nor libgomp.so is available for tests in gcc/testsuite/.
>
> * * *
>
> Jerry D wrote:
> >     PR fortran/120286
> >     * gfortran.dg/pr120286.f90: New test.
> > Signed-off-by: Christopher Albert <[email protected]>
>
> Can you move that test to libgomp/testsuite/ – as it is 'dg-do run'; the
> 'dg-additional-options "-fopenmp"' can be then also be removed.
>
> * * *
>
> Regarding patch reviews: I know that I am way behind reviewing patches :-(
>
> Tobias
>
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.