From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-bugzilla@gcc.gnu.org>
Received: by sourceware.org (Postfix, from userid 48)
	id EEE523857346; Fri, 28 Oct 2022 08:25:51 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org EEE523857346
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
	s=default; t=1666945551;
	bh=bT7uRb5zLr/N5fZMqUksnxWIuWjntDDPcZVOVtcFXlc=;
	h=From:To:Subject:Date:In-Reply-To:References:From;
	b=v6S8/8mi+YgtzKNhft/KrEcgA036VXhEtcJ9+24o/IYS1wo+Ok1pStyxd8IENU927
	 46L/AWIWRpOjXLCO4JOCnfux+u1CaWLdKJBp/Tc7jf4AUg38kPeYR/hN6HAG3SyWAL
	 4M4Webcn1sIw5M/rHb4fCZk7uuS9PjpLgs17+sT4=
From: "cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug middle-end/90115] OpenACC: predetermined private levels for
 variables declared in blocks
Date: Fri, 28 Oct 2022 08:25:22 +0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: middle-end
X-Bugzilla-Version: 9.0
X-Bugzilla-Keywords: openacc, wrong-code
X-Bugzilla-Severity: normal
X-Bugzilla-Who: cvs-commit at gcc dot gnu.org
X-Bugzilla-Status: NEW
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-90115-4-t0BK4CSu8i@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-90115-4@http.gcc.gnu.org/bugzilla/>
References: <bug-90115-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
List-Id: <gcc-bugs.sourceware.org>

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

--- Comment #18 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Thomas Schwinge <tschwinge@gcc.gnu.or=
g>:

https://gcc.gnu.org/g:11e811d8e2f63667f60f73731bb934273f5882b8

commit r13-3541-g11e811d8e2f63667f60f73731bb934273f5882b8
Author: Julian Brown <julian@codesourcery.com>
Date:   Wed Oct 12 20:44:57 2022 +0000

    OpenACC: Don't gang-privatize artificial variables [PR90115]

    This patch prevents compiler-generated artificial variables from being
    treated as privatization candidates for OpenACC.

    The rationale is that e.g. "gang-private" variables actually must be
    shared by each worker and vector spawned within a particular gang, but
    that sharing is not necessary for any compiler-generated variable (at
    least at present, but no such need is anticipated either).  Variables on
    the stack (and machine registers) are already private per-"thread"
    (gang, worker and/or vector), and that's fine for artificial variables.

    We're restricting this to blocks, as we still need to understand what it
    means for a 'DECL_ARTIFICIAL' to appear in a 'private' clause.

    Several tests need their scan output patterns adjusted to compensate.

    2022-10-14  Julian Brown  <julian@codesourcery.com>

            PR middle-end/90115
    gcc/
            * omp-low.cc (oacc_privatization_candidate_p): Artificial vars =
are
not
            privatization candidates.

    libgomp/
            * testsuite/libgomp.oacc-fortran/declare-1.f90: Adjust scan out=
put.
            * testsuite/libgomp.oacc-fortran/host_data-5.F90: Likewise.
            * testsuite/libgomp.oacc-fortran/if-1.f90: Likewise.
            * testsuite/libgomp.oacc-fortran/print-1.f90: Likewise.
            * testsuite/libgomp.oacc-fortran/privatized-ref-2.f90: Likewise.

    Co-authored-by: Thomas Schwinge <thomas@codesourcery.com>=