This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [tree-ssa]: Get PRE bootstrapping again
- From: Diego Novillo <dnovillo at redhat dot com>
- To: Daniel Berlin <dberlin at dberlin dot org>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: 22 Jul 2003 23:36:43 -0400
- Subject: Re: [tree-ssa]: Get PRE bootstrapping again
- Organization: Red Hat Canada
- References: <Pine.LNX.4.56.0307222308240.13776@dberlin.org>
On Tue, 2003-07-22 at 23:09, Daniel Berlin wrote:
> With this patch, PRE bootstraps with all languages enabled again.
>
Cool. I've installed the same patch we used to have when CCP wasn't
enabled by default. I plan to leave the environment variable for a few
days and if everything goes well we should probably enable PRE by
default.
All the overnight testers will be setting TREE_SSA_DO_PRE to include PRE
in the default optimization path.
Diego.
* opts.c (decode_options): Add temporary test for environment
variable TREE_SSA_DO_PRE.
Do not disable dominator optimizations when PRE is enabled.
Index: opts.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/opts.c,v
retrieving revision 1.31.2.1
diff -d -u -p -r1.31.2.1 opts.c
--- opts.c 21 Jul 2003 13:50:56 -0000 1.31.2.1
+++ opts.c 23 Jul 2003 03:14:53 -0000
@@ -492,6 +492,9 @@ decode_options (unsigned int argc, const
flag_tree_dce = 1;
flag_tree_copyprop = 1;
flag_tree_dom = 1;
+ /* FIXME: Temporary hack to facilitate testing PRE. */
+ if (getenv ("TREE_SSA_DO_PRE"))
+ flag_tree_pre = 1;
}
if (optimize >= 2)
@@ -516,11 +519,6 @@ decode_options (unsigned int argc, const
flag_delete_null_pointer_checks = 1;
flag_reorder_blocks = 1;
flag_reorder_functions = 1;
- /* FIXME: Apparently, dominator optimizations get in the way of
- SSA-PRE. For now, if -ftree-pre is given, disable dominator
- optimizations at -O2 and higher. */
- if (flag_tree_pre)
- flag_tree_dom = 0;
}
if (optimize >= 3)