This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[PATCH][ARM] Enable code hoisting with -Os (PR80155)
- From: Wilco Dijkstra <Wilco dot Dijkstra at arm dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>, Kyrylo Tkachov <Kyrylo dot Tkachov at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>
- Cc: nd <nd at arm dot com>
- Date: Wed, 11 Sep 2019 15:49:46 +0000
- Subject: [PATCH][ARM] Enable code hoisting with -Os (PR80155)
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=zDf4K3zYNhBU+PpmbqGZRuCaTAvlTtActHc8NfLGD3o=; b=Y1e6p9mJIvVdrufvRSWQTZg1Tw2Htb0yO7wYoEioxO30CiR/+Gm1DB73vqUL3857xoC9sXxTwCYvV2fveVXvDNUtNZzJkkFmYZftp1dB/i9hVtcultFuAW0azHdMBWQxjtliGXO0pUJVHYDFjEezTV6/sGbWsUZZMkwl6GW0UMkDv803JGQPGgL6xf15uCN2jk53RsYctxXnbN96FJ4hBEHXSxfYupYuZbIcSpKxorye8dMYw+pQQ0/EeKYx1aarG/DuuxhsH+G+BJdOcKyKc8vB9I1YS6ZZ0rV3gcBPabmaN6TCmBL4FT/TFadhTDFHkChdnYXlSnw//7Evm9/Zqw==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=mVNelu256QMpCgJb5uTLbyFOxkXCeP3xAl8UZ7I9EKY4knBKxg7+to8GlSPApLOr69BLJdReOen4vZxusHs9uy5RT0wRhZA6Z59Luo4UY+Km3XgGDL9iAHY+L6mIqRbmSW1bL77DTFGJCv3IaMZfA390BnRZE1umUyJgnhtZ9vyXPoDMlXzLqwd+QHVbi1JeJ/8LciBCEAdqz2gOROVDGuAf5Se7TRnHvXc3Ilma21AaJEOwogID2b2Ulx1BB+6tK8T5EoolPg1y2PM68lO/VMVSH7f5VXioPuH4WG4HdNtJ8QYt0NQM5dEaPbFsBaJ5lIYxiB/owkDt8xiudB855w==
- Original-authentication-results: spf=none (sender IP is ) smtp.mailfrom=Wilco dot Dijkstra at arm dot com;
While code hoisting generally improves codesize, it can affect performance
negatively. Benchmarking shows it doesn't help SPEC and negatively affects
embedded benchmarks, so only enable code hoisting with -Os on Arm.
Bootstrap OK, OK for commit?
ChangeLog:
2019-09-11 Wilco Dijkstra <wdijkstr@arm.com>
PR tree-optimization/80155
* common/config/arm/arm-common.c (arm_option_optimization_table):
Enable -fcode-hoisting with -Os.
--
diff --git a/gcc/common/config/arm/arm-common.c b/gcc/common/config/arm/arm-common.c
index 41a920f6dc96833e778faa8dbcc19beac483734c..b0d5fb300bf01acc1fb6f4631635f8a1bfe6441c 100644
--- a/gcc/common/config/arm/arm-common.c
+++ b/gcc/common/config/arm/arm-common.c
@@ -39,6 +39,8 @@ static const struct default_options arm_option_optimization_table[] =
/* Enable section anchors by default at -O1 or higher. */
{ OPT_LEVELS_1_PLUS, OPT_fsection_anchors, NULL, 1 },
{ OPT_LEVELS_1_PLUS, OPT_fsched_pressure, NULL, 1 },
+ /* Enable code hoisting only with -Os. */
+ { OPT_LEVELS_SIZE, OPT_fcode_hoisting, NULL, 1 },
{ OPT_LEVELS_NONE, 0, NULL, 0 }
};