diff options
| author | Jan Hubicka <jh@suse.cz> | 2010-06-30 18:51:36 +0200 |
|---|---|---|
| committer | Jan Hubicka <hubicka@gcc.gnu.org> | 2010-06-30 16:51:36 +0000 |
| commit | 8a9c1ae6bfcdfa9a589c6f8f98b6a9b8f495d9e5 (patch) | |
| tree | f361e2a80e7a7827dd9e5ce2c1670aec9d8c2d38 /gcc | |
| parent | 8a1561bcd07c8e630f1e7492c364a664dd6f3d69 (diff) | |
| download | gcc-8a9c1ae6bfcdfa9a589c6f8f98b6a9b8f495d9e5.zip gcc-8a9c1ae6bfcdfa9a589c6f8f98b6a9b8f495d9e5.tar.gz gcc-8a9c1ae6bfcdfa9a589c6f8f98b6a9b8f495d9e5.tar.bz2 | |
PR middle-end/PR44706
PR middle-end/PR44706
* ipa-split (split_function): Refine conditions when to use DECL_RESULT
to return the value.
* gcc.dg/tree-ssa/ipa-split-4.c: New testcase.
From-SVN: r161615
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/ipa-split.c | 7 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 4 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.dg/tree-ssa/ipa-split-4.c | 27 |
4 files changed, 44 insertions, 0 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index ff37f0c..b19c289 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2010-06-30 Jan Hubicka <jh@suse.cz> + + PR middle-end/PR44706 + * ipa-split (split_function): Refine conditions when to use DECL_RESULT + to return the value. + 2010-06-30 Michael Matz <matz@suse.de> PR bootstrap/44699 diff --git a/gcc/ipa-split.c b/gcc/ipa-split.c index ae784b6..289a7b3 100644 --- a/gcc/ipa-split.c +++ b/gcc/ipa-split.c @@ -931,6 +931,13 @@ split_function (struct split_point *split_point) if (!VOID_TYPE_P (TREE_TYPE (TREE_TYPE (current_function_decl)))) { retval = DECL_RESULT (current_function_decl); + + /* We use temporary register to hold value when aggregate_value_p + is false. Similarly for DECL_BY_REFERENCE we must avoid extra + copy. */ + if (!aggregate_value_p (retval, TREE_TYPE (current_function_decl)) + && !DECL_BY_REFERENCE (retval)) + retval = create_tmp_reg (TREE_TYPE (retval), NULL); if (is_gimple_reg (retval)) retval = make_ssa_name (retval, call); gimple_call_set_lhs (call, retval); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 136c37b5..d9e9a40 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2010-06-30 Jan Hubicka <jh@suse.cz> + + * gcc.dg/tree-ssa/ipa-split-4.c: New testcase. + 2010-06-30 Michael Matz <matz@suse.de> PR bootstrap/44699 diff --git a/gcc/testsuite/gcc.dg/tree-ssa/ipa-split-4.c b/gcc/testsuite/gcc.dg/tree-ssa/ipa-split-4.c new file mode 100644 index 0000000..c2c3960 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/ipa-split-4.c @@ -0,0 +1,27 @@ +/* { dg-do compile } */ +/* { dg-options "-O3 -fdump-tree-fnsplit" } */ +int make_me_big (void); + +int +split_me (int a) +{ + if (__builtin_expect(a<10, 1)) + { + abort (); + } + else + { + make_me_big (); + make_me_big (); + make_me_big (); + make_me_big (); + return a+1; + } +} + +main() +{ + return split_me (0)+split_me(1)+split_me(2); +} +/* { dg-final { scan-tree-dump-times "Splitting function" 1 "fnsplit"} } */ +/* { dg-final { cleanup-tree-dump "fnsplit" } } */ |