diff mbox

AArch64: Default to -fsched-pressure

Message ID 000101cfd42f$18b59ae0$4a20d0a0$@com
State New
Headers show

Commit Message

Wilco Sept. 19, 2014, 5:28 p.m. UTC 
This patch makes -fsched-pressure the default on AArch64, like on ARM. This improves performance and
reduces codesize due to fewer unnecessary spills.

OK for commit?

ChangeLog:
2014-09-19  Wilco Dijkstra  <wdijkstr@arm.com>

	* gcc/common/config/aarch64/aarch64-common.c:
	(default_options aarch_option_optimization_table):
	Default to -fsched-pressure.

---
 gcc/common/config/aarch64/aarch64-common.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Marcus Shawcroft Sept. 23, 2014, 11:20 a.m. UTC | #1 
On 19 September 2014 18:28, Wilco Dijkstra <wdijkstr@arm.com> wrote:
> This patch makes -fsched-pressure the default on AArch64, like on ARM. This improves performance and
> reduces codesize due to fewer unnecessary spills.
>
> OK for commit?
>
> ChangeLog:
> 2014-09-19  Wilco Dijkstra  <wdijkstr@arm.com>
>
>         * gcc/common/config/aarch64/aarch64-common.c:
>         (default_options aarch_option_optimization_table):
>         Default to -fsched-pressure.

OK, committed as 215503. This patch, like the last has been munged by
the email system. I cut-n-paste the changes by hand.  For future
patches can you either fix your email client or send patches as
attachments please.
Cheers
 /Marcus
diff mbox

Patch

diff --git a/gcc/common/config/aarch64/aarch64-common.c b/gcc/common/config/aarch64/aarch64-common.c
index e44b40a..63f2212 100644
--- a/gcc/common/config/aarch64/aarch64-common.c
+++ b/gcc/common/config/aarch64/aarch64-common.c
@@ -44,6 +44,8 @@  static const struct default_options aarch_option_optimization_table[] =
   {
     /* Enable section anchors by default at -O1 or higher.  */
     { OPT_LEVELS_1_PLUS, OPT_fsection_anchors, NULL, 1 },
+    /* Enable -fsched-pressure by default when optimizing.  */
+    { OPT_LEVELS_1_PLUS, OPT_fsched_pressure, 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 }