This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Reduction support for parloop, OMP_ATOMIC Changes
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Razya Ladelsky <RAZYA at il dot ibm dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Zdenek Dvorak <rakdver at kam dot mff dot cuni dot cz>
- Date: Mon, 24 Sep 2007 11:34:46 -0400
- Subject: Re: [PATCH] Reduction support for parloop, OMP_ATOMIC Changes
- References: <OF0B3EE0F0.24B1C41E-ONC225735F.006BACAC-C225735F.006BB041@LocalDomain> <OF8C4F7F4C.28193E91-ONC2257360.0051E8E0-C2257360.00531246@il.ibm.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Mon, Sep 24, 2007 at 05:10:39PM +0200, Razya Ladelsky wrote:
> Now the testsuite zipped file was corrupted...
> This is the final patch and testsuite directory :)
Can you please explain why exactly you want to avoid lowering
OMP_ATOMIC early? This OMP_ATOMIC_LOAD/OMP_ATOMIC_STORE
separation looks terribly fragile to me. What's wrong with just
emitting __sync etc. immediately when autoparallelizing?
You could either create OMP_ATOMIC and gimplify that there,
or you could perhaps tweak the OMP_ATOMIC gimplifiers so that
they are usable from both OMP_ATOMIC gimplification and the autopar
code.
Jakub