On 09/27/13 03:18, Richard Biener wrote:
On Thu, Sep 26, 2013 at 9:35 PM, Aldy Hernandez <[email protected]> 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?
It would be placed there by the front-end when parsing Cilk Plus simd-enabled functions. It's only in the the omp stage that we transfer that information to the cgraph bits.
