On Thu, Dec 22, 2011 at 1:58 PM, Joseph S. Myers
<jos...@codesourcery.com> wrote:

> Is there a reason you are doing it this way rather than by adding an entry
> to alpha_option_optimization_table (in
> common/config/alpha/alpha-common.c), which would generally be the
> preferred way of enabling an option at certain optimization levels on a
> target-specific basis?

This patch updates flag setting approach according to Joseph's comments.

2011-12-22  Uros Bizjak  <ubiz...@gmail.com>

        PR target/27468
        * common/config/alpha/alpha-common.c (alpha_option_optimization_table):
        Enable flag_ree at -O2 or higher.

Patch was bootstrapped and regression tested on alphaev68-pc-linux-gnu.

OK for mainline?

Uros.

Index: common/config/alpha/alpha-common.c
===================================================================
--- common/config/alpha/alpha-common.c  (revision 182617)
+++ common/config/alpha/alpha-common.c  (working copy)
@@ -33,6 +33,8 @@
 static const struct default_options alpha_option_optimization_table[] =
   {
     { OPT_LEVELS_1_PLUS, OPT_fomit_frame_pointer, NULL, 1 },
+    /* Enable redundant extension instructions removal at -O2 and higher.  */
+    { OPT_LEVELS_2_PLUS, OPT_free, NULL, 1 },
     { OPT_LEVELS_NONE, 0, NULL, 0 }
   };

Reply via email to