diff mbox

Use combined_fn in tree-vect-patterns.c

Message ID 87611eq8ge.fsf@e105548-lin.cambridge.arm.com
State New
Headers show

Commit Message

Richard Sandiford Nov. 7, 2015, 12:50 p.m. UTC 
Another patch to extend uses of built_in_function to combined_fn,
this time in tree-vect-patterns.c.  The old code didn't handle the
long double pow variants, but I think that's because noone had a target
that would benefit rather than because the code would mishandle them.

Tested on x86_64-linux-gnu, aarch64-linux-gnu and arm-linux-gnueabi.
OK to install?

Thanks,
Richard


gcc/
	* tree-vect-patterns.c: Include case-cfn-macros.h.
	(vect_recog_pow_pattern): Use combined_fn instead of built-in codes.

Comments

Jeff Law Nov. 9, 2015, 10:40 p.m. UTC | #1 
On 11/07/2015 05:50 AM, Richard Sandiford wrote:
> Another patch to extend uses of built_in_function to combined_fn,
> this time in tree-vect-patterns.c.  The old code didn't handle the
> long double pow variants, but I think that's because noone had a target
> that would benefit rather than because the code would mishandle them.
>
> Tested on x86_64-linux-gnu, aarch64-linux-gnu and arm-linux-gnueabi.
> OK to install?
>
> Thanks,
> Richard
>
>
> gcc/
> 	* tree-vect-patterns.c: Include case-cfn-macros.h.
> 	(vect_recog_pow_pattern): Use combined_fn instead of built-in codes.
OK.
jeff
diff mbox

Patch

diff --git a/gcc/tree-vect-patterns.c b/gcc/tree-vect-patterns.c
index d003d33..bab9a4f 100644
--- a/gcc/tree-vect-patterns.c
+++ b/gcc/tree-vect-patterns.c
@@ -39,6 +39,7 @@  along with GCC; see the file COPYING3.  If not see
 #include "tree-vectorizer.h"
 #include "dumpfile.h"
 #include "builtins.h"
+#include "case-cfn-macros.h"
 
 /* Pattern recognition functions  */
 static gimple *vect_recog_widen_sum_pattern (vec<gimple *> *, tree *,
@@ -1007,23 +1008,17 @@  vect_recog_pow_pattern (vec<gimple *> *stmts, tree *type_in,
 			tree *type_out)
 {
   gimple *last_stmt = (*stmts)[0];
-  tree fn, base, exp = NULL;
+  tree base, exp = NULL;
   gimple *stmt;
   tree var;
 
   if (!is_gimple_call (last_stmt) || gimple_call_lhs (last_stmt) == NULL)
     return NULL;
 
-  fn = gimple_call_fndecl (last_stmt);
-  if (fn == NULL_TREE || DECL_BUILT_IN_CLASS (fn) != BUILT_IN_NORMAL)
-   return NULL;
-
-  switch (DECL_FUNCTION_CODE (fn))
+  switch (gimple_call_combined_fn (last_stmt))
     {
-    case BUILT_IN_POWIF:
-    case BUILT_IN_POWI:
-    case BUILT_IN_POWF:
-    case BUILT_IN_POW:
+    CASE_CFN_POW:
+    CASE_CFN_POWI:
       base = gimple_call_arg (last_stmt, 0);
       exp = gimple_call_arg (last_stmt, 1);
       if (TREE_CODE (exp) != REAL_CST