This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [tuples] openmp: omp_parallel
- From: "Rafael Espindola" <espindola at google dot com>
- To: "Aldy Hernandez" <aldyh at redhat dot com>
- Cc: dnovillo at google dot com, gcc-patches at gcc dot gnu dot org
- Date: Wed, 30 Jan 2008 12:50:48 +0000
- Subject: Re: [tuples] openmp: omp_parallel
- References: <20080125140934.GA15873@redhat.com>
On 25/01/2008, Aldy Hernandez <aldyh@redhat.com> wrote:
> Hi folks.
Hello,
> I started tuplefying the openmp support in the compiler. The plan is to
> convert the openmp constructs to tuples in the gimplifier, and then
> change omp-low to work with tuples instead of trees.
>
> This patch adds support for OMP_PARALLEL to get the ball rolling. I've
> also cleaned up a few things, like unecessary variables and redundant
> return values.
I think this broke the build. The attached partial revert makes it build again.
> Committed to branch.
> Aldy
Cheers,
--
Rafael Avila de Espindola
Google Ireland Ltd.
Gordon House
Barrow Street
Dublin 4
Ireland
Registered in Dublin, Ireland
Registration Number: 368047
diff --git a/gcc/gimplify.c b/gcc/gimplify.c
index 7e57984..48a4827 100644
--- a/gcc/gimplify.c
+++ b/gcc/gimplify.c
@@ -5114,10 +5114,14 @@ gimplify_scan_omp_clauses (tree *list_p, gimple_seq pre_p, bool in_parallel,
push_gimplify_context ();
gimplify_stmt (&OMP_CLAUSE_REDUCTION_INIT (c), pre_p);
/* FIXME tuples. */
+#if 0
pop_gimplify_context (OMP_CLAUSE_REDUCTION_INIT (c));
push_gimplify_context ();
gimplify_stmt (&OMP_CLAUSE_REDUCTION_MERGE (c), pre_p);
pop_gimplify_context (OMP_CLAUSE_REDUCTION_MERGE (c));
+#else
+ gimple_unreachable ();
+#endif
gimplify_omp_ctxp = outer_ctx;
}
if (notice_outer)