From: Martin Jambor Date: Wed, 11 Jan 2017 10:35:31 +0000 (+0100) Subject: [hsa] Fix hsa function cloning test X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=51020892466d37f71ee13137bdadcbb32545c55d;p=gcc.git [hsa] Fix hsa function cloning test 2017-01-11 Martin Jambor * hsa.c (hsa_callable_function_p): Revert addition of DECL_ARTIFICIAL test. * ipa-hsa.c (process_hsa_functions): Only duplicate non-artificial decorated functions. From-SVN: r244308 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index caa26f8ea55..3c8f977e317 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2017-01-11 Martin Jambor + + * hsa.c (hsa_callable_function_p): Revert addition of DECL_ARTIFICIAL + test. + * ipa-hsa.c (process_hsa_functions): Only duplicate non-artificial + decorated functions. + 2017-01-11 Richard Biener * tree-vrp.c (evrp_dom_walker::before_dom_children): Also diff --git a/gcc/hsa.c b/gcc/hsa.c index c0ed9f82bf3..2035ce446a1 100644 --- a/gcc/hsa.c +++ b/gcc/hsa.c @@ -90,10 +90,7 @@ bool hsa_callable_function_p (tree fndecl) { return (lookup_attribute ("omp declare target", DECL_ATTRIBUTES (fndecl)) - && !lookup_attribute ("oacc function", DECL_ATTRIBUTES (fndecl)) - /* At this point, this is enough to identify clones for - parallel, which for HSA would need to be kernels anyway. */ - && !DECL_ARTIFICIAL (fndecl)); + && !lookup_attribute ("oacc function", DECL_ATTRIBUTES (fndecl))); } /* Allocate HSA structures that are are used when dealing with different diff --git a/gcc/ipa-hsa.c b/gcc/ipa-hsa.c index 4391b580566..6a3f660672e 100644 --- a/gcc/ipa-hsa.c +++ b/gcc/ipa-hsa.c @@ -100,7 +100,10 @@ process_hsa_functions (void) clone->name (), s->m_kind == HSA_KERNEL ? "kernel" : "function"); } - else if (hsa_callable_function_p (node->decl)) + else if (hsa_callable_function_p (node->decl) + /* At this point, this is enough to identify clones for + parallel, which for HSA would need to be kernels anyway. */ + && !DECL_ARTIFICIAL (node->decl)) { if (!check_warn_node_versionable (node)) continue;