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.greenha...@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.
