This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [ARM Churn] Rename LOGICAL_OP_NON_SC to LOGICAL_OP_NON_SHORT_CIRCUIT
- From: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- To: James Greenhalgh <James dot Greenhalgh at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>
- Date: Tue, 16 Jun 2015 10:13:49 +0100
- Subject: Re: [ARM Churn] Rename LOGICAL_OP_NON_SC to LOGICAL_OP_NON_SHORT_CIRCUIT
- Authentication-results: sourceware.org; auth=none
- References: <1434445630-1697-1-git-send-email-james dot greenhalgh at arm dot com>
On 16/06/15 10:07, James Greenhalgh wrote:
Hi,
SC is an acronym used when discussing memory models to mean
sequentially consistent, seeing it in the context of tuning
structures threw me. This patch just expands the acronym, to
save others from this confusion.
For me SC is more associated with Standards Committee or Steering Committee ;)
Built an ARM compiler to ensure I hadn't damaged anything.
Happy to drop the patch as too pedantic, but I'll leave that to
the ARM reviewers.
OK?
Yes, it does make it easier to parse when quickly reading the code.
Ok for trunk.
Thanks,
Kyrill
Thanks,
James
---
gcc/ChangeLog:
2015-06-16 James Greenhalgh <james.greenhalgh@arm.com>
* config/arm/arm-protos.h (struct tune_params): Rename
log_op_non_sc to log_op_non_short_circuit, and rename enum
values to expand SC to SHORT_CIRCUIT.
* config/arm/arm.c (arm_slowmul_tune): Expand LOG_OP_NON_SC
to LOG_OP_NON_SHORT_CIRCUIT.
(arm_fastmul_tune):Likewise
(arm_strongarm_tune): Likewise.
(arm_xscale_tune): Likewise.
(arm_9e_tune): Likewise.
(arm_marvell_pj4_tune): Likewise.
(arm_v6t2_tune): Likewise.
(arm_cortex_tune): Likewise.
(arm_cortex_a8_tune): Likewise.
(arm_cortex_a7_tune): Likewise.
(arm_cortex_a15_tune): Likewise.
(arm_cortex_a53_tune): Likewise.
(arm_cortex_a57_tune): Likewise.
(arm_xgene1_tune): Likewise.
(arm_cortex_a5_tune): Likewise.
(arm_cortex_a9_tune): Likewise.
(arm_cortex_a12_tune): Likewise.
(arm_v7m_tune): Likewise.
(arm_cortex_m7_tune): Likewise.
(arm_v6m_tune): Likewise.
(arm_fa726te_tune): Likewise.