* [PATCH 1/4] Clean up of new format of -falign-FOO.
2018-07-16 8:53 [PATCH 0/4] Alignment option enhancements marxin
2018-07-16 8:53 ` [PATCH 3/4] Define MAX_CODE_ALIGN globally marxin
2018-07-16 8:53 ` [PATCH 2/4] Fix coding style of ASM_OUTPUT_ALIGN marxin
@ 2018-07-16 8:53 ` marxin
2018-07-16 19:52 ` Pat Haugen
` (2 more replies)
2018-07-16 8:53 ` [PATCH 4/4] Do not enable OPT_falign_* for -Os marxin
2018-07-16 14:10 ` [PATCH 0/4] Alignment option enhancements Richard Biener
4 siblings, 3 replies; 15+ messages in thread
From: marxin @ 2018-07-16 8:53 UTC (permalink / raw)
To: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 6184 bytes --]
gcc/ChangeLog:
2018-07-11 Martin Liska <mliska@suse.cz>
* align.h: New file.
* config/alpha/alpha.c (alpha_align_insns_1): Use align_functions directly.
* config/i386/i386.c (ix86_avoid_jump_mispredicts): Use new return type
align_flags of label_to_alignment.
* config/m32r/m32r.h (LOOP_ALIGN): Wrap returned values into align_flags
class.
* config/m68k/m68k.c: Do not use removed align_labels_value and
align_loops_value.
* config/nds32/nds32.h (JUMP_ALIGN): Wrap result into align_flags class.
(LOOP_ALIGN): Likewise.
(LABEL_ALIGN): Likewise.
* config/powerpcspe/powerpcspe.c (TARGET_ASM_LOOP_ALIGN_MAX_SKIP):
Remove not used macro.
(rs6000_loop_align): Change return type to align_flags.
(rs6000_loop_align_max_skip): Remove.
* config/rs6000/rs6000-protos.h (rs6000_loop_align):
Change return type to align_flags.
* config/rs6000/rs6000.c (TARGET_ASM_LOOP_ALIGN_MAX_SKIP):
Remove not used macro.
(rs6000_loop_align): Change return type to align_flags.
(rs6000_loop_align_max_skip): Remove.
* config/rx/rx.h (JUMP_ALIGN): Wrap integer values
* config/rx/rx-protos.h (rx_align_for_label): Make it
static function.
* config/rx/rx.c (rx_align_for_label): Change return type
to align_flags.
(rx_max_skip_for_label): Remove TARGET_ASM_*_ALIGN_MAX_SKIP
macro definitions.
into align_flags class.
(LABEL_ALIGN): Likewise.
(LOOP_ALIGN): Likewise.
* config/s390/s390.c (s390_label_align): Use align_flags
class member.
(s390_asm_output_function_label): Likewise.
* config/sh/sh.c (sh_override_options_after_change):
Use align_flags class directly without macros.
(find_barrier): Likewise.
(barrier_align): Likewise.
(sh_loop_align): Likewise.
* config/spu/spu.c (spu_option_override):
Use align_flags_tuple::get_value instead of removed macros.
(spu_sched_init): Likewise.
* config/spu/spu.h (GTY): Likewise.
* config/visium/visium.c (visium_option_override):
Set "8" as default secondary alignment.
* config/visium/visium.h (SUBALIGN_LOG): Define to 3
in order to guarantee secondary alignment of 8.
* coretypes.h: Include align.h header file.
* doc/tm.texi: Remove TARGET_ASM_JUMP_ALIGN_MAX_SKIP,
TARGET_ASM_LOOP_ALIGN_MAX_SKIP, TARGET_ASM_LABEL_ALIGN_MAX_SKIP
and TARGET_ASM_LABEL_ALIGN_AFTER_BARRIER_MAX_SKIP macros.
* doc/tm.texi.in: Likewise.
* final.c (struct label_alignment): Remove not used structure.
(LABEL_ALIGN): Change type to align_flags.
(LOOP_ALIGN): Likewise.
(JUMP_ALIGN): Likewise.
(default_loop_align_max_skip): Remove.
(default_label_align_max_skip): Likewise.
(default_jump_align_max_skip): Likewise.
(default_label_align_after_barrier_max_skip):
(LABEL_TO_ALIGNMENT): Change to access label_align vector.
(LABEL_TO_MAX_SKIP): Remove.
(label_to_alignment): Return align_flags type instead of integer.
(label_to_max_skip): Remove.
(align_fuzz): Use align_flags type.
(compute_alignments): Use align_flags type and use align_flags::max
to combine multiple alignments.
(grow_label_align): Grow vec instead of C array.
(update_alignments): Assign just LABEL_TO_ALIGNMENT.
(shorten_branches): Use align_flags type and use align_flags::max
to combine multiple alignments.
(final_scan_insn_1): Remove usage of secondary alignment that comes
from label alignment, but instead use proper secondary alignment
which is computed in grow_label_align.
* flags.h (struct align_flags_tuple): Move to align.h.
(struct align_flags): Likewise.
(state_align_loops): Rename to align_loops.
(state_align_jumps): Rename to align_jumps.
(state_align_labels): Rename to align_labels.
(state_align_functions): Rename to align_functions.
(align_loops_log): Remove.
(align_jumps_log): Remove.
(align_labels_log): Remove.
(align_functions_log): Remove.
(align_loops_max_skip): Remove.
(align_jumps_max_skip): Remove.
(align_labels_max_skip): Remove.
(align_functions_max_skip): Remove.
(align_loops_value): Remove.
(align_jumps_value): Remove.
(align_labels_value): Remove.
(align_functions_value): Remove.
* output.h (label_to_alignment): Change return type to align_flags.
(label_to_max_skip): Remove.
* target.def: Remove loop_align_max_skip, label_align_max_skip,
jump_align_max_skip macros.
* targhooks.h (default_loop_align_max_skip): Remove.
(default_label_align_max_skip): Likewise.
(default_jump_align_max_skip): Likewise.
(default_label_align_after_barrier_max_skip): Remove.
* toplev.c (read_log_maxskip): Use ::normalize function.
(parse_N_M): Remove not used argument and also call ::normalize.
(parse_alignment_opts): Do not pass unused arguments.
* varasm.c (assemble_start_function): Use directly align_functions
instead of removed macros.
* system.h: Do not poison removed macros.
---
gcc/align.h | 76 ++++++++++++
gcc/config/alpha/alpha.c | 5 +-
gcc/config/i386/i386.c | 5 +-
gcc/config/m32r/m32r.h | 3 +-
gcc/config/m68k/m68k.c | 10 +-
gcc/config/nds32/nds32.h | 9 +-
gcc/config/powerpcspe/powerpcspe.c | 18 +--
gcc/config/rs6000/rs6000-protos.h | 2 +-
gcc/config/rs6000/rs6000.c | 18 +--
gcc/config/rx/rx-protos.h | 2 +-
gcc/config/rx/rx.c | 59 ++++-----
gcc/config/rx/rx.h | 6 +-
gcc/config/s390/s390.c | 9 +-
gcc/config/sh/sh.c | 19 +--
gcc/config/spu/spu.c | 6 +-
gcc/config/spu/spu.h | 2 +-
gcc/config/visium/visium.c | 2 +-
gcc/config/visium/visium.h | 13 +-
gcc/coretypes.h | 1 +
gcc/doc/tm.texi | 24 ----
gcc/doc/tm.texi.in | 8 --
gcc/final.c | 188 ++++++++---------------------
gcc/flags.h | 38 +-----
gcc/output.h | 6 +-
gcc/system.h | 2 -
gcc/target.def | 40 ------
gcc/targhooks.h | 4 -
gcc/toplev.c | 44 ++-----
gcc/varasm.c | 14 +--
29 files changed, 244 insertions(+), 389 deletions(-)
create mode 100644 gcc/align.h
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Clean-up-of-new-format-of-falign-FOO.patch --]
[-- Type: text/x-patch; name="0001-Clean-up-of-new-format-of-falign-FOO.patch", Size: 49109 bytes --]
diff --git a/gcc/align.h b/gcc/align.h
new file mode 100644
index 00000000000..5a5d6c752a3
--- /dev/null
+++ b/gcc/align.h
@@ -0,0 +1,76 @@
+/* Alignment-related classes.
+ Copyright (C) 2018 Free Software Foundation, Inc.
+
+This file is part of GCC.
+
+GCC is free software; you can redistribute it and/or modify it under
+the terms of the GNU General Public License as published by the Free
+Software Foundation; either version 3, or (at your option) any later
+version.
+
+GCC is distributed in the hope that it will be useful, but WITHOUT ANY
+WARRANTY; without even the implied warranty of MERCHANTABILITY or
+FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+for more details.
+
+You should have received a copy of the GNU General Public License
+along with GCC; see the file COPYING3. If not see
+<http://www.gnu.org/licenses/>. */
+
+/* Align flags tuple with alignment in log form and with a maximum skip. */
+
+struct align_flags_tuple
+{
+ /* Values of the -falign-* flags: how much to align labels in code.
+ log is "align to 2^log" (so 0 means no alignment).
+ maxskip is the maximum allowed amount of padding to insert. */
+ int log;
+ int maxskip;
+
+ /* Normalize filled values so that maxskip is not bigger than 1 << log. */
+ void normalize ()
+ {
+ int n = (1 << log);
+ if (maxskip > n)
+ maxskip = n - 1;
+ }
+
+ /* Return original value of an alignment flag. */
+ int get_value ()
+ {
+ return maxskip + 1;
+ }
+};
+
+/* Alignment flags is structure used as value of -align-* options.
+ It's used in target-dependant code. */
+
+struct align_flags
+{
+ /* Default constructor. */
+ align_flags (int log0 = 0, int maxskip0 = 0, int log1 = 0, int maxskip1 = 0)
+ {
+ levels[0] = {log0, maxskip0};
+ levels[1] = {log1, maxskip1};
+ normalize ();
+ }
+
+ /* Normalize both components of align_flags. */
+ void normalize ()
+ {
+ for (unsigned i = 0; i < 2; i++)
+ levels[i].normalize ();
+ }
+
+ /* Get alignment that is common bigger alignment of alignments F0 and F1. */
+ static align_flags max (const align_flags f0, const align_flags f1)
+ {
+ int log0 = MAX (f0.levels[0].log, f1.levels[0].log);
+ int maxskip0 = MAX (f0.levels[0].maxskip, f1.levels[0].maxskip);
+ int log1 = MAX (f0.levels[1].log, f1.levels[1].log);
+ int maxskip1 = MAX (f0.levels[1].maxskip, f1.levels[1].maxskip);
+ return align_flags (log0, maxskip0, log1, maxskip1);
+ }
+
+ align_flags_tuple levels[2];
+};
diff --git a/gcc/config/alpha/alpha.c b/gcc/config/alpha/alpha.c
index 9adfe159381..993d2324464 100644
--- a/gcc/config/alpha/alpha.c
+++ b/gcc/config/alpha/alpha.c
@@ -9269,7 +9269,7 @@ alpha_align_insns_1 (unsigned int max_align,
/* Let shorten branches care for assigning alignments to code labels. */
shorten_branches (get_insns ());
- unsigned int option_alignment = align_functions_max_skip + 1;
+ unsigned int option_alignment = align_functions.levels[0].get_value ();
if (option_alignment < 4)
align = 4;
else if ((unsigned int) option_alignment < max_align)
@@ -9291,7 +9291,8 @@ alpha_align_insns_1 (unsigned int max_align,
/* When we see a label, resync alignment etc. */
if (LABEL_P (i))
{
- unsigned int new_align = 1 << label_to_alignment (i);
+ unsigned int new_align
+ = label_to_alignment (i).levels[0].get_value ();
if (new_align >= align)
{
diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index 9e46b7b136f..9a9f9424084 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -41924,8 +41924,9 @@ ix86_avoid_jump_mispredicts (void)
if (LABEL_P (insn))
{
- int align = label_to_alignment (insn);
- int max_skip = label_to_max_skip (insn);
+ align_flags alignment = label_to_alignment (insn);
+ int align = alignment.levels[0].log;
+ int max_skip = alignment.levels[0].maxskip;
if (max_skip > 15)
max_skip = 15;
diff --git a/gcc/config/m32r/m32r.h b/gcc/config/m32r/m32r.h
index 426398d2957..4eb15a7ba60 100644
--- a/gcc/config/m32r/m32r.h
+++ b/gcc/config/m32r/m32r.h
@@ -866,7 +866,8 @@ L2: .word STATIC
of a loop. */
/* On the M32R, align loops to 32 byte boundaries (cache line size)
if -malign-loops. */
-#define LOOP_ALIGN(LABEL) (TARGET_ALIGN_LOOPS ? 5 : 0)
+#define LOOP_ALIGN(LABEL) ((TARGET_ALIGN_LOOPS \
+ ? align_flags (5) : align_flags ()))
/* Define this to be the maximum number of insns to move around when moving
a loop test from the top of a loop to the bottom
diff --git a/gcc/config/m68k/m68k.c b/gcc/config/m68k/m68k.c
index cea5c0ecab5..ef8604ebe3d 100644
--- a/gcc/config/m68k/m68k.c
+++ b/gcc/config/m68k/m68k.c
@@ -653,15 +653,17 @@ m68k_option_override (void)
#ifndef ASM_OUTPUT_ALIGN_WITH_NOP
parse_alignment_opts ();
- if (align_labels_value > 2)
+ int label_alignment = align_labels.levels[0].get_value ();
+ if (label_alignment > 2)
{
- warning (0, "-falign-labels=%d is not supported", align_labels_value);
+ warning (0, "-falign-labels=%d is not supported", label_alignment);
str_align_labels = "1";
}
- if (align_loops_value > 2)
+ int loop_alignment = align_loops.levels[0].get_value ();
+ if (loop_alignment > 2)
{
- warning (0, "-falign-loops=%d is not supported", align_loops_value);
+ warning (0, "-falign-loops=%d is not supported", loop_alignment);
str_align_loops = "1";
}
#endif
diff --git a/gcc/config/nds32/nds32.h b/gcc/config/nds32/nds32.h
index 9e36c8f2fac..3aac6a21bb8 100644
--- a/gcc/config/nds32/nds32.h
+++ b/gcc/config/nds32/nds32.h
@@ -1630,13 +1630,16 @@ enum reg_class
#define DWARF2_UNWIND_INFO 1
#define JUMP_ALIGN(x) \
- (align_jumps_log ? align_jumps_log : nds32_target_alignment (x))
+ (align_jumps.levels[0].log \
+ ? align_jumps : align_flags (nds32_target_alignment (x)))
#define LOOP_ALIGN(x) \
- (align_loops_log ? align_loops_log : nds32_target_alignment (x))
+ (align_loops.levels[0].log \
+ ? align_loops : align_flags (nds32_target_alignment (x)))
#define LABEL_ALIGN(x) \
- (align_labels_log ? align_labels_log : nds32_target_alignment (x))
+ (align_labels.levels[0].log \
+ ? align_labels : align_flags (nds32_target_alignment (x)))
#define ASM_OUTPUT_ALIGN(stream, power) \
fprintf (stream, "\t.align\t%d\n", power)
diff --git a/gcc/config/powerpcspe/powerpcspe.c b/gcc/config/powerpcspe/powerpcspe.c
index 80f67de12fc..7dd24e47641 100644
--- a/gcc/config/powerpcspe/powerpcspe.c
+++ b/gcc/config/powerpcspe/powerpcspe.c
@@ -1829,9 +1829,6 @@ static const struct attribute_spec rs6000_attribute_table[] =
#undef TARGET_INVALID_ARG_FOR_UNPROTOTYPED_FN
#define TARGET_INVALID_ARG_FOR_UNPROTOTYPED_FN invalid_arg_for_unprototyped_fn
-#undef TARGET_ASM_LOOP_ALIGN_MAX_SKIP
-#define TARGET_ASM_LOOP_ALIGN_MAX_SKIP rs6000_loop_align_max_skip
-
#undef TARGET_MD_ASM_ADJUST
#define TARGET_MD_ASM_ADJUST rs6000_md_asm_adjust
@@ -5723,7 +5720,7 @@ rs6000_builtin_mask_for_load (void)
}
/* Implement LOOP_ALIGN. */
-int
+align_flags
rs6000_loop_align (rtx label)
{
basic_block bb;
@@ -5731,7 +5728,7 @@ rs6000_loop_align (rtx label)
/* Don't override loop alignment if -falign-loops was specified. */
if (!can_override_loop_align)
- return align_loops_log;
+ return align_loops;
bb = BLOCK_FOR_INSN (label);
ninsns = num_loop_insns(bb->loop_father);
@@ -5744,16 +5741,9 @@ rs6000_loop_align (rtx label)
|| rs6000_cpu == PROCESSOR_POWER7
|| rs6000_cpu == PROCESSOR_POWER8
|| rs6000_cpu == PROCESSOR_POWER9))
- return 5;
+ return align_flags (5);
else
- return align_loops_log;
-}
-
-/* Implement TARGET_LOOP_ALIGN_MAX_SKIP. */
-static int
-rs6000_loop_align_max_skip (rtx_insn *label)
-{
- return (1 << rs6000_loop_align (label)) - 1;
+ return align_loops;
}
/* Return true iff, data reference of TYPE can reach vector alignment (16)
diff --git a/gcc/config/rs6000/rs6000-protos.h b/gcc/config/rs6000/rs6000-protos.h
index d548d801fa4..714b8a8f43a 100644
--- a/gcc/config/rs6000/rs6000-protos.h
+++ b/gcc/config/rs6000/rs6000-protos.h
@@ -156,7 +156,7 @@ extern rtx rs6000_machopic_legitimize_pic_address (rtx, machine_mode,
rtx);
extern rtx rs6000_address_for_fpconvert (rtx);
extern rtx rs6000_allocate_stack_temp (machine_mode, bool, bool);
-extern int rs6000_loop_align (rtx);
+extern align_flags rs6000_loop_align (rtx);
extern void rs6000_split_logical (rtx [], enum rtx_code, bool, bool, bool);
#endif /* RTX_CODE */
diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c
index 197607214ff..5c3fd475cbd 100644
--- a/gcc/config/rs6000/rs6000.c
+++ b/gcc/config/rs6000/rs6000.c
@@ -1817,9 +1817,6 @@ static const struct attribute_spec rs6000_attribute_table[] =
#undef TARGET_INVALID_ARG_FOR_UNPROTOTYPED_FN
#define TARGET_INVALID_ARG_FOR_UNPROTOTYPED_FN invalid_arg_for_unprototyped_fn
-#undef TARGET_ASM_LOOP_ALIGN_MAX_SKIP
-#define TARGET_ASM_LOOP_ALIGN_MAX_SKIP rs6000_loop_align_max_skip
-
#undef TARGET_MD_ASM_ADJUST
#define TARGET_MD_ASM_ADJUST rs6000_md_asm_adjust
@@ -5275,7 +5272,7 @@ rs6000_builtin_mask_for_load (void)
}
/* Implement LOOP_ALIGN. */
-int
+align_flags
rs6000_loop_align (rtx label)
{
basic_block bb;
@@ -5283,7 +5280,7 @@ rs6000_loop_align (rtx label)
/* Don't override loop alignment if -falign-loops was specified. */
if (!can_override_loop_align)
- return align_loops_log;
+ return align_loops;
bb = BLOCK_FOR_INSN (label);
ninsns = num_loop_insns(bb->loop_father);
@@ -5295,16 +5292,9 @@ rs6000_loop_align (rtx label)
|| rs6000_tune == PROCESSOR_POWER6
|| rs6000_tune == PROCESSOR_POWER7
|| rs6000_tune == PROCESSOR_POWER8))
- return 5;
+ return align_flags (5);
else
- return align_loops_log;
-}
-
-/* Implement TARGET_LOOP_ALIGN_MAX_SKIP. */
-static int
-rs6000_loop_align_max_skip (rtx_insn *label)
-{
- return (1 << rs6000_loop_align (label)) - 1;
+ return align_loops;
}
/* Return true iff, data reference of TYPE can reach vector alignment (16)
diff --git a/gcc/config/rx/rx-protos.h b/gcc/config/rx/rx-protos.h
index 0bb885d2da1..7e9817cfaf7 100644
--- a/gcc/config/rx/rx-protos.h
+++ b/gcc/config/rx/rx-protos.h
@@ -50,7 +50,7 @@ private:
#ifdef RTX_CODE
extern int rx_adjust_insn_length (rtx_insn *, int);
-extern int rx_align_for_label (rtx, int);
+extern align_flags rx_align_for_label (rtx_insn *, int);
extern void rx_emit_stack_popm (rtx *, bool);
extern void rx_emit_stack_pushm (rtx *);
extern char * rx_gen_move_template (rtx *, bool);
diff --git a/gcc/config/rx/rx.c b/gcc/config/rx/rx.c
index af97bef301d..c2669edc62c 100644
--- a/gcc/config/rx/rx.c
+++ b/gcc/config/rx/rx.c
@@ -3308,23 +3308,6 @@ rx_match_ccmode (rtx insn, machine_mode cc_mode)
return true;
}
\f
-int
-rx_align_for_label (rtx lab, int uses_threshold)
-{
- /* This is a simple heuristic to guess when an alignment would not be useful
- because the delay due to the inserted NOPs would be greater than the delay
- due to the misaligned branch. If uses_threshold is zero then the alignment
- is always useful. */
- if (LABEL_P (lab) && LABEL_NUSES (lab) < uses_threshold)
- return 0;
-
- if (optimize_size)
- return 0;
- /* These values are log, not bytes. */
- if (rx_cpu_type == RX100 || rx_cpu_type == RX200)
- return 2; /* 4 bytes */
- return 3; /* 8 bytes */
-}
static int
rx_max_skip_for_label (rtx_insn *lab)
@@ -3350,10 +3333,41 @@ rx_max_skip_for_label (rtx_insn *lab)
opsize = get_attr_length (op);
if (opsize >= 0 && opsize < 8)
- return opsize - 1;
+ return MAX (0, opsize - 1);
return 0;
}
+static int
+rx_align_log_for_label (rtx_insn *lab, int uses_threshold)
+{
+ /* This is a simple heuristic to guess when an alignment would not be useful
+ because the delay due to the inserted NOPs would be greater than the delay
+ due to the misaligned branch. If uses_threshold is zero then the alignment
+ is always useful. */
+ if (LABEL_P (lab) && LABEL_NUSES (lab) < uses_threshold)
+ return 0;
+
+ if (optimize_size)
+ return 0;
+
+ /* Return zero if max_skip not a positive number. */
+ int max_skip = rx_max_skip_for_label (lab);
+ if (max_skip <= 0)
+ return 0;
+
+ /* These values are log, not bytes. */
+ if (rx_cpu_type == RX100 || rx_cpu_type == RX200)
+ return 2; /* 4 bytes */
+ return 3; /* 8 bytes */
+}
+
+align_flags
+rx_align_for_label (rtx_insn *lab, int uses_threshold)
+{
+ return align_flags (rx_align_log_for_label (lab, uses_threshold),
+ rx_max_skip_for_label (lab));
+}
+
/* Compute the real length of the extending load-and-op instructions. */
int
@@ -3633,15 +3647,6 @@ rx_modes_tieable_p (machine_mode mode1, machine_mode mode2)
#undef TARGET_CAN_INLINE_P
#define TARGET_CAN_INLINE_P rx_ok_to_inline
-#undef TARGET_ASM_JUMP_ALIGN_MAX_SKIP
-#define TARGET_ASM_JUMP_ALIGN_MAX_SKIP rx_max_skip_for_label
-#undef TARGET_ASM_LOOP_ALIGN_MAX_SKIP
-#define TARGET_ASM_LOOP_ALIGN_MAX_SKIP rx_max_skip_for_label
-#undef TARGET_LABEL_ALIGN_AFTER_BARRIER_MAX_SKIP
-#define TARGET_LABEL_ALIGN_AFTER_BARRIER_MAX_SKIP rx_max_skip_for_label
-#undef TARGET_ASM_LABEL_ALIGN_MAX_SKIP
-#define TARGET_ASM_LABEL_ALIGN_MAX_SKIP rx_max_skip_for_label
-
#undef TARGET_FUNCTION_VALUE
#define TARGET_FUNCTION_VALUE rx_function_value
diff --git a/gcc/config/rx/rx.h b/gcc/config/rx/rx.h
index 2f5a0e94677..c87bb96b2f1 100644
--- a/gcc/config/rx/rx.h
+++ b/gcc/config/rx/rx.h
@@ -417,9 +417,9 @@ typedef unsigned int CUMULATIVE_ARGS;
/* Compute the alignment needed for label X in various situations.
If the user has specified an alignment then honour that, otherwise
use rx_align_for_label. */
-#define JUMP_ALIGN(x) (align_jumps_log > 0 ? align_jumps_log : rx_align_for_label (x, 0))
-#define LABEL_ALIGN(x) (align_labels_log > 0 ? align_labels_log : rx_align_for_label (x, 3))
-#define LOOP_ALIGN(x) (align_loops_log > 0 ? align_loops_log : rx_align_for_label (x, 2))
+#define JUMP_ALIGN(x) (align_jumps.levels[0].log > 0 ? align_jumps : align_flags (rx_align_for_label (x, 0)))
+#define LABEL_ALIGN(x) (align_labels.levels[0].log > 0 ? align_labels : align_flags (rx_align_for_label (x, 3)))
+#define LOOP_ALIGN(x) (align_loops.levels[0].log > 0 ? align_loops : align_flags (rx_align_for_label (x, 2)))
#define LABEL_ALIGN_AFTER_BARRIER(x) rx_align_for_label (x, 0)
#define ASM_OUTPUT_MAX_SKIP_ALIGN(STREAM, LOG, MAX_SKIP) \
diff --git a/gcc/config/s390/s390.c b/gcc/config/s390/s390.c
index 23c3f3db621..056a56e437e 100644
--- a/gcc/config/s390/s390.c
+++ b/gcc/config/s390/s390.c
@@ -1233,7 +1233,7 @@ s390_label_align (rtx_insn *label)
return 0;
old:
- return align_labels_log;
+ return align_labels.levels[0].log;
}
static GTY(()) rtx got_symbol;
@@ -7542,10 +7542,11 @@ s390_asm_output_function_label (FILE *asm_out_file, const char *fname,
NOPs. */
function_alignment = MAX (8, DECL_ALIGN (decl) / BITS_PER_UNIT);
if (! DECL_USER_ALIGN (decl))
- function_alignment = MAX (function_alignment,
- (unsigned int) align_functions_max_skip + 1);
+ function_alignment
+ = MAX (function_alignment,
+ (unsigned int) align_functions.levels[0].get_value ());
fputs ("\t# alignment for hotpatch\n", asm_out_file);
- ASM_OUTPUT_ALIGN (asm_out_file, align_functions_log);
+ ASM_OUTPUT_ALIGN (asm_out_file, align_functions.levels[0].log);
}
if (S390_USE_TARGET_ATTRIBUTE && TARGET_DEBUG_ARG)
diff --git a/gcc/config/sh/sh.c b/gcc/config/sh/sh.c
index a1cad42eb70..06de84d7b5a 100644
--- a/gcc/config/sh/sh.c
+++ b/gcc/config/sh/sh.c
@@ -1015,7 +1015,7 @@ sh_override_options_after_change (void)
parse_alignment_opts ();
if (flag_align_jumps && !str_align_jumps)
str_align_jumps = "2";
- else if (align_jumps_value < 2)
+ else if (align_jumps.levels[0].get_value () < 2)
str_align_jumps = "2";
if (flag_align_functions && !str_align_functions)
@@ -1028,12 +1028,13 @@ sh_override_options_after_change (void)
{
/* Parse values so that we can compare for current value. */
parse_alignment_opts ();
- int min_align = MAX (align_loops_value, align_jumps_value);
+ int min_align = MAX (align_loops.levels[0].get_value (),
+ align_jumps.levels[0].get_value ());
/* Also take possible .long constants / mova tables into account. */
if (min_align < 4)
min_align = 4;
- if (align_functions_value < min_align)
+ if (align_functions.levels[0].get_value () < min_align)
{
char *r = XNEWVEC (char, 16);
sprintf (r, "%d", min_align);
@@ -4986,7 +4987,7 @@ find_barrier (int num_mova, rtx_insn *mova, rtx_insn *from)
&& CODE_LABEL_NUMBER (from) <= max_labelno_before_reorg)
{
if (optimize)
- new_align = 1 << label_to_alignment (from);
+ new_align = 1 << label_to_alignment (from).levels[0].log;
else if (BARRIER_P (prev_nonnote_insn (from)))
new_align = 1 << barrier_align (from);
else
@@ -5118,7 +5119,7 @@ find_barrier (int num_mova, rtx_insn *mova, rtx_insn *from)
&& (prev_nonnote_insn (from)
== XEXP (MOVA_LABELREF (mova), 0))))
num_mova--;
- if (barrier_align (next_real_insn (from)) == align_jumps_log)
+ if (barrier_align (next_real_insn (from)) == align_jumps.levels[0].log)
{
/* We have just passed the barrier in front of the
ADDR_DIFF_VEC, which is stored in found_barrier. Since
@@ -5752,7 +5753,7 @@ barrier_align (rtx_insn *barrier_or_label)
return ((optimize_size
|| ((unsigned) XVECLEN (pat, 1) * GET_MODE_SIZE (GET_MODE (pat))
<= (unsigned) 1 << (CACHE_LOG - 2)))
- ? 1 : align_jumps_log);
+ ? 1 : align_jumps.levels[0].log);
}
rtx_insn *next = next_active_insn (barrier_or_label);
@@ -5770,7 +5771,7 @@ barrier_align (rtx_insn *barrier_or_label)
return 0;
if (! TARGET_SH2 || ! optimize)
- return align_jumps_log;
+ return align_jumps.levels[0].log;
/* When fixing up pcloads, a constant table might be inserted just before
the basic block that ends with the barrier. Thus, we can't trust the
@@ -5848,7 +5849,7 @@ barrier_align (rtx_insn *barrier_or_label)
}
}
- return align_jumps_log;
+ return align_jumps.levels[0].log;
}
/* If we are inside a phony loop, almost any kind of label can turn up as the
@@ -5874,7 +5875,7 @@ sh_loop_align (rtx_insn *label)
|| recog_memoized (next) == CODE_FOR_consttable_2)
return 0;
- return align_loops_log;
+ return align_loops.levels[0].log;
}
/* Do a final pass over the function, just before delayed branch
diff --git a/gcc/config/spu/spu.c b/gcc/config/spu/spu.c
index fe2a2a34a05..e2f45ddb72a 100644
--- a/gcc/config/spu/spu.c
+++ b/gcc/config/spu/spu.c
@@ -241,7 +241,7 @@ spu_option_override (void)
/* Functions must be 8 byte aligned so we correctly handle dual issue */
parse_alignment_opts ();
- if (align_functions_value < 8)
+ if (align_functions.levels[0].get_value () < 8)
str_align_functions = "8";
spu_hint_dist = 8*4 - spu_max_nops*4;
@@ -2772,7 +2772,9 @@ static void
spu_sched_init (FILE *file ATTRIBUTE_UNUSED, int verbose ATTRIBUTE_UNUSED,
int max_ready ATTRIBUTE_UNUSED)
{
- if (align_labels_value > 4 || align_loops_value > 4 || align_jumps_value > 4)
+ if (align_labels.levels[0].get_value () > 4
+ || align_loops.levels[0].get_value () > 4
+ || align_jumps.levels[0].get_value () > 4)
{
/* When any block might be at least 8-byte aligned, assume they
will all be at least 8-byte aligned to make sure dual issue
diff --git a/gcc/config/spu/spu.h b/gcc/config/spu/spu.h
index af7395852ce..99e2d4dd507 100644
--- a/gcc/config/spu/spu.h
+++ b/gcc/config/spu/spu.h
@@ -107,7 +107,7 @@ extern GTY(()) int spu_tune;
(GET_CODE (X) == SYMBOL_REF \
&& (SYMBOL_REF_FLAGS (X) & SYMBOL_FLAG_ALIGN1) == 0 \
&& (! SYMBOL_REF_FUNCTION_P (X) \
- || align_functions_value >= 16))
+ || align_functions.levels[0].get_value () >= 16))
#define PCC_BITFIELD_TYPE_MATTERS 1
diff --git a/gcc/config/visium/visium.c b/gcc/config/visium/visium.c
index 37de6249797..2b402d11f76 100644
--- a/gcc/config/visium/visium.c
+++ b/gcc/config/visium/visium.c
@@ -466,7 +466,7 @@ visium_option_override (void)
else
{
/* But not if they are too far away from a 256-byte boundary. */
- str_align_loops = "256:32";
+ str_align_loops = "256:32:8";
}
}
diff --git a/gcc/config/visium/visium.h b/gcc/config/visium/visium.h
index f1bd00fadb7..edfe3fc499e 100644
--- a/gcc/config/visium/visium.h
+++ b/gcc/config/visium/visium.h
@@ -1450,6 +1450,10 @@ do \
#define ASM_OUTPUT_CASE_END(STREAM, NUM, TABLE) \
asm_fprintf (STREAM, "\t.long 0\n")
+/* Support subalignment values. */
+
+#define SUBALIGN_LOG 3
+
/* Assembler Commands for Alignment
This describes commands for alignment.
@@ -1495,15 +1499,8 @@ do \
if ((LOG) != 0) { \
if ((MAX_SKIP) == 0 || (MAX_SKIP) >= (1<<(LOG))-1) \
fprintf ((STREAM), "\t.p2align %d\n", (LOG)); \
- else { \
+ else \
fprintf ((STREAM), "\t.p2align %d,,%d\n", (LOG), (MAX_SKIP)); \
- /* Make sure that we have at least 8-byte alignment if > 8-byte \
- alignment is preferred. */ \
- if ((LOG) > 3 \
- && (1 << (LOG)) > ((MAX_SKIP) + 1) \
- && (MAX_SKIP) >= 7) \
- fputs ("\t.p2align 3\n", (STREAM)); \
- } \
}
/* Controlling Debugging Information Format
diff --git a/gcc/coretypes.h b/gcc/coretypes.h
index 283b4eb33fe..09d2270e188 100644
--- a/gcc/coretypes.h
+++ b/gcc/coretypes.h
@@ -416,6 +416,7 @@ typedef unsigned char uchar;
#include "insn-modes-inline.h"
#include "machmode.h"
#include "double-int.h"
+#include "align.h"
/* Most host source files will require the following headers. */
#if !defined (GENERATOR_FILE)
#include "real.h"
diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
index 7e2cdc2f202..bd5d13bf0d6 100644
--- a/gcc/doc/tm.texi
+++ b/gcc/doc/tm.texi
@@ -9544,12 +9544,6 @@ to set the variable @var{align_jumps} in the target's
selection in @var{align_jumps} in a @code{JUMP_ALIGN} implementation.
@end defmac
-@deftypefn {Target Hook} int TARGET_ASM_JUMP_ALIGN_MAX_SKIP (rtx_insn *@var{label})
-The maximum number of bytes to skip before @var{label} when applying
-@code{JUMP_ALIGN}. This works only if
-@code{ASM_OUTPUT_MAX_SKIP_ALIGN} is defined.
-@end deftypefn
-
@defmac LABEL_ALIGN_AFTER_BARRIER (@var{label})
The alignment (log base 2) to put in front of @var{label}, which follows
a @code{BARRIER}.
@@ -9559,12 +9553,6 @@ to be done at such a time. Most machine descriptions do not currently
define the macro.
@end defmac
-@deftypefn {Target Hook} int TARGET_ASM_LABEL_ALIGN_AFTER_BARRIER_MAX_SKIP (rtx_insn *@var{label})
-The maximum number of bytes to skip before @var{label} when applying
-@code{LABEL_ALIGN_AFTER_BARRIER}. This works only if
-@code{ASM_OUTPUT_MAX_SKIP_ALIGN} is defined.
-@end deftypefn
-
@defmac LOOP_ALIGN (@var{label})
The alignment (log base 2) to put in front of @var{label} that heads
a frequently executed basic block (usually the header of a loop).
@@ -9579,12 +9567,6 @@ to set the variable @code{align_loops} in the target's
selection in @code{align_loops} in a @code{LOOP_ALIGN} implementation.
@end defmac
-@deftypefn {Target Hook} int TARGET_ASM_LOOP_ALIGN_MAX_SKIP (rtx_insn *@var{label})
-The maximum number of bytes to skip when applying @code{LOOP_ALIGN} to
-@var{label}. This works only if @code{ASM_OUTPUT_MAX_SKIP_ALIGN} is
-defined.
-@end deftypefn
-
@defmac LABEL_ALIGN (@var{label})
The alignment (log base 2) to put in front of @var{label}.
If @code{LABEL_ALIGN_AFTER_BARRIER} / @code{LOOP_ALIGN} specify a different alignment,
@@ -9596,12 +9578,6 @@ to set the variable @code{align_labels} in the target's
selection in @code{align_labels} in a @code{LABEL_ALIGN} implementation.
@end defmac
-@deftypefn {Target Hook} int TARGET_ASM_LABEL_ALIGN_MAX_SKIP (rtx_insn *@var{label})
-The maximum number of bytes to skip when applying @code{LABEL_ALIGN}
-to @var{label}. This works only if @code{ASM_OUTPUT_MAX_SKIP_ALIGN}
-is defined.
-@end deftypefn
-
@defmac ASM_OUTPUT_SKIP (@var{stream}, @var{nbytes})
A C statement to output to the stdio stream @var{stream} an assembler
instruction to advance the location counter by @var{nbytes} bytes.
diff --git a/gcc/doc/tm.texi.in b/gcc/doc/tm.texi.in
index b7b0e8ac8dc..bd4de6a6449 100644
--- a/gcc/doc/tm.texi.in
+++ b/gcc/doc/tm.texi.in
@@ -6522,8 +6522,6 @@ to set the variable @var{align_jumps} in the target's
selection in @var{align_jumps} in a @code{JUMP_ALIGN} implementation.
@end defmac
-@hook TARGET_ASM_JUMP_ALIGN_MAX_SKIP
-
@defmac LABEL_ALIGN_AFTER_BARRIER (@var{label})
The alignment (log base 2) to put in front of @var{label}, which follows
a @code{BARRIER}.
@@ -6533,8 +6531,6 @@ to be done at such a time. Most machine descriptions do not currently
define the macro.
@end defmac
-@hook TARGET_ASM_LABEL_ALIGN_AFTER_BARRIER_MAX_SKIP
-
@defmac LOOP_ALIGN (@var{label})
The alignment (log base 2) to put in front of @var{label} that heads
a frequently executed basic block (usually the header of a loop).
@@ -6549,8 +6545,6 @@ to set the variable @code{align_loops} in the target's
selection in @code{align_loops} in a @code{LOOP_ALIGN} implementation.
@end defmac
-@hook TARGET_ASM_LOOP_ALIGN_MAX_SKIP
-
@defmac LABEL_ALIGN (@var{label})
The alignment (log base 2) to put in front of @var{label}.
If @code{LABEL_ALIGN_AFTER_BARRIER} / @code{LOOP_ALIGN} specify a different alignment,
@@ -6562,8 +6556,6 @@ to set the variable @code{align_labels} in the target's
selection in @code{align_labels} in a @code{LABEL_ALIGN} implementation.
@end defmac
-@hook TARGET_ASM_LABEL_ALIGN_MAX_SKIP
-
@defmac ASM_OUTPUT_SKIP (@var{stream}, @var{nbytes})
A C statement to output to the stdio stream @var{stream} an assembler
instruction to advance the location counter by @var{nbytes} bytes.
diff --git a/gcc/final.c b/gcc/final.c
index ea238656d34..59eb75c3d63 100644
--- a/gcc/final.c
+++ b/gcc/final.c
@@ -327,15 +327,9 @@ int insn_current_align;
for each insn we'll call the alignment chain of this insn in the following
comments. */
-struct label_alignment
-{
- short alignment;
- short max_skip;
-};
-
static rtx *uid_align;
static int *uid_shuid;
-static struct label_alignment *label_align;
+static vec<align_flags> label_align;
/* Indicate that branch shortening hasn't yet been done. */
@@ -473,11 +467,11 @@ get_attr_min_length (rtx_insn *insn)
address mod X to one mod Y, which is Y - X. */
#ifndef LABEL_ALIGN
-#define LABEL_ALIGN(LABEL) align_labels_log
+#define LABEL_ALIGN(LABEL) align_labels
#endif
#ifndef LOOP_ALIGN
-#define LOOP_ALIGN(LABEL) align_loops_log
+#define LOOP_ALIGN(LABEL) align_loops
#endif
#ifndef LABEL_ALIGN_AFTER_BARRIER
@@ -485,33 +479,9 @@ get_attr_min_length (rtx_insn *insn)
#endif
#ifndef JUMP_ALIGN
-#define JUMP_ALIGN(LABEL) align_jumps_log
+#define JUMP_ALIGN(LABEL) align_jumps
#endif
-int
-default_label_align_after_barrier_max_skip (rtx_insn *insn ATTRIBUTE_UNUSED)
-{
- return 0;
-}
-
-int
-default_loop_align_max_skip (rtx_insn *insn ATTRIBUTE_UNUSED)
-{
- return align_loops_max_skip;
-}
-
-int
-default_label_align_max_skip (rtx_insn *insn ATTRIBUTE_UNUSED)
-{
- return align_labels_max_skip;
-}
-
-int
-default_jump_align_max_skip (rtx_insn *insn ATTRIBUTE_UNUSED)
-{
- return align_jumps_max_skip;
-}
-
#ifndef ADDR_VEC_ALIGN
static int
final_addr_vec_align (rtx_jump_table_data *addr_vec)
@@ -536,27 +506,16 @@ final_addr_vec_align (rtx_jump_table_data *addr_vec)
static int min_labelno, max_labelno;
#define LABEL_TO_ALIGNMENT(LABEL) \
- (label_align[CODE_LABEL_NUMBER (LABEL) - min_labelno].alignment)
-
-#define LABEL_TO_MAX_SKIP(LABEL) \
- (label_align[CODE_LABEL_NUMBER (LABEL) - min_labelno].max_skip)
+ (label_align[CODE_LABEL_NUMBER (LABEL) - min_labelno])
/* For the benefit of port specific code do this also as a function. */
-int
+align_flags
label_to_alignment (rtx label)
{
if (CODE_LABEL_NUMBER (label) <= max_labelno)
return LABEL_TO_ALIGNMENT (label);
- return 0;
-}
-
-int
-label_to_max_skip (rtx label)
-{
- if (CODE_LABEL_NUMBER (label) <= max_labelno)
- return LABEL_TO_MAX_SKIP (label);
- return 0;
+ return align_flags ();
}
/* The differences in addresses
@@ -604,8 +563,8 @@ align_fuzz (rtx start, rtx end, int known_align_log, unsigned int growth)
align_addr = INSN_ADDRESSES (uid) - insn_lengths[uid];
if (uid_shuid[uid] > end_shuid)
break;
- known_align_log = LABEL_TO_ALIGNMENT (align_label);
- new_align = 1 << known_align_log;
+ align_flags alignment = LABEL_TO_ALIGNMENT (align_label);
+ new_align = 1 << alignment.levels[0].log;
if (new_align < known_align)
continue;
fuzz += (-align_addr ^ growth) & (new_align - known_align);
@@ -667,18 +626,14 @@ insn_current_reference_address (rtx_insn *branch)
unsigned int
compute_alignments (void)
{
- int log, max_skip, max_log;
basic_block bb;
+ align_flags max_alignment;
- if (label_align)
- {
- free (label_align);
- label_align = 0;
- }
+ label_align.truncate (0);
max_labelno = max_label_num ();
min_labelno = get_first_label_num ();
- label_align = XCNEWVEC (struct label_alignment, max_labelno - min_labelno + 1);
+ label_align.safe_grow_cleared (max_labelno - min_labelno + 1);
/* If not optimizing or optimizing for size, don't assign any alignments. */
if (! optimize || optimize_function_for_size_p (cfun))
@@ -718,8 +673,7 @@ compute_alignments (void)
bb_loop_depth (bb));
continue;
}
- max_log = LABEL_ALIGN (label);
- max_skip = targetm.asm_out.label_align_max_skip (label);
+ max_alignment = LABEL_ALIGN (label);
profile_count fallthru_count = profile_count::zero ();
profile_count branch_count = profile_count::zero ();
@@ -765,14 +719,10 @@ compute_alignments (void)
<= ENTRY_BLOCK_PTR_FOR_FN (cfun)
->count.apply_scale (1, 2)))))
{
- log = JUMP_ALIGN (label);
+ align_flags alignment = JUMP_ALIGN (label);
if (dump_file)
fprintf (dump_file, " jump alignment added.\n");
- if (max_log < log)
- {
- max_log = log;
- max_skip = targetm.asm_out.jump_align_max_skip (label);
- }
+ max_alignment = align_flags::max (max_alignment, alignment);
}
/* In case block is frequent and reached mostly by non-fallthru edge,
align it. It is most likely a first block of loop. */
@@ -785,17 +735,12 @@ compute_alignments (void)
> fallthru_count.apply_scale
(PARAM_VALUE (PARAM_ALIGN_LOOP_ITERATIONS), 1)))
{
- log = LOOP_ALIGN (label);
+ align_flags alignment = LOOP_ALIGN (label);
if (dump_file)
fprintf (dump_file, " internal loop alignment added.\n");
- if (max_log < log)
- {
- max_log = log;
- max_skip = targetm.asm_out.loop_align_max_skip (label);
- }
+ max_alignment = align_flags::max (max_alignment, alignment);
}
- LABEL_TO_ALIGNMENT (label) = max_log;
- LABEL_TO_MAX_SKIP (label) = max_skip;
+ LABEL_TO_ALIGNMENT (label) = max_alignment;
}
loop_optimizer_finalize ();
@@ -817,14 +762,11 @@ grow_label_align (void)
n_labels = max_labelno - min_labelno + 1;
n_old_labels = old - min_labelno + 1;
- label_align = XRESIZEVEC (struct label_alignment, label_align, n_labels);
+ label_align.safe_grow_cleared (n_labels);
/* Range of labels grows monotonically in the function. Failing here
means that the initialization of array got lost. */
gcc_assert (n_old_labels <= n_labels);
-
- memset (label_align + n_old_labels, 0,
- (n_labels - n_old_labels) * sizeof (struct label_alignment));
}
/* Update the already computed alignment information. LABEL_PAIRS is a vector
@@ -842,10 +784,7 @@ update_alignments (vec<rtx> &label_pairs)
FOR_EACH_VEC_ELT (label_pairs, i, iter)
if (i & 1)
- {
- LABEL_TO_ALIGNMENT (label) = LABEL_TO_ALIGNMENT (iter);
- LABEL_TO_MAX_SKIP (label) = LABEL_TO_MAX_SKIP (iter);
- }
+ LABEL_TO_ALIGNMENT (label) = LABEL_TO_ALIGNMENT (iter);
else
label = iter;
}
@@ -903,8 +842,6 @@ shorten_branches (rtx_insn *first)
rtx_insn *insn;
int max_uid;
int i;
- int max_log;
- int max_skip;
#define MAX_CODE_ALIGN 16
rtx_insn *seq;
int something_changed = 1;
@@ -926,17 +863,14 @@ shorten_branches (rtx_insn *first)
/* Initialize label_align and set up uid_shuid to be strictly
monotonically rising with insn order. */
- /* We use max_log here to keep track of the maximum alignment we want to
+ /* We use alignment here to keep track of the maximum alignment we want to
impose on the next CODE_LABEL (or the current one if we are processing
the CODE_LABEL itself). */
- max_log = 0;
- max_skip = 0;
+ align_flags max_alignment;
for (insn = get_insns (), i = 1; insn; insn = NEXT_INSN (insn))
{
- int log;
-
INSN_SHUID (insn) = i++;
if (INSN_P (insn))
continue;
@@ -944,22 +878,14 @@ shorten_branches (rtx_insn *first)
if (rtx_code_label *label = dyn_cast <rtx_code_label *> (insn))
{
/* Merge in alignments computed by compute_alignments. */
- log = LABEL_TO_ALIGNMENT (label);
- if (max_log < log)
- {
- max_log = log;
- max_skip = LABEL_TO_MAX_SKIP (label);
- }
+ align_flags alignment = LABEL_TO_ALIGNMENT (label);
+ max_alignment = align_flags::max (max_alignment, alignment);
rtx_jump_table_data *table = jump_table_for_label (label);
if (!table)
{
- log = LABEL_ALIGN (label);
- if (max_log < log)
- {
- max_log = log;
- max_skip = targetm.asm_out.label_align_max_skip (label);
- }
+ align_flags alignment = LABEL_ALIGN (label);
+ max_alignment = align_flags::max (max_alignment, alignment);
}
/* ADDR_VECs only take room if read-only data goes into the text
section. */
@@ -967,17 +893,11 @@ shorten_branches (rtx_insn *first)
|| readonly_data_section == text_section)
&& table)
{
- log = ADDR_VEC_ALIGN (table);
- if (max_log < log)
- {
- max_log = log;
- max_skip = targetm.asm_out.label_align_max_skip (label);
- }
+ align_flags alignment = align_flags (ADDR_VEC_ALIGN (table));
+ max_alignment = align_flags::max (max_alignment, alignment);
}
- LABEL_TO_ALIGNMENT (label) = max_log;
- LABEL_TO_MAX_SKIP (label) = max_skip;
- max_log = 0;
- max_skip = 0;
+ LABEL_TO_ALIGNMENT (label) = max_alignment;
+ max_alignment = align_flags ();
}
else if (BARRIER_P (insn))
{
@@ -987,12 +907,9 @@ shorten_branches (rtx_insn *first)
label = NEXT_INSN (label))
if (LABEL_P (label))
{
- log = LABEL_ALIGN_AFTER_BARRIER (insn);
- if (max_log < log)
- {
- max_log = log;
- max_skip = targetm.asm_out.label_align_after_barrier_max_skip (label);
- }
+ align_flags alignment
+ = align_flags (LABEL_ALIGN_AFTER_BARRIER (insn));
+ max_alignment = align_flags::max (max_alignment, alignment);
break;
}
}
@@ -1023,11 +940,12 @@ shorten_branches (rtx_insn *first)
{
int uid = INSN_UID (seq);
int log;
- log = (LABEL_P (seq) ? LABEL_TO_ALIGNMENT (seq) : 0);
+ log = (LABEL_P (seq) ? LABEL_TO_ALIGNMENT (seq).levels[0].log : 0);
uid_align[uid] = align_tab[0];
if (log)
{
/* Found an alignment label. */
+ gcc_checking_assert (log < MAX_CODE_ALIGN + 1);
uid_align[uid] = align_tab[log];
for (i = log - 1; i >= 0; i--)
align_tab[i] = seq;
@@ -1078,8 +996,10 @@ shorten_branches (rtx_insn *first)
max = shuid;
max_lab = lab;
}
- if (min_align > LABEL_TO_ALIGNMENT (lab))
- min_align = LABEL_TO_ALIGNMENT (lab);
+
+ int label_alignment = LABEL_TO_ALIGNMENT (lab).levels[0].log;
+ if (min_align > label_alignment)
+ min_align = label_alignment;
}
XEXP (pat, 2) = gen_rtx_LABEL_REF (Pmode, min_lab);
XEXP (pat, 3) = gen_rtx_LABEL_REF (Pmode, max_lab);
@@ -1113,7 +1033,7 @@ shorten_branches (rtx_insn *first)
if (LABEL_P (insn))
{
- int log = LABEL_TO_ALIGNMENT (insn);
+ int log = LABEL_TO_ALIGNMENT (insn).levels[0].log;
if (log)
{
int align = 1 << log;
@@ -1221,7 +1141,7 @@ shorten_branches (rtx_insn *first)
if (rtx_code_label *label = dyn_cast <rtx_code_label *> (insn))
{
- int log = LABEL_TO_ALIGNMENT (label);
+ int log = LABEL_TO_ALIGNMENT (label).levels[0].log;
#ifdef CASE_VECTOR_SHORTEN_MODE
/* If the mode of a following jump table was changed, we
@@ -1296,7 +1216,7 @@ shorten_branches (rtx_insn *first)
prev = PREV_INSN (prev))
if (varying_length[INSN_UID (prev)] & 2)
{
- rel_align = LABEL_TO_ALIGNMENT (prev);
+ rel_align = LABEL_TO_ALIGNMENT (prev).levels[0].log;
break;
}
@@ -2519,26 +2439,20 @@ final_scan_insn_1 (rtx_insn *insn, FILE *file, int optimize_p ATTRIBUTE_UNUSED,
some insn, e.g. sh.c output_branchy_insn. */
if (CODE_LABEL_NUMBER (insn) <= max_labelno)
{
- int align = LABEL_TO_ALIGNMENT (insn);
-#ifdef ASM_OUTPUT_MAX_SKIP_ALIGN
- int max_skip = LABEL_TO_MAX_SKIP (insn);
-#endif
-
- if (align && NEXT_INSN (insn))
+ align_flags alignment = LABEL_TO_ALIGNMENT (insn);
+ if (alignment.levels[0].log && NEXT_INSN (insn))
{
#ifdef ASM_OUTPUT_MAX_SKIP_ALIGN
- ASM_OUTPUT_MAX_SKIP_ALIGN (file, align, max_skip);
- /* Above, we don't know whether a label, jump or loop
- alignment was used. Conservatively apply
- label subalignment, not jump or loop
- subalignment (they are almost always larger). */
- ASM_OUTPUT_MAX_SKIP_ALIGN (file, state_align_labels.levels[1].log,
- state_align_labels.levels[1].maxskip);
+ /* Output both primary and secondary alignment. */
+ ASM_OUTPUT_MAX_SKIP_ALIGN (file, alignment.levels[0].log,
+ alignment.levels[0].maxskip);
+ ASM_OUTPUT_MAX_SKIP_ALIGN (file, alignment.levels[1].log,
+ alignment.levels[1].maxskip);
#else
#ifdef ASM_OUTPUT_ALIGN_WITH_NOP
- ASM_OUTPUT_ALIGN_WITH_NOP (file, align);
+ ASM_OUTPUT_ALIGN_WITH_NOP (file, alignment.levels[0].log);
#else
- ASM_OUTPUT_ALIGN (file, align);
+ ASM_OUTPUT_ALIGN (file, alignment.levels[0].log);
#endif
#endif
}
diff --git a/gcc/flags.h b/gcc/flags.h
index bfd645b7f29..f025efb4b8e 100644
--- a/gcc/flags.h
+++ b/gcc/flags.h
@@ -42,24 +42,6 @@ extern bool final_insns_dump_p;
\f
/* Other basic status info about current function. */
-/* Align flags tuple with alignment in log form and with a maximum skip. */
-
-struct align_flags_tuple
-{
- /* Values of the -falign-* flags: how much to align labels in code.
- log is "align to 2^log" (so 0 means no alignment).
- maxskip is the maximum allowed amount of padding to insert. */
- int log;
- int maxskip;
-};
-
-/* Target-dependent global state. */
-
-struct align_flags
-{
- align_flags_tuple levels[2];
-};
-
struct target_flag_state
{
/* Each falign-foo can generate up to two levels of alignment:
@@ -80,22 +62,10 @@ extern struct target_flag_state *this_target_flag_state;
#define this_target_flag_state (&default_target_flag_state)
#endif
-#define state_align_loops (this_target_flag_state->x_align_loops)
-#define state_align_jumps (this_target_flag_state->x_align_jumps)
-#define state_align_labels (this_target_flag_state->x_align_labels)
-#define state_align_functions (this_target_flag_state->x_align_functions)
-#define align_loops_log (state_align_loops.levels[0].log)
-#define align_jumps_log (state_align_jumps.levels[0].log)
-#define align_labels_log (state_align_labels.levels[0].log)
-#define align_functions_log (state_align_functions.levels[0].log)
-#define align_loops_max_skip (state_align_loops.levels[0].maxskip)
-#define align_jumps_max_skip (state_align_jumps.levels[0].maxskip)
-#define align_labels_max_skip (state_align_labels.levels[0].maxskip)
-#define align_functions_max_skip (state_align_functions.levels[0].maxskip)
-#define align_loops_value (align_loops_max_skip + 1)
-#define align_jumps_value (align_jumps_max_skip + 1)
-#define align_labels_value (align_labels_max_skip + 1)
-#define align_functions_value (align_functions_max_skip + 1)
+#define align_loops (this_target_flag_state->x_align_loops)
+#define align_jumps (this_target_flag_state->x_align_jumps)
+#define align_labels (this_target_flag_state->x_align_labels)
+#define align_functions (this_target_flag_state->x_align_functions)
/* String representaions of the above options are available in
const char *str_align_foo. NULL if not set. */
diff --git a/gcc/output.h b/gcc/output.h
index afe72bece31..333389e484c 100644
--- a/gcc/output.h
+++ b/gcc/output.h
@@ -96,11 +96,7 @@ extern int insn_current_reference_address (rtx_insn *);
/* Find the alignment associated with a CODE_LABEL.
Defined in final.c. */
-extern int label_to_alignment (rtx);
-
-/* Find the alignment maximum skip associated with a CODE_LABEL.
- Defined in final.c. */
-extern int label_to_max_skip (rtx);
+extern align_flags label_to_alignment (rtx);
/* Output a LABEL_REF, or a bare CODE_LABEL, as an assembler symbol. */
extern void output_asm_label (rtx);
diff --git a/gcc/system.h b/gcc/system.h
index 8295577618a..f87fbaae69e 100644
--- a/gcc/system.h
+++ b/gcc/system.h
@@ -918,8 +918,6 @@ extern void fancy_abort (const char *, int, const char *)
RETURN_POPS_ARGS UNITS_PER_SIMD_WORD OVERRIDE_OPTIONS \
OPTIMIZATION_OPTIONS CLASS_LIKELY_SPILLED_P \
USING_SJLJ_EXCEPTIONS TARGET_UNWIND_INFO \
- LABEL_ALIGN_MAX_SKIP LOOP_ALIGN_MAX_SKIP \
- LABEL_ALIGN_AFTER_BARRIER_MAX_SKIP JUMP_ALIGN_MAX_SKIP \
CAN_DEBUG_WITHOUT_FP UNLIKELY_EXECUTED_TEXT_SECTION_NAME \
HOT_TEXT_SECTION_NAME LEGITIMATE_CONSTANT_P ALWAYS_STRIP_DOTDOT \
OUTPUT_ADDR_CONST_EXTRA SMALL_REGISTER_CLASSES ASM_OUTPUT_IDENT \
diff --git a/gcc/target.def b/gcc/target.def
index 112c77222a9..eb6288218b6 100644
--- a/gcc/target.def
+++ b/gcc/target.def
@@ -65,46 +65,6 @@ the string should contain a tab, a pseudo-op, and then another tab.",
DEFHOOKPOD (aligned_op, "*", struct asm_int_op, TARGET_ASM_ALIGNED_INT_OP)
DEFHOOKPOD (unaligned_op, "*", struct asm_int_op, TARGET_ASM_UNALIGNED_INT_OP)
-/* The maximum number of bytes to skip when applying
- LABEL_ALIGN_AFTER_BARRIER. */
-DEFHOOK
-(label_align_after_barrier_max_skip,
- "The maximum number of bytes to skip before @var{label} when applying\n\
-@code{LABEL_ALIGN_AFTER_BARRIER}. This works only if\n\
-@code{ASM_OUTPUT_MAX_SKIP_ALIGN} is defined.",
- int, (rtx_insn *label),
- default_label_align_after_barrier_max_skip)
-
-/* The maximum number of bytes to skip when applying
- LOOP_ALIGN. */
-DEFHOOK
-(loop_align_max_skip,
- "The maximum number of bytes to skip when applying @code{LOOP_ALIGN} to\n\
-@var{label}. This works only if @code{ASM_OUTPUT_MAX_SKIP_ALIGN} is\n\
-defined.",
- int, (rtx_insn *label),
- default_loop_align_max_skip)
-
-/* The maximum number of bytes to skip when applying
- LABEL_ALIGN. */
-DEFHOOK
-(label_align_max_skip,
- "The maximum number of bytes to skip when applying @code{LABEL_ALIGN}\n\
-to @var{label}. This works only if @code{ASM_OUTPUT_MAX_SKIP_ALIGN}\n\
-is defined.",
- int, (rtx_insn *label),
- default_label_align_max_skip)
-
-/* The maximum number of bytes to skip when applying
- JUMP_ALIGN. */
-DEFHOOK
-(jump_align_max_skip,
- "The maximum number of bytes to skip before @var{label} when applying\n\
-@code{JUMP_ALIGN}. This works only if\n\
-@code{ASM_OUTPUT_MAX_SKIP_ALIGN} is defined.",
- int, (rtx_insn *label),
- default_jump_align_max_skip)
-
/* Try to output the assembler code for an integer object whose
value is given by X. SIZE is the size of the object in bytes and
ALIGNED_P indicates whether it is aligned. Return true if
diff --git a/gcc/targhooks.h b/gcc/targhooks.h
index 4107e22d97c..0d3ecc42658 100644
--- a/gcc/targhooks.h
+++ b/gcc/targhooks.h
@@ -236,10 +236,6 @@ extern enum unwind_info_type default_debug_unwind_info (void);
extern void default_canonicalize_comparison (int *, rtx *, rtx *, bool);
-extern int default_label_align_after_barrier_max_skip (rtx_insn *);
-extern int default_loop_align_max_skip (rtx_insn *);
-extern int default_label_align_max_skip (rtx_insn *);
-extern int default_jump_align_max_skip (rtx_insn *);
extern section * default_function_section(tree decl, enum node_frequency freq,
bool startup, bool exit);
extern unsigned int default_dwarf_poly_indeterminate_value (unsigned int,
diff --git a/gcc/toplev.c b/gcc/toplev.c
index cf7bab655bd..1fb8fcc1c8b 100644
--- a/gcc/toplev.c
+++ b/gcc/toplev.c
@@ -1207,24 +1207,26 @@ read_log_maxskip (auto_vec<unsigned> &values, align_flags_tuple *a)
unsigned n = values.pop ();
if (n != 0)
a->log = floor_log2 (n * 2 - 1);
+
if (values.is_empty ())
a->maxskip = n ? n - 1 : 0;
else
{
unsigned m = values.pop ();
- if (m > n)
- m = n;
/* -falign-foo=N:M means M-1 max bytes of padding, not M. */
if (m > 0)
m--;
a->maxskip = m;
}
+
+ /* Normalize the tuple. */
+ a->normalize ();
}
/* Parse "N[:M[:N2[:M2]]]" string FLAG into a pair of struct align_flags. */
static void
-parse_N_M (const char *flag, align_flags &a, unsigned int min_align_log)
+parse_N_M (const char *flag, align_flags &a)
{
if (flag)
{
@@ -1269,7 +1271,10 @@ parse_N_M (const char *flag, align_flags &a, unsigned int min_align_log)
{
/* Set N2 unless subalign can never have any effect. */
if (align > a.levels[0].maxskip + 1)
- a.levels[1].log = SUBALIGN_LOG;
+ {
+ a.levels[1].log = SUBALIGN_LOG;
+ a.levels[1].normalize ();
+ }
}
}
#endif
@@ -1277,40 +1282,17 @@ parse_N_M (const char *flag, align_flags &a, unsigned int min_align_log)
/* Cache seen value. */
cache.put (flag, a);
}
- else
- {
- /* Reset values to zero. */
- for (unsigned i = 0; i < 2; i++)
- {
- a.levels[i].log = 0;
- a.levels[i].maxskip = 0;
- }
- }
-
- if ((unsigned int)a.levels[0].log < min_align_log)
- {
- a.levels[0].log = min_align_log;
- a.levels[0].maxskip = (1 << min_align_log) - 1;
- }
}
-/* Minimum alignment requirements, if arch has them. */
-
-unsigned int min_align_loops_log = 0;
-unsigned int min_align_jumps_log = 0;
-unsigned int min_align_labels_log = 0;
-unsigned int min_align_functions_log = 0;
-
/* Process -falign-foo=N[:M[:N2[:M2]]] options. */
void
parse_alignment_opts (void)
{
- parse_N_M (str_align_loops, state_align_loops, min_align_loops_log);
- parse_N_M (str_align_jumps, state_align_jumps, min_align_jumps_log);
- parse_N_M (str_align_labels, state_align_labels, min_align_labels_log);
- parse_N_M (str_align_functions, state_align_functions,
- min_align_functions_log);
+ parse_N_M (str_align_loops, align_loops);
+ parse_N_M (str_align_jumps, align_jumps);
+ parse_N_M (str_align_labels, align_labels);
+ parse_N_M (str_align_functions, align_functions);
}
/* Process the options that have been parsed. */
diff --git a/gcc/varasm.c b/gcc/varasm.c
index 81f460643ea..0d3609e2807 100644
--- a/gcc/varasm.c
+++ b/gcc/varasm.c
@@ -1801,25 +1801,25 @@ assemble_start_function (tree decl, const char *fnname)
Note that we still need to align to DECL_ALIGN, as above,
because ASM_OUTPUT_MAX_SKIP_ALIGN might not do any alignment at all. */
if (! DECL_USER_ALIGN (decl)
- && align_functions_log > align
+ && align_functions.levels[0].log > align
&& optimize_function_for_speed_p (cfun))
{
#ifdef ASM_OUTPUT_MAX_SKIP_ALIGN
- int align_log = state_align_functions.levels[0].log;
+ int align_log = align_functions.levels[0].log;
#endif
- int max_skip = state_align_functions.levels[0].maxskip;
+ int max_skip = align_functions.levels[0].maxskip;
if (flag_limit_function_alignment && crtl->max_insn_address > 0
&& max_skip >= crtl->max_insn_address)
max_skip = crtl->max_insn_address - 1;
#ifdef ASM_OUTPUT_MAX_SKIP_ALIGN
ASM_OUTPUT_MAX_SKIP_ALIGN (asm_out_file, align_log, max_skip);
- if (max_skip == state_align_functions.levels[0].maxskip)
+ if (max_skip == align_functions.levels[0].maxskip)
ASM_OUTPUT_MAX_SKIP_ALIGN (asm_out_file,
- state_align_functions.levels[1].log,
- state_align_functions.levels[1].maxskip);
+ align_functions.levels[1].log,
+ align_functions.levels[1].maxskip);
#else
- ASM_OUTPUT_ALIGN (asm_out_file, state_align_functions.levels[0].log);
+ ASM_OUTPUT_ALIGN (asm_out_file, align_functions.levels[0].log);
#endif
}
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 2/4] Fix coding style of ASM_OUTPUT_ALIGN.
2018-07-16 8:53 [PATCH 0/4] Alignment option enhancements marxin
2018-07-16 8:53 ` [PATCH 3/4] Define MAX_CODE_ALIGN globally marxin
@ 2018-07-16 8:53 ` marxin
2018-07-16 8:53 ` [PATCH 1/4] Clean up of new format of -falign-FOO marxin
` (2 subsequent siblings)
4 siblings, 0 replies; 15+ messages in thread
From: marxin @ 2018-07-16 8:53 UTC (permalink / raw)
To: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 1021 bytes --]
gcc/ChangeLog:
2018-07-11 Martin Liska <mliska@suse.cz>
* config/i386/att.h (ASM_OUTPUT_ALIGN): Fix spacing
in order to fulfil coding style.
* config/i386/cygming.h (ASM_OUTPUT_ALIGN): Likewise.
* config/i386/gas.h (ASM_OUTPUT_MAX_SKIP_ALIGN): Likewise.
* config/i386/x86-64.h (ASM_OUTPUT_MAX_SKIP_PAD): Likewise.
* config/iq2000/iq2000.h (ASM_OUTPUT_ALIGN): Likewise.
* config/pa/pa.h (ASM_OUTPUT_ALIGN): Likewise.
* config/sparc/sol2.h (ASM_OUTPUT_ALIGN_WITH_NOP): Likewise.
* config/sparc/sparc.h (ASM_OUTPUT_ALIGN): Likewise.
* config/visium/visium.h (ASM_OUTPUT_ALIGN): Likewise.
(ASM_OUTPUT_MAX_SKIP_ALIGN): Likewise.
---
gcc/config/i386/att.h | 2 +-
gcc/config/i386/cygming.h | 2 +-
gcc/config/i386/gas.h | 2 +-
gcc/config/i386/x86-64.h | 2 +-
gcc/config/iq2000/iq2000.h | 2 +-
gcc/config/pa/pa.h | 2 +-
gcc/config/sparc/sol2.h | 2 +-
gcc/config/sparc/sparc.h | 2 +-
gcc/config/visium/visium.h | 4 ++--
9 files changed, 10 insertions(+), 10 deletions(-)
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0002-Fix-coding-style-of-ASM_OUTPUT_ALIGN.patch --]
[-- Type: text/x-patch; name="0002-Fix-coding-style-of-ASM_OUTPUT_ALIGN.patch", Size: 5171 bytes --]
diff --git a/gcc/config/i386/att.h b/gcc/config/i386/att.h
index 8e4bc2e8d94..7dd359b9001 100644
--- a/gcc/config/i386/att.h
+++ b/gcc/config/i386/att.h
@@ -57,7 +57,7 @@ do \
to a multiple of 2**LOG bytes. */
#define ASM_OUTPUT_ALIGN(FILE,LOG) \
- if ((LOG)!=0) fprintf ((FILE), "\t.align %d\n", 1<<(LOG))
+ if ((LOG) != 0) fprintf ((FILE), "\t.align %d\n", 1 << (LOG))
/* This is how to output an assembler line
that says to advance the location counter by SIZE bytes. */
diff --git a/gcc/config/i386/cygming.h b/gcc/config/i386/cygming.h
index 80a0a37e308..cfe563f47af 100644
--- a/gcc/config/i386/cygming.h
+++ b/gcc/config/i386/cygming.h
@@ -290,7 +290,7 @@ do { \
#undef ASM_OUTPUT_ALIGN
#define ASM_OUTPUT_ALIGN(FILE,LOG) \
- if ((LOG)!=0) fprintf ((FILE), "\t.align %d\n", 1<<(LOG))
+ if ((LOG) != 0) fprintf ((FILE), "\t.align %d\n", 1 << (LOG))
/* Windows uses explicit import from shared libraries. */
#define MULTIPLE_SYMBOL_SPACES 1
diff --git a/gcc/config/i386/gas.h b/gcc/config/i386/gas.h
index e149ab1360c..eb30db5d326 100644
--- a/gcc/config/i386/gas.h
+++ b/gcc/config/i386/gas.h
@@ -69,7 +69,7 @@ along with GCC; see the file COPYING3. If not see
#ifdef HAVE_GAS_MAX_SKIP_P2ALIGN
# define ASM_OUTPUT_MAX_SKIP_ALIGN(FILE,LOG,MAX_SKIP) \
if ((LOG) != 0) { \
- if ((MAX_SKIP) == 0 || (MAX_SKIP) >= (1<<(LOG))-1) \
+ if ((MAX_SKIP) == 0 || (MAX_SKIP) >= (1 << (LOG)) - 1) \
fprintf ((FILE), "\t.p2align %d\n", (LOG)); \
else \
fprintf ((FILE), "\t.p2align %d,,%d\n", (LOG), (MAX_SKIP)); \
diff --git a/gcc/config/i386/x86-64.h b/gcc/config/i386/x86-64.h
index 66bbd5edc59..bc97454d2f6 100644
--- a/gcc/config/i386/x86-64.h
+++ b/gcc/config/i386/x86-64.h
@@ -77,7 +77,7 @@ see the files COPYING3 and COPYING.RUNTIME respectively. If not, see
#define ASM_OUTPUT_MAX_SKIP_PAD(FILE, LOG, MAX_SKIP) \
if ((LOG) != 0) \
{ \
- if ((MAX_SKIP) == 0 || (MAX_SKIP) >= (1<<(LOG))-1) \
+ if ((MAX_SKIP) == 0 || (MAX_SKIP) >= (1 << (LOG)) - 1) \
fprintf ((FILE), "\t.p2align %d\n", (LOG)); \
else \
fprintf ((FILE), "\t.p2align %d,,%d\n", (LOG), (MAX_SKIP)); \
diff --git a/gcc/config/iq2000/iq2000.h b/gcc/config/iq2000/iq2000.h
index 87c4114b409..1dff91f6c0f 100644
--- a/gcc/config/iq2000/iq2000.h
+++ b/gcc/config/iq2000/iq2000.h
@@ -489,7 +489,7 @@ while (0)
#define ASM_OUTPUT_ALIGN(STREAM,LOG) \
if ((LOG) != 0) \
- fprintf (STREAM, "\t.balign %d\n", 1<<(LOG))
+ fprintf (STREAM, "\t.balign %d\n", 1 << (LOG))
\f
/* Macros Affecting all Debug Formats. */
diff --git a/gcc/config/pa/pa.h b/gcc/config/pa/pa.h
index 67b3c714ea8..024e7b831ec 100644
--- a/gcc/config/pa/pa.h
+++ b/gcc/config/pa/pa.h
@@ -1187,7 +1187,7 @@ do { \
location counter to a multiple of 2**LOG bytes. */
#define ASM_OUTPUT_ALIGN(FILE,LOG) \
- fprintf (FILE, "\t.align %d\n", (1<<(LOG)))
+ fprintf (FILE, "\t.align %d\n", (1 << (LOG)))
#define ASM_OUTPUT_SKIP(FILE,SIZE) \
fprintf (FILE, "\t.blockz " HOST_WIDE_INT_PRINT_UNSIGNED"\n", \
diff --git a/gcc/config/sparc/sol2.h b/gcc/config/sparc/sol2.h
index b84f218c811..1bd4d61b2f3 100644
--- a/gcc/config/sparc/sol2.h
+++ b/gcc/config/sparc/sol2.h
@@ -413,7 +413,7 @@ extern const char *host_detect_local_cpu (int argc, const char **argv);
with GNU as. */
#define ASM_OUTPUT_ALIGN_WITH_NOP(FILE,LOG) \
if ((LOG) != 0) \
- fprintf (FILE, "\t.align %d,0x1000000\n", (1<<(LOG)))
+ fprintf (FILE, "\t.align %d,0x1000000\n", (1 << (LOG)))
/* Use Solaris ELF section syntax with Sun as. */
#undef TARGET_ASM_NAMED_SECTION
diff --git a/gcc/config/sparc/sparc.h b/gcc/config/sparc/sparc.h
index 419995431e2..032a91d467a 100644
--- a/gcc/config/sparc/sparc.h
+++ b/gcc/config/sparc/sparc.h
@@ -1659,7 +1659,7 @@ do { \
#define ASM_OUTPUT_ALIGN(FILE,LOG) \
if ((LOG) != 0) \
- fprintf (FILE, "\t.align %d\n", (1<<(LOG)))
+ fprintf (FILE, "\t.align %d\n", (1 << (LOG)))
#define ASM_OUTPUT_SKIP(FILE,SIZE) \
fprintf (FILE, "\t.skip " HOST_WIDE_INT_PRINT_UNSIGNED"\n", (SIZE))
diff --git a/gcc/config/visium/visium.h b/gcc/config/visium/visium.h
index edfe3fc499e..5bd56a0633e 100644
--- a/gcc/config/visium/visium.h
+++ b/gcc/config/visium/visium.h
@@ -1486,7 +1486,7 @@ do \
POWER bytes. POWER will be a C expression of type `int'. */
#define ASM_OUTPUT_ALIGN(STREAM,LOG) \
if ((LOG) != 0) \
- fprintf (STREAM, "\t.align %d\n", (1<<(LOG)))
+ fprintf (STREAM, "\t.align %d\n", (1 << (LOG)))
/* `ASM_OUTPUT_MAX_SKIP_ALIGN (STREAM, POWER, MAX_SKIP)`
@@ -1497,7 +1497,7 @@ do \
expression of type `int'. */
#define ASM_OUTPUT_MAX_SKIP_ALIGN(STREAM,LOG,MAX_SKIP) \
if ((LOG) != 0) { \
- if ((MAX_SKIP) == 0 || (MAX_SKIP) >= (1<<(LOG))-1) \
+ if ((MAX_SKIP) == 0 || (MAX_SKIP) >= (1 << (LOG)) - 1) \
fprintf ((STREAM), "\t.p2align %d\n", (LOG)); \
else \
fprintf ((STREAM), "\t.p2align %d,,%d\n", (LOG), (MAX_SKIP)); \
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 0/4] Alignment option enhancements
@ 2018-07-16 8:53 marxin
2018-07-16 8:53 ` [PATCH 3/4] Define MAX_CODE_ALIGN globally marxin
` (4 more replies)
0 siblings, 5 replies; 15+ messages in thread
From: marxin @ 2018-07-16 8:53 UTC (permalink / raw)
To: gcc-patches
Hi.
As you probably noticed, current trunk accepts extended format
of -falign-FOO options.
However there are some limitations that are addressed in the patchset:
1) E.g. align_labels_max_skip and align_labels_log are separate
values and targets can override them via LABEL_ALIGN and
LABEL_ALIGN_MAX_SKIP. If you take a look at code in final.c:
if (max_log < log)
{
max_log = log;
max_skip = targetm.asm_out.label_align_max_skip (label);
}
It improves alignment, but max_skip can be zero and then we end up
with no alignment. Thus I see it saner to provide just a single hook
and return align_flags class instance.
2) In final.c we mix label, jump and loop alignments. I believe when
combining them together, one should do maximum and not rely on:
#ifdef ASM_OUTPUT_MAX_SKIP_ALIGN
ASM_OUTPUT_MAX_SKIP_ALIGN (file, align, max_skip);
/* Above, we don't know whether a label, jump or loop
alignment was used. Conservatively apply
label subalignment, not jump or loop
subalignment (they are almost always larger). */
That's implemented in align_flags::max function. When combining
for instance -falign-labels=1000 and -falign-jumps=2000, I hope
using alignment of 2000 is the right choice.
3) I removed various align_* defines and use directly
align_{*}.
4) I ported visium target to provide always secondary alignment of 8B.
That's the same what x86_64 does and was factored out in previous commits
I did.
5) Globally MAX_CODE_ALIGN is introduced.
6) Various coding styles are fixes in target code.
7) OPT_falign_* is not set with -Os. Then --help=common -Q reports proper values.
I tested that on x86_64-linux-gnu and ppc64le-linux. And also I tested all targets
that I touched, these should provide equal results.
Martin
marxin (4):
Clean up of new format of -falign-FOO.
Fix coding style of ASM_OUTPUT_ALIGN.
Define MAX_CODE_ALIGN globally.
Do not enable OPT_falign_* for -Os.
gcc/align.h | 80 ++++++++++++
gcc/common/config/i386/i386-common.c | 3 -
gcc/config/alpha/alpha.c | 5 +-
gcc/config/i386/att.h | 2 +-
gcc/config/i386/cygming.h | 2 +-
gcc/config/i386/gas.h | 2 +-
gcc/config/i386/i386.c | 5 +-
gcc/config/i386/x86-64.h | 2 +-
gcc/config/iq2000/iq2000.h | 2 +-
gcc/config/m32r/m32r.h | 3 +-
gcc/config/m68k/m68k.c | 10 +-
gcc/config/nds32/nds32.h | 9 +-
gcc/config/pa/pa.h | 2 +-
gcc/config/powerpcspe/powerpcspe.c | 18 +--
gcc/config/rs6000/rs6000-protos.h | 2 +-
gcc/config/rs6000/rs6000.c | 18 +--
gcc/config/rx/rx-protos.h | 2 +-
gcc/config/rx/rx.c | 59 +++++----
gcc/config/rx/rx.h | 6 +-
gcc/config/s390/s390.c | 9 +-
gcc/config/sh/sh.c | 19 +--
gcc/config/sparc/sol2.h | 2 +-
gcc/config/sparc/sparc.h | 2 +-
gcc/config/spu/spu.c | 6 +-
gcc/config/spu/spu.h | 2 +-
gcc/config/visium/visium.c | 2 +-
gcc/config/visium/visium.h | 17 +--
gcc/coretypes.h | 1 +
gcc/doc/tm.texi | 24 ----
gcc/doc/tm.texi.in | 8 --
gcc/final.c | 189 ++++++++-------------------
gcc/flags.h | 38 +-----
gcc/opts.c | 12 +-
gcc/output.h | 6 +-
gcc/system.h | 2 -
gcc/target.def | 40 ------
gcc/targhooks.h | 4 -
gcc/toplev.c | 44 ++-----
gcc/varasm.c | 14 +-
39 files changed, 262 insertions(+), 411 deletions(-)
create mode 100644 gcc/align.h
--
2.18.0
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 4/4] Do not enable OPT_falign_* for -Os.
2018-07-16 8:53 [PATCH 0/4] Alignment option enhancements marxin
` (2 preceding siblings ...)
2018-07-16 8:53 ` [PATCH 1/4] Clean up of new format of -falign-FOO marxin
@ 2018-07-16 8:53 ` marxin
2018-07-16 14:10 ` [PATCH 0/4] Alignment option enhancements Richard Biener
4 siblings, 0 replies; 15+ messages in thread
From: marxin @ 2018-07-16 8:53 UTC (permalink / raw)
To: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 187 bytes --]
gcc/ChangeLog:
2018-07-12 Martin Liska <mliska@suse.cz>
* opts.c: Do not enable OPT_falign_* for -Os.
---
gcc/opts.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0004-Do-not-enable-OPT_falign_-for-Os.patch --]
[-- Type: text/x-patch; name="0004-Do-not-enable-OPT_falign_-for-Os.patch", Size: 1073 bytes --]
diff --git a/gcc/opts.c b/gcc/opts.c
index 0625b15b27b..b8ae8756b4f 100644
--- a/gcc/opts.c
+++ b/gcc/opts.c
@@ -510,10 +510,10 @@ static const struct default_options default_options_table[] =
{ OPT_LEVELS_2_PLUS, OPT_fdevirtualize, NULL, 1 },
{ OPT_LEVELS_2_PLUS, OPT_fdevirtualize_speculatively, NULL, 1 },
{ OPT_LEVELS_2_PLUS, OPT_fipa_sra, NULL, 1 },
- { OPT_LEVELS_2_PLUS, OPT_falign_loops, NULL, 1 },
- { OPT_LEVELS_2_PLUS, OPT_falign_jumps, NULL, 1 },
- { OPT_LEVELS_2_PLUS, OPT_falign_labels, NULL, 1 },
- { OPT_LEVELS_2_PLUS, OPT_falign_functions, NULL, 1 },
+ { OPT_LEVELS_2_PLUS_SPEED_ONLY, OPT_falign_loops, NULL, 1 },
+ { OPT_LEVELS_2_PLUS_SPEED_ONLY, OPT_falign_jumps, NULL, 1 },
+ { OPT_LEVELS_2_PLUS_SPEED_ONLY, OPT_falign_labels, NULL, 1 },
+ { OPT_LEVELS_2_PLUS_SPEED_ONLY, OPT_falign_functions, NULL, 1 },
{ OPT_LEVELS_2_PLUS, OPT_ftree_tail_merge, NULL, 1 },
{ OPT_LEVELS_2_PLUS, OPT_fvect_cost_model_, NULL, VECT_COST_MODEL_CHEAP },
{ OPT_LEVELS_2_PLUS_SPEED_ONLY, OPT_foptimize_strlen, NULL, 1 },
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 3/4] Define MAX_CODE_ALIGN globally.
2018-07-16 8:53 [PATCH 0/4] Alignment option enhancements marxin
@ 2018-07-16 8:53 ` marxin
2018-07-16 8:53 ` [PATCH 2/4] Fix coding style of ASM_OUTPUT_ALIGN marxin
` (3 subsequent siblings)
4 siblings, 0 replies; 15+ messages in thread
From: marxin @ 2018-07-16 8:53 UTC (permalink / raw)
To: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 604 bytes --]
gcc/ChangeLog:
2018-07-11 Martin Liska <mliska@suse.cz>
* align.h (MAX_CODE_ALIGN): New.
(MAX_CODE_ALIGN_VALUE): New.
* common/config/i386/i386-common.c (ix86_handle_option):
(MAX_CODE_ALIGN): Moved to align.h.
* final.c (MAX_CODE_ALIGN): Likewise.
* opts.c (parse_and_check_align_values):
(MAX_CODE_ALIGN): Likewise.
(MAX_CODE_ALIGN_VALUE): Likewise.
---
gcc/align.h | 4 ++++
gcc/common/config/i386/i386-common.c | 3 ---
gcc/final.c | 1 -
gcc/opts.c | 4 ----
4 files changed, 4 insertions(+), 8 deletions(-)
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0003-Define-MAX_CODE_ALIGN-globally.patch --]
[-- Type: text/x-patch; name="0003-Define-MAX_CODE_ALIGN-globally.patch", Size: 1635 bytes --]
diff --git a/gcc/align.h b/gcc/align.h
index 5a5d6c752a3..9acce47cf9b 100644
--- a/gcc/align.h
+++ b/gcc/align.h
@@ -74,3 +74,7 @@ struct align_flags
align_flags_tuple levels[2];
};
+
+/* Define maximum supported code alignment. */
+#define MAX_CODE_ALIGN 16
+#define MAX_CODE_ALIGN_VALUE (1 << MAX_CODE_ALIGN)
diff --git a/gcc/common/config/i386/i386-common.c b/gcc/common/config/i386/i386-common.c
index 277ee55a093..70b3c3f2fc3 100644
--- a/gcc/common/config/i386/i386-common.c
+++ b/gcc/common/config/i386/i386-common.c
@@ -1318,9 +1318,6 @@ ix86_handle_option (struct gcc_options *opts,
return true;
- /* Comes from final.c -- no real reason to change it. */
-#define MAX_CODE_ALIGN 16
-
case OPT_malign_loops_:
warning_at (loc, 0, "-malign-loops is obsolete, use -falign-loops");
if (value > MAX_CODE_ALIGN)
diff --git a/gcc/final.c b/gcc/final.c
index 59eb75c3d63..445a3fe938a 100644
--- a/gcc/final.c
+++ b/gcc/final.c
@@ -842,7 +842,6 @@ shorten_branches (rtx_insn *first)
rtx_insn *insn;
int max_uid;
int i;
-#define MAX_CODE_ALIGN 16
rtx_insn *seq;
int something_changed = 1;
char *varying_length;
diff --git a/gcc/opts.c b/gcc/opts.c
index e536607fe79..0625b15b27b 100644
--- a/gcc/opts.c
+++ b/gcc/opts.c
@@ -1805,10 +1805,6 @@ parse_and_check_align_values (const char *flag,
return false;
}
- /* Comes from final.c -- no real reason to change it. */
-#define MAX_CODE_ALIGN 16
-#define MAX_CODE_ALIGN_VALUE (1 << MAX_CODE_ALIGN)
-
for (unsigned i = 0; i < result_values.length (); i++)
if (result_values[i] > MAX_CODE_ALIGN_VALUE)
{
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 0/4] Alignment option enhancements
2018-07-16 8:53 [PATCH 0/4] Alignment option enhancements marxin
` (3 preceding siblings ...)
2018-07-16 8:53 ` [PATCH 4/4] Do not enable OPT_falign_* for -Os marxin
@ 2018-07-16 14:10 ` Richard Biener
4 siblings, 0 replies; 15+ messages in thread
From: Richard Biener @ 2018-07-16 14:10 UTC (permalink / raw)
To: mliska; +Cc: GCC Patches
On Mon, Jul 16, 2018 at 10:54 AM marxin <mliska@suse.cz> wrote:
>
> Hi.
>
> As you probably noticed, current trunk accepts extended format
> of -falign-FOO options.
> However there are some limitations that are addressed in the patchset:
>
> 1) E.g. align_labels_max_skip and align_labels_log are separate
> values and targets can override them via LABEL_ALIGN and
> LABEL_ALIGN_MAX_SKIP. If you take a look at code in final.c:
>
> if (max_log < log)
> {
> max_log = log;
> max_skip = targetm.asm_out.label_align_max_skip (label);
> }
>
> It improves alignment, but max_skip can be zero and then we end up
> with no alignment. Thus I see it saner to provide just a single hook
> and return align_flags class instance.
>
> 2) In final.c we mix label, jump and loop alignments. I believe when
> combining them together, one should do maximum and not rely on:
>
> #ifdef ASM_OUTPUT_MAX_SKIP_ALIGN
> ASM_OUTPUT_MAX_SKIP_ALIGN (file, align, max_skip);
> /* Above, we don't know whether a label, jump or loop
> alignment was used. Conservatively apply
> label subalignment, not jump or loop
> subalignment (they are almost always larger). */
>
> That's implemented in align_flags::max function. When combining
> for instance -falign-labels=1000 and -falign-jumps=2000, I hope
> using alignment of 2000 is the right choice.
It's documented that way btw.
>
> 3) I removed various align_* defines and use directly
> align_{*}.
>
> 4) I ported visium target to provide always secondary alignment of 8B.
> That's the same what x86_64 does and was factored out in previous commits
> I did.
>
> 5) Globally MAX_CODE_ALIGN is introduced.
>
> 6) Various coding styles are fixes in target code.
>
> 7) OPT_falign_* is not set with -Os. Then --help=common -Q reports proper values.
>
> I tested that on x86_64-linux-gnu and ppc64le-linux. And also I tested all targets
> that I touched, these should provide equal results.
>
> Martin
>
> marxin (4):
> Clean up of new format of -falign-FOO.
> Fix coding style of ASM_OUTPUT_ALIGN.
> Define MAX_CODE_ALIGN globally.
> Do not enable OPT_falign_* for -Os.
>
> gcc/align.h | 80 ++++++++++++
> gcc/common/config/i386/i386-common.c | 3 -
> gcc/config/alpha/alpha.c | 5 +-
> gcc/config/i386/att.h | 2 +-
> gcc/config/i386/cygming.h | 2 +-
> gcc/config/i386/gas.h | 2 +-
> gcc/config/i386/i386.c | 5 +-
> gcc/config/i386/x86-64.h | 2 +-
> gcc/config/iq2000/iq2000.h | 2 +-
> gcc/config/m32r/m32r.h | 3 +-
> gcc/config/m68k/m68k.c | 10 +-
> gcc/config/nds32/nds32.h | 9 +-
> gcc/config/pa/pa.h | 2 +-
> gcc/config/powerpcspe/powerpcspe.c | 18 +--
> gcc/config/rs6000/rs6000-protos.h | 2 +-
> gcc/config/rs6000/rs6000.c | 18 +--
> gcc/config/rx/rx-protos.h | 2 +-
> gcc/config/rx/rx.c | 59 +++++----
> gcc/config/rx/rx.h | 6 +-
> gcc/config/s390/s390.c | 9 +-
> gcc/config/sh/sh.c | 19 +--
> gcc/config/sparc/sol2.h | 2 +-
> gcc/config/sparc/sparc.h | 2 +-
> gcc/config/spu/spu.c | 6 +-
> gcc/config/spu/spu.h | 2 +-
> gcc/config/visium/visium.c | 2 +-
> gcc/config/visium/visium.h | 17 +--
> gcc/coretypes.h | 1 +
> gcc/doc/tm.texi | 24 ----
> gcc/doc/tm.texi.in | 8 --
> gcc/final.c | 189 ++++++++-------------------
> gcc/flags.h | 38 +-----
> gcc/opts.c | 12 +-
> gcc/output.h | 6 +-
> gcc/system.h | 2 -
> gcc/target.def | 40 ------
> gcc/targhooks.h | 4 -
> gcc/toplev.c | 44 ++-----
> gcc/varasm.c | 14 +-
> 39 files changed, 262 insertions(+), 411 deletions(-)
> create mode 100644 gcc/align.h
>
> --
> 2.18.0
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/4] Clean up of new format of -falign-FOO.
2018-07-16 8:53 ` [PATCH 1/4] Clean up of new format of -falign-FOO marxin
@ 2018-07-16 19:52 ` Pat Haugen
2018-07-16 21:24 ` Jeff Law
2018-07-18 15:49 ` Martin Sebor
2 siblings, 0 replies; 15+ messages in thread
From: Pat Haugen @ 2018-07-16 19:52 UTC (permalink / raw)
To: marxin, gcc-patches
testsuite/gcc.target/powerpc/loop_align.c fails with this patch. It just needs a simple tweak to the scan-assembler line since we're no longer generating the ",,31" portion on the .p2align.
-Pat
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/4] Clean up of new format of -falign-FOO.
2018-07-16 8:53 ` [PATCH 1/4] Clean up of new format of -falign-FOO marxin
2018-07-16 19:52 ` Pat Haugen
@ 2018-07-16 21:24 ` Jeff Law
2018-07-17 9:52 ` Martin Liška
2018-07-18 15:49 ` Martin Sebor
2 siblings, 1 reply; 15+ messages in thread
From: Jeff Law @ 2018-07-16 21:24 UTC (permalink / raw)
To: marxin, gcc-patches
On 07/04/2018 04:23 AM, marxin wrote:
> gcc/ChangeLog:
>
> 2018-07-11 Martin Liska <mliska@suse.cz>
>
> * align.h: New file.
> * config/alpha/alpha.c (alpha_align_insns_1): Use align_functions directly.
> * config/i386/i386.c (ix86_avoid_jump_mispredicts): Use new return type
> align_flags of label_to_alignment.
> * config/m32r/m32r.h (LOOP_ALIGN): Wrap returned values into align_flags
> class.
> * config/m68k/m68k.c: Do not use removed align_labels_value and
> align_loops_value.
> * config/nds32/nds32.h (JUMP_ALIGN): Wrap result into align_flags class.
> (LOOP_ALIGN): Likewise.
> (LABEL_ALIGN): Likewise.
> * config/powerpcspe/powerpcspe.c (TARGET_ASM_LOOP_ALIGN_MAX_SKIP):
> Remove not used macro.
> (rs6000_loop_align): Change return type to align_flags.
> (rs6000_loop_align_max_skip): Remove.
> * config/rs6000/rs6000-protos.h (rs6000_loop_align):
> Change return type to align_flags.
> * config/rs6000/rs6000.c (TARGET_ASM_LOOP_ALIGN_MAX_SKIP):
> Remove not used macro.
> (rs6000_loop_align): Change return type to align_flags.
> (rs6000_loop_align_max_skip): Remove.
> * config/rx/rx.h (JUMP_ALIGN): Wrap integer values
> * config/rx/rx-protos.h (rx_align_for_label): Make it
> static function.
> * config/rx/rx.c (rx_align_for_label): Change return type
> to align_flags.
> (rx_max_skip_for_label): Remove TARGET_ASM_*_ALIGN_MAX_SKIP
> macro definitions.
> into align_flags class.
> (LABEL_ALIGN): Likewise.
> (LOOP_ALIGN): Likewise.
> * config/s390/s390.c (s390_label_align): Use align_flags
> class member.
> (s390_asm_output_function_label): Likewise.
> * config/sh/sh.c (sh_override_options_after_change):
> Use align_flags class directly without macros.
> (find_barrier): Likewise.
> (barrier_align): Likewise.
> (sh_loop_align): Likewise.
> * config/spu/spu.c (spu_option_override):
> Use align_flags_tuple::get_value instead of removed macros.
> (spu_sched_init): Likewise.
> * config/spu/spu.h (GTY): Likewise.
> * config/visium/visium.c (visium_option_override):
> Set "8" as default secondary alignment.
> * config/visium/visium.h (SUBALIGN_LOG): Define to 3
> in order to guarantee secondary alignment of 8.
> * coretypes.h: Include align.h header file.
> * doc/tm.texi: Remove TARGET_ASM_JUMP_ALIGN_MAX_SKIP,
> TARGET_ASM_LOOP_ALIGN_MAX_SKIP, TARGET_ASM_LABEL_ALIGN_MAX_SKIP
> and TARGET_ASM_LABEL_ALIGN_AFTER_BARRIER_MAX_SKIP macros.
> * doc/tm.texi.in: Likewise.
> * final.c (struct label_alignment): Remove not used structure.
> (LABEL_ALIGN): Change type to align_flags.
> (LOOP_ALIGN): Likewise.
> (JUMP_ALIGN): Likewise.
> (default_loop_align_max_skip): Remove.
> (default_label_align_max_skip): Likewise.
> (default_jump_align_max_skip): Likewise.
> (default_label_align_after_barrier_max_skip):
> (LABEL_TO_ALIGNMENT): Change to access label_align vector.
> (LABEL_TO_MAX_SKIP): Remove.
> (label_to_alignment): Return align_flags type instead of integer.
> (label_to_max_skip): Remove.
> (align_fuzz): Use align_flags type.
> (compute_alignments): Use align_flags type and use align_flags::max
> to combine multiple alignments.
> (grow_label_align): Grow vec instead of C array.
> (update_alignments): Assign just LABEL_TO_ALIGNMENT.
> (shorten_branches): Use align_flags type and use align_flags::max
> to combine multiple alignments.
> (final_scan_insn_1): Remove usage of secondary alignment that comes
> from label alignment, but instead use proper secondary alignment
> which is computed in grow_label_align.
> * flags.h (struct align_flags_tuple): Move to align.h.
> (struct align_flags): Likewise.
> (state_align_loops): Rename to align_loops.
> (state_align_jumps): Rename to align_jumps.
> (state_align_labels): Rename to align_labels.
> (state_align_functions): Rename to align_functions.
> (align_loops_log): Remove.
> (align_jumps_log): Remove.
> (align_labels_log): Remove.
> (align_functions_log): Remove.
> (align_loops_max_skip): Remove.
> (align_jumps_max_skip): Remove.
> (align_labels_max_skip): Remove.
> (align_functions_max_skip): Remove.
> (align_loops_value): Remove.
> (align_jumps_value): Remove.
> (align_labels_value): Remove.
> (align_functions_value): Remove.
> * output.h (label_to_alignment): Change return type to align_flags.
> (label_to_max_skip): Remove.
> * target.def: Remove loop_align_max_skip, label_align_max_skip,
> jump_align_max_skip macros.
> * targhooks.h (default_loop_align_max_skip): Remove.
> (default_label_align_max_skip): Likewise.
> (default_jump_align_max_skip): Likewise.
> (default_label_align_after_barrier_max_skip): Remove.
> * toplev.c (read_log_maxskip): Use ::normalize function.
> (parse_N_M): Remove not used argument and also call ::normalize.
> (parse_alignment_opts): Do not pass unused arguments.
> * varasm.c (assemble_start_function): Use directly align_functions
> instead of removed macros.
> * system.h: Do not poison removed macros.
All 4 patches in this kit are fine after you fix the minor issue Pat
reported with patch #1 on ppc.
Jeff
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/4] Clean up of new format of -falign-FOO.
2018-07-16 21:24 ` Jeff Law
@ 2018-07-17 9:52 ` Martin Liška
0 siblings, 0 replies; 15+ messages in thread
From: Martin Liška @ 2018-07-17 9:52 UTC (permalink / raw)
To: Jeff Law, gcc-patches
On 07/16/2018 11:24 PM, Jeff Law wrote:
> All 4 patches in this kit are fine after you fix the minor issue Pat
> reported with patch #1 on ppc.
>
> Jeff
Thank you Jeff for the trust. I fixed the minor issue and installed
the patch set.
Martin
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/4] Clean up of new format of -falign-FOO.
2018-07-16 8:53 ` [PATCH 1/4] Clean up of new format of -falign-FOO marxin
2018-07-16 19:52 ` Pat Haugen
2018-07-16 21:24 ` Jeff Law
@ 2018-07-18 15:49 ` Martin Sebor
2018-07-22 8:32 ` Gerald Pfeifer
2 siblings, 1 reply; 15+ messages in thread
From: Martin Sebor @ 2018-07-18 15:49 UTC (permalink / raw)
To: marxin, gcc-patches
On 07/04/2018 04:23 AM, marxin wrote:
>
> gcc/ChangeLog:
>
> 2018-07-11 Martin Liska <mliska@suse.cz>
>
> * align.h: New file.
Martin,
I'm seeing lots of warnings for this file:
/ssd/src/gcc/svn/gcc/align.h:53:32: warning: extended initializer lists
only available with -std=c++11 or -std=gnu++11
The code that triggers them is:
+struct align_flags
+{
+ /* Default constructor. */
+ align_flags (int log0 = 0, int maxskip0 = 0, int log1 = 0, int
maxskip1 = 0)
+ {
+ levels[0] = {log0, maxskip0};
+ levels[1] = {log1, maxskip1};
+ normalize ();
+ }
This form of assignment isn't valid in C++ 98.
Thanks
Martin
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/4] Clean up of new format of -falign-FOO.
2018-07-18 15:49 ` Martin Sebor
@ 2018-07-22 8:32 ` Gerald Pfeifer
2018-07-22 11:04 ` Gerald Pfeifer
0 siblings, 1 reply; 15+ messages in thread
From: Gerald Pfeifer @ 2018-07-22 8:32 UTC (permalink / raw)
To: Martin Sebor, Martin Liska; +Cc: gcc-patches
On Wed, 18 Jul 2018, Martin Sebor wrote:
> I'm seeing lots of warnings for this file:
>
> /ssd/src/gcc/svn/gcc/align.h:53:32: warning: extended initializer lists only
> available with -std=c++11 or -std=gnu++11
With clang version 3.4 (system compiler on FreeBSD 10.x) this is
even a hard error and GCC failed to build. So thanks for fixing
this, Martin!
Gerald
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/4] Clean up of new format of -falign-FOO.
2018-07-22 8:32 ` Gerald Pfeifer
@ 2018-07-22 11:04 ` Gerald Pfeifer
2018-07-22 12:19 ` Rainer Orth
0 siblings, 1 reply; 15+ messages in thread
From: Gerald Pfeifer @ 2018-07-22 11:04 UTC (permalink / raw)
To: Martin Sebor, Martin Liska; +Cc: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 1593 bytes --]
On Sun, 22 Jul 2018, Gerald Pfeifer wrote:
> With clang version 3.4 (system compiler on FreeBSD 10.x) this is
> even a hard error and GCC failed to build. So thanks for fixing
> this, Martin!
Unfortunately it appears there was another bootstrap failure hidden
behind that one:
In file included from /scratch/tmp/gerald/GCC-HEAD/gcc/system.h:691,
from /scratch/tmp/gerald/GCC-HEAD/gcc/tree-vect-slp.c:23:
/scratch/tmp/gerald/GCC-HEAD/gcc/tree-vect-slp.c: In function
Β‘_slp_tree* vect_build_slp_tree_2(vec_info*, vec<gimple*>, unsigned int,
poly_uint64*, vec<_slp_tree*>*, bool*, unsigned int*, unsigned int*,
unsigned int)Β’:
/scratch/tmp/gerald/GCC-HEAD/gcc/../include/libiberty.h:722:36: error: Β‘allocaΒ’ bound is unknown [-Werror=alloca-larger-than=]
# define alloca(x) __builtin_alloca(x)
~~~~~~~~~~~~~~~~^~~
/scratch/tmp/gerald/GCC-HEAD/gcc/../include/libiberty.h:356:33: note: in expansion of macro Β‘allocaΒ’
#define XALLOCAVEC(T, N) ((T *) alloca (sizeof (T) * (N)))
^~~~~~
/scratch/tmp/gerald/GCC-HEAD/gcc/tree-vect-slp.c:1437:16: note: in expansion of macro Β‘XALLOCAVECΒ’
bool *tem = XALLOCAVEC (bool, group_size);
^~~~~~~~~~
cc1plus: all warnings being treated as errors
gmake[3]: *** [Makefile:1112: tree-vect-slp.o] Error 1
gmake[3]: Leaving directory '/scratch/tmp/gerald/OBJ-0722-0939/gcc'
gmake[2]: *** [Makefile:4644: all-stage2-gcc] Error 2
This is on FreeBSD 10.4 which features clang 3.4.1 as system compiler;
FreeBSD 11.2 with clang 6.0.0 does not trigger that.
Gerald
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/4] Clean up of new format of -falign-FOO.
2018-07-22 11:04 ` Gerald Pfeifer
@ 2018-07-22 12:19 ` Rainer Orth
0 siblings, 0 replies; 15+ messages in thread
From: Rainer Orth @ 2018-07-22 12:19 UTC (permalink / raw)
To: Gerald Pfeifer; +Cc: Martin Sebor, Martin Liska, gcc-patches
Hi Gerald,
> On Sun, 22 Jul 2018, Gerald Pfeifer wrote:
>> With clang version 3.4 (system compiler on FreeBSD 10.x) this is
>> even a hard error and GCC failed to build. So thanks for fixing
>> this, Martin!
>
> Unfortunately it appears there was another bootstrap failure hidden
> behind that one:
>
>
> In file included from /scratch/tmp/gerald/GCC-HEAD/gcc/system.h:691,
> from /scratch/tmp/gerald/GCC-HEAD/gcc/tree-vect-slp.c:23:
> /scratch/tmp/gerald/GCC-HEAD/gcc/tree-vect-slp.c: In function
> ‘_slp_tree* vect_build_slp_tree_2(vec_info*, vec<gimple*>, unsigned int,
> poly_uint64*, vec<_slp_tree*>*, bool*, unsigned int*, unsigned int*,
> unsigned int)’:
> /scratch/tmp/gerald/GCC-HEAD/gcc/../include/libiberty.h:722:36: error: ‘alloca’ bound is unknown [-Werror=alloca-larger-than=]
> # define alloca(x) __builtin_alloca(x)
> ~~~~~~~~~~~~~~~~^~~
> /scratch/tmp/gerald/GCC-HEAD/gcc/../include/libiberty.h:356:33: note: in expansion of macro ‘alloca’
> #define XALLOCAVEC(T, N) ((T *) alloca (sizeof (T) * (N)))
> ^~~~~~
> /scratch/tmp/gerald/GCC-HEAD/gcc/tree-vect-slp.c:1437:16: note: in expansion of macro ‘XALLOCAVEC’
> bool *tem = XALLOCAVEC (bool, group_size);
> ^~~~~~~~~~
> cc1plus: all warnings being treated as errors
> gmake[3]: *** [Makefile:1112: tree-vect-slp.o] Error 1
> gmake[3]: Leaving directory '/scratch/tmp/gerald/OBJ-0722-0939/gcc'
> gmake[2]: *** [Makefile:4644: all-stage2-gcc] Error 2
>
>
> This is on FreeBSD 10.4 which features clang 3.4.1 as system compiler;
> FreeBSD 11.2 with clang 6.0.0 does not trigger that.
known issue: PR bootstrap/86621.
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/4] Clean up of new format of -falign-FOO.
2018-07-18 3:52 [PATCH 1/4] Clean up of new format of -falign-FOO Michael Collison
@ 2018-07-18 8:46 ` Martin Liška
0 siblings, 0 replies; 15+ messages in thread
From: Martin Liška @ 2018-07-18 8:46 UTC (permalink / raw)
To: Michael Collison, GCC Patches; +Cc: nd
On 07/18/2018 05:52 AM, Michael Collison wrote:
> Hi Martin,
>
> Your alignment patch breaks the arm port. In the file arm.c, function 'get_label_padding' the code uses:
>
> static HOST_WIDE_INT
> get_label_padding (rtx label)
> {
> HOST_WIDE_INT align, min_insn_size;
>
> align = 1 << label_to_alignment (label);
> min_insn_size = TARGET_THUMB ? 2 : 4;
> return align > min_insn_size ? align - min_insn_size : 0;
> }
>
> Which breaks with your current change. I think this needs to be modified to:
>
> 'align = 1 << label_to_alignment (label).levels[0].log'
Hello.
Sorry for the breakage, thank to Jeff it's fixed in this way.
r262848 should be fine.
Martin
>
> Regards,
>
> Michael Collison
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/4] Clean up of new format of -falign-FOO.
@ 2018-07-18 3:52 Michael Collison
2018-07-18 8:46 ` Martin Liška
0 siblings, 1 reply; 15+ messages in thread
From: Michael Collison @ 2018-07-18 3:52 UTC (permalink / raw)
To: mliska, GCC Patches; +Cc: nd
Hi Martin,
Your alignment patch breaks the arm port. In the file arm.c, function 'get_label_padding' the code uses:
static HOST_WIDE_INT
get_label_padding (rtx label)
{
HOST_WIDE_INT align, min_insn_size;
align = 1 << label_to_alignment (label);
min_insn_size = TARGET_THUMB ? 2 : 4;
return align > min_insn_size ? align - min_insn_size : 0;
}
Which breaks with your current change. I think this needs to be modified to:
'align = 1 << label_to_alignment (label).levels[0].log'
Regards,
Michael Collison
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2018-07-22 12:19 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-16 8:53 [PATCH 0/4] Alignment option enhancements marxin
2018-07-16 8:53 ` [PATCH 3/4] Define MAX_CODE_ALIGN globally marxin
2018-07-16 8:53 ` [PATCH 2/4] Fix coding style of ASM_OUTPUT_ALIGN marxin
2018-07-16 8:53 ` [PATCH 1/4] Clean up of new format of -falign-FOO marxin
2018-07-16 19:52 ` Pat Haugen
2018-07-16 21:24 ` Jeff Law
2018-07-17 9:52 ` Martin Liška
2018-07-18 15:49 ` Martin Sebor
2018-07-22 8:32 ` Gerald Pfeifer
2018-07-22 11:04 ` Gerald Pfeifer
2018-07-22 12:19 ` Rainer Orth
2018-07-16 8:53 ` [PATCH 4/4] Do not enable OPT_falign_* for -Os marxin
2018-07-16 14:10 ` [PATCH 0/4] Alignment option enhancements Richard Biener
2018-07-18 3:52 [PATCH 1/4] Clean up of new format of -falign-FOO Michael Collison
2018-07-18 8:46 ` Martin Liška
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).