This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [01/77] Add an E_ prefix to mode names


On 07/13/2017 02:36 AM, Richard Sandiford wrote:
> Later patches will add wrapper types for specific classes
> of mode.  E.g. SImode will be a scalar_int_mode, SFmode will be a
> scalar_float_mode, etc.  This patch prepares for that change by adding
> an E_ prefix to the mode enum values.  It also adds #defines that map
> the unprefixed names to the prefixed names; e.g:
> 
>   #define QImode E_QImode
> 
> Later patches will change this to use things like scalar_int_mode
> where appropriate.
> 
> The patch continues to use enum values to initialise static data.
> This isn't necessary for correctness, but it cuts down on the amount
> of load-time initialisation and shouldn't have any downsides.
> 
> The patch also changes things like:
> 
>   cmp_mode == DImode ? DFmode : DImode
> 
> to:
> 
>   cmp_mode == DImode ? E_DFmode : E_DImode
> 
> This is because DImode and DFmode will eventually be different
> classes, so the original ?: wouldn't be well-formed.
> 
> 2017-07-13  Richard Sandiford  <richard.sandiford@linaro.org>
> 	    Alan Hayward  <alan.hayward@arm.com>
> 	    David Sherwood  <david.sherwood@arm.com>
> 
> gcc/
> 	* genmodes.c (mode_size_inline): Add an E_ prefix to mode names.
> 	(mode_nunits_inline): Likewise.
> 	(mode_inner_inline): Likewise.
> 	(mode_unit_size_inline): Likewise.
> 	(mode_unit_precision_inline): Likewise.
> 	(emit_insn_modes_h): Likewise.  Also emit a #define of the
> 	unprefixed name.
> 	(emit_mode_wider): Add an E_ prefix to mode names.
> 	(emit_mode_complex): Likewise.
> 	(emit_mode_inner): Likewise.
> 	(emit_mode_adjustments): Likewise.
> 	(emit_mode_int_n): Likewise.
> 	* config/aarch64/aarch64-builtins.c (v8qi_UP, v4hi_UP, v4hf_UP)
> 	(v2si_UP, v2sf_UP, v1df_UP, di_UP, df_UP, v16qi_UP, v8hi_UP, v8hf_UP)
> 	(v4si_UP, v4sf_UP, v2di_UP, v2df_UP, ti_UP, oi_UP, ci_UP, xi_UP)
> 	(si_UP, sf_UP, hi_UP, hf_UP, qi_UP): Likewise.
> 	(CRC32_BUILTIN, ENTRY): Likewise.
> 	* config/aarch64/aarch64.c (aarch64_push_regs): Likewise.
> 	(aarch64_pop_regs): Likewise.
> 	(aarch64_process_components): Likewise.
> 	* config/alpha/alpha.c (alpha_emit_conditional_move): Likewise.
> 	* config/arm/arm-builtins.c (v8qi_UP, v4hi_UP, v4hf_UP, v2si_UP)
> 	(v2sf_UP, di_UP, v16qi_UP, v8hi_UP, v8hf_UP, v4si_UP, v4sf_UP)
> 	(v2di_UP, ti_UP, ei_UP, oi_UP, hf_UP, si_UP, void_UP): Likewise.
> 	* config/arm/arm.c (arm_init_libfuncs): Likewise.
> 	* config/i386/i386-builtin-types.awk (ix86_builtin_type_vect_mode):
> 	Likewise.
> 	* config/i386/i386-builtin.def (pcmpestr): Likewise.
> 	(pcmpistr): Likewise.
> 	* config/microblaze/microblaze.c (double_memory_operand): Likewise.
> 	* config/mmix/mmix.c (mmix_output_condition): Likewise.
> 	* config/powerpcspe/powerpcspe.c (rs6000_init_hard_regno_mode_ok):
> 	Likewise.
> 	* config/rl78/rl78.c (mduc_regs): Likewise.
> 	* config/rs6000/rs6000.c (rs6000_init_hard_regno_mode_ok): Likewise.
> 	(htm_expand_builtin): Likewise.
> 	* config/sh/sh.h (REGISTER_NATURAL_MODE): Likewise.
> 	* config/sparc/sparc.c (emit_save_or_restore_regs): Likewise.
> 	* config/xtensa/xtensa.c (print_operand): Likewise.
> 	* expmed.h (NUM_MODE_PARTIAL_INT): Likewise.
> 	(NUM_MODE_VECTOR_INT): Likewise.
> 	* genoutput.c (null_operand): Likewise.
> 	(output_operand_data): Likewise.
> 	* genrecog.c (print_parameter_value): Likewise.
> 	* lra.c (debug_operand_data): Likewise.
OK.  As are any natural tweaks to deal with changes that have occurred
on the trunk since this was submitted.

Jeff


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]