This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [gomp merge] Stage 2: OpenMP for the C++ front end
- From: Andrew Pinski <pinskia at physics dot uc dot edu>
- To: Diego Novillo <dnovillo at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Richard Henderson <rth at redhat dot com>
- Date: Fri, 3 Feb 2006 16:58:40 -0500
- Subject: Re: [gomp merge] Stage 2: OpenMP for the C++ front end
- References: <43E38AD3.1010009@redhat.com>
On Feb 3, 2006, at 11:54 AM, Diego Novillo wrote:
This patch merges all the changes needed to enable OpenMP for the C++
front end.
Some comments
Since the changes to front end are more self-contained, I just added
all
the ChangeLog entries from the branch. I could probably create new
ChangeLog entries if that proves too noisy.
You should as most other merges used the changelog entries except
for ones which are small enough to do.
Bootstrapped and tested on x86, x86-64, ppc64 and ia64.
OK for mainline?
For future reference the changelog should not be in the gziped patch
file (and yes this is documented).
Index: gcc/tree-cfg.c
===================================================================
--- gcc/tree-cfg.c (revision 110544)
+++ gcc/tree-cfg.c (working copy)
@@ -4735,6 +4735,8 @@ move_block_to_fn (struct function *dest_
if (uid >= dest_cfun->last_label_uid)
dest_cfun->last_label_uid = uid + 1;
}
+
+ remove_stmt_from_eh_region (stmt);
}
}
Can't this part be submitted separately? It seems like better to.
Also it seems like the cp/decl.c part could be submitted as a
speedup separately before submitting the rest.
Thanks,
Andrew Pinski