This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: OMP4/cilkplus: simd clone function mangling

From: Aldy Hernandez <aldyh at redhat dot com>
To: Jakub Jelinek <jakub at redhat dot com>, Richard Biener <richard dot guenther at gmail dot com>, Jan Hubicka <jh at suse dot cz>, Richard Henderson <rth at redhat dot com>
Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, "Iyer, Balaji V" <balaji dot v dot iyer at intel dot com>
Date: Thu, 26 Sep 2013 14:35:56 -0500
Subject: Re: OMP4/cilkplus: simd clone function mangling
Authentication-results: sourceware.org; auth=none
References: <52448B95 dot 9020901 at redhat dot com>

+  /* To distinguish from an OpenMP simd clone, Cilk Plus functions to
+     be cloned have a distinctive artificial label in addition to "omp
+     declare simd".  */
+  bool cilk_clone = flag_enable_cilkplus
+    && lookup_attribute ("cilk plus elemental",
+			 DECL_ATTRIBUTES (new_node->symbol.decl));
+  if (cilk_clone)
+    remove_attribute ("cilk plus elemental",
+		      DECL_ATTRIBUTES (new_node->symbol.decl));

Oh yeah, rth had asked me why I remove the attribute. My initialthoughts were that whether or not a function is a simd clone can beaccessed through the cgraph bits ("node->simdclone != NULL" for theclone, and "node->has_simd_clones" for the parent). No sense keepingthe attribute. But I can leave it if you think it's better.


Aldy

Follow-Ups:
- Re: OMP4/cilkplus: simd clone function mangling
  - From: Richard Biener

References:
- OMP4/cilkplus: simd clone function mangling
  - From: Aldy Hernandez

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]