This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[committed] Add todo comment for move_sese_region_to_fn
- From: Tom de Vries <Tom_deVries at mentor dot com>
- To: Richard Biener <rguenther at suse dot de>
- Cc: Thomas Schwinge <thomas at codesourcery dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Jakub Jelinek <jakub at redhat dot com>
- Date: Tue, 11 Aug 2015 11:23:31 +0200
- Subject: [committed] Add todo comment for move_sese_region_to_fn
- Authentication-results: sourceware.org; auth=none
- References: <546743BC dot 5070804 at mentor dot com> <54678B3D dot 8020009 at mentor dot com> <54730EE7 dot 40000 at mentor dot com> <5474665A dot 4070101 at mentor dot com> <87iocp1d8o dot fsf at kepler dot schwinge dot homeip dot net> <alpine dot LSU dot 2 dot 11 dot 1504220932150 dot 20496 at zhemvz dot fhfr dot qr> <557076A5 dot 7050207 at mentor dot com> <55C1BA11 dot 70008 at mentor dot com> <alpine dot LSU dot 2 dot 11 dot 1508050928320 dot 19642 at zhemvz dot fhfr dot qr> <55C1CDAF dot 8050402 at mentor dot com> <alpine dot LSU dot 2 dot 11 dot 1508051116450 dot 19642 at zhemvz dot fhfr dot qr> <55C1EA05 dot 5070904 at mentor dot com> <alpine dot LSU dot 2 dot 11 dot 1508051310230 dot 19642 at zhemvz dot fhfr dot qr>
[ was: Re: [committed, gomp4] Fix release_dangling_ssa_names ]
On 05/08/15 13:13, Richard Biener wrote:
I think that the whole dance of actually moving things instead of
> >just copying it isn't worth the extra maintainance (well, if we already
> >have a machinery duplicating a SESE region to another function - I
> >suppose gimple_duplicate_sese_region could be trivially changed to
> >support that).
> >
>
>I'll mention that as todo. For now, I think the fastest way to get a working
>version is to fix move_sese_region_to_fn.
Sure.
This patch adds the todo discussed above in the function header comment
of move_sese_region_to_fn.
Committed as obvious.
Thanks,
- Tom
Add todo comment for move_sese_region_to_fn
2015-08-11 Tom de Vries <tom@codesourcery.com>
* tree-cfg.c (move_sese_region_to_fn): Add todo comment.
---
gcc/tree-cfg.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/gcc/tree-cfg.c b/gcc/tree-cfg.c
index e26454a..588ab69 100644
--- a/gcc/tree-cfg.c
+++ b/gcc/tree-cfg.c
@@ -7011,7 +7011,11 @@ verify_sese (basic_block entry, basic_block exit, vec<basic_block> *bbs_p)
All local variables referenced in the region are assumed to be in
the corresponding BLOCK_VARS and unexpanded variable lists
- associated with DEST_CFUN. */
+ associated with DEST_CFUN.
+
+ TODO: investigate whether we can reuse gimple_duplicate_sese_region to
+ reimplement move_sese_region_to_fn by duplicating the region rather than
+ moving it. */
basic_block
move_sese_region_to_fn (struct function *dest_cfun, basic_block entry_bb,
--
1.9.1