On Thu, Sep 26, 2013 at 9:35 PM, Aldy Hernandez <al...@redhat.com> wrote:
>> + /* 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 initial thoughts
> were that whether or not a function is a simd clone can be accessed through
> the cgraph bits ("node->simdclone != NULL" for the clone, and
> "node->has_simd_clones" for the parent). No sense keeping the attribute.
> But I can leave it if you think it's better.
Why have it in the first place if it's marked in the cgraph?
Richard.
> Aldy
