diff options
| author | Richard Sandiford <richard.sandiford@arm.com> | 2016-05-26 15:49:22 +0000 |
|---|---|---|
| committer | Richard Sandiford <rsandifo@gcc.gnu.org> | 2016-05-26 15:49:22 +0000 |
| commit | b3d2acb6dce9a25f433c2d5d5bae48e9d2b4149a (patch) | |
| tree | 150da3f17c5686ef4e763fcfa4fc50e15617bf76 | |
| parent | a2d5b8c99393978188790649571aeb2fc1fd6944 (diff) | |
| download | gcc-b3d2acb6dce9a25f433c2d5d5bae48e9d2b4149a.zip gcc-b3d2acb6dce9a25f433c2d5d5bae48e9d2b4149a.tar.gz gcc-b3d2acb6dce9a25f433c2d5d5bae48e9d2b4149a.tar.bz2 | |
Fix ivopts estimates for internal functions
tree-ssa-loop-ivopts.c:loop_body_includes_call was treating internal
calls such as IFN_SQRT as clobbering all caller-saved registers, which
I don't think is appropriate for any current internal function.
Tested on aarch64-linux-gnu and x86_64-linux-gnu.
gcc/
* tree-ssa-loop-ivopts.c (loop_body_includes_call): Don't assume
that internal functions will clobber all caller-saved registers.
From-SVN: r236780
| -rw-r--r-- | gcc/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/tree-ssa-loop-ivopts.c | 1 |
2 files changed, 6 insertions, 0 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 8fe8a26..d31de13 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2016-05-26 Richard Sandiford <richard.sandiford@arm.com> + + * tree-ssa-loop-ivopts.c (loop_body_includes_call): Don't assume + that internal functions will clobber all caller-saved registers. + 2016-05-26 Wilco Dijkstra <wdijkstr@arm.com> * config/aarch64/aarch64.c (aarch64_case_values_threshold): diff --git a/gcc/tree-ssa-loop-ivopts.c b/gcc/tree-ssa-loop-ivopts.c index 9ce6b64..23c9886 100644 --- a/gcc/tree-ssa-loop-ivopts.c +++ b/gcc/tree-ssa-loop-ivopts.c @@ -7643,6 +7643,7 @@ loop_body_includes_call (basic_block *body, unsigned num_nodes) { gimple *stmt = gsi_stmt (gsi); if (is_gimple_call (stmt) + && !gimple_call_internal_p (stmt) && !is_inexpensive_builtin (gimple_call_fndecl (stmt))) return true; } |