https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103416

Tobias Burnus <burnus at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[12 Regression][OpenMP]     |[OpenMP] Double mapping via
                   |Bogus firstprivate(n)       |firstprivate(n) map(to:n
                   |map(to:n [len:              |[len: 4][implicit])
                   |4][implicit])               |
           Keywords|wrong-code                  |missed-optimization

--- Comment #6 from Tobias Burnus <burnus at gcc dot gnu.org> ---
I changed the bug subject as the "12 Regression" issue has been fixed (cf.
below,
committed with reference to PR90030 not to this PR) – but the code-improvement
part still remains to be done.

 * * *

(In reply to Chung-Lin Tang from comment #1)
> Can you see if adding this patch:
> https://gcc.gnu.org/pipermail/gcc-patches/2021-November/583279.html
> fixes this problem? If so, then it should be another occurrence of PR90030

This patch is now committed:

commit r12-5706-g1ac7a8c9e4798d352eb8c64905dd38086af4e1cd
Author: Chung-Lin Tang <clt...@codesourcery.com>
Date:   Thu Dec 2 18:24:03 2021 +0800

    fortran: OpenMP/OpenACC array mapping alignment fix (PR90030)

* * *

TODO:  LOOP VAR double mapping – cf. PR80330, i.e.
  map(to:D.4217 [len: 4][implicit]) map(tofrom:n [len: 4][implicit])
where the loop variable is mapped twice.

See Comment 4 for what remains to be done and a draft patch.

Reply via email to