From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31408 invoked by alias); 24 Aug 2017 21:37:28 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 31393 invoked by uid 89); 24 Aug 2017 21:37:27 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS autolearn=ham version=3.3.2 spammy= X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 24 Aug 2017 21:37:25 +0000 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id B5AC14DD7C; Thu, 24 Aug 2017 21:37:24 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com B5AC14DD7C Authentication-Results: ext-mx09.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx09.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=law@redhat.com Received: from localhost.localdomain (ovpn-116-59.phx2.redhat.com [10.3.116.59]) by smtp.corp.redhat.com (Postfix) with ESMTP id 756F2890BC; Thu, 24 Aug 2017 21:37:24 +0000 (UTC) Subject: Re: [62/77] Big machine_mode to scalar_int_mode replacement To: gcc-patches@gcc.gnu.org, richard.sandiford@linaro.org References: <8760ewohsv.fsf@linaro.org> <87pod4d83g.fsf@linaro.org> From: Jeff Law Message-ID: <3bd7b340-7709-ebe4-4c2b-29ff0d649ba3@redhat.com> Date: Thu, 24 Aug 2017 21:57:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <87pod4d83g.fsf@linaro.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2017-08/txt/msg01474.txt.bz2 On 07/13/2017 03:00 AM, Richard Sandiford wrote: > This patch changes the types of various things from machine_mode > to scalar_int_mode, in cases where (after previous patches) > simply changing the type is enough on its own. The patch does > nothing other than that. > > 2017-07-13 Richard Sandiford > Alan Hayward > David Sherwood > > gcc/ > * builtins.h (builtin_strncpy_read_str): Take a scalar_int_mode > instead of a machine_mode. > (builtin_memset_read_str): Likewise. > * builtins.c (c_readstr): Likewise. > (builtin_memcpy_read_str): Likewise. > (builtin_strncpy_read_str): Likewise. > (builtin_memset_read_str): Likewise. > (builtin_memset_gen_str): Likewise. > (expand_builtin_signbit): Use scalar_int_mode for local variables. > * cfgexpand.c (convert_debug_memory_address): Take a scalar_int_mode > instead of a machine_mode. > * combine.c (simplify_if_then_else): Use scalar_int_mode for local > variables. > (make_extraction): Likewise. > (try_widen_shift_mode): Take and return scalar_int_modes instead > of machine_modes. > * config/aarch64/aarch64.c (aarch64_libgcc_cmp_return_mode): Return > a scalar_int_mode instead of a machine_mode. > * config/avr/avr.c (avr_addr_space_address_mode): Likewise. > (avr_addr_space_pointer_mode): Likewise. > * config/cr16/cr16.c (cr16_unwind_word_mode): Likewise. > * config/msp430/msp430.c (msp430_addr_space_pointer_mode): Likewise. > (msp430_unwind_word_mode): Likewise. > * config/spu/spu.c (spu_unwind_word_mode): Likewise. > (spu_addr_space_pointer_mode): Likewise. > (spu_addr_space_address_mode): Likewise. > (spu_libgcc_cmp_return_mode): Likewise. > (spu_libgcc_shift_count_mode): Likewise. > * config/rl78/rl78.c (rl78_addr_space_address_mode): Likewise. > (rl78_addr_space_pointer_mode): Likewise. > (fl78_unwind_word_mode): Likewise. > (rl78_valid_pointer_mode): Take a scalar_int_mode instead of a > machine_mode. > * config/alpha/alpha.c (vms_valid_pointer_mode): Likewise. > * config/ia64/ia64.c (ia64_vms_valid_pointer_mode): Likewise. > * config/mips/mips.c (mips_mode_rep_extended): Likewise. > (mips_valid_pointer_mode): Likewise. > * config/tilegx/tilegx.c (tilegx_mode_rep_extended): Likewise. > * config/ft32/ft32.c (ft32_valid_pointer_mode): Likewise. > (ft32_addr_space_pointer_mode): Return a scalar_int_mode instead > of a machine_mode. > (ft32_addr_space_address_mode): Likewise. > * config/m32c/m32c.c (m32c_valid_pointer_mode): Take a > scalar_int_mode instead of a machine_mode. > (m32c_addr_space_pointer_mode): Return a scalar_int_mode instead > of a machine_mode. > (m32c_addr_space_address_mode): Likewise. > * config/powerpcspe/powerpcspe.c (rs6000_abi_word_mode): Likewise. > (rs6000_eh_return_filter_mode): Likewise. > * config/rs6000/rs6000.c (rs6000_abi_word_mode): Likewise. > (rs6000_eh_return_filter_mode): Likewise. > * config/s390/s390.c (s390_libgcc_cmp_return_mode): Likewise. > (s390_libgcc_shift_count_mode): Likewise. > (s390_unwind_word_mode): Likewise. > (s390_valid_pointer_mode): Take a scalar_int_mode rather than a > machine_mode. > * target.def (mode_rep_extended): Likewise. > (valid_pointer_mode): Likewise. > (addr_space.valid_pointer_mode): Likewise. > (eh_return_filter_mode): Return a scalar_int_mode rather than > a machine_mode. > (libgcc_cmp_return_mode): Likewise. > (libgcc_shift_count_mode): Likewise. > (unwind_word_mode): Likewise. > (addr_space.pointer_mode): Likewise. > (addr_space.address_mode): Likewise. > * doc/tm.texi: Regenerate. > * dojump.c (prefer_and_bit_test): Take a scalar_int_mode rather than > a machine_mode. > (do_jump): Use scalar_int_mode for local variables. > * dwarf2cfi.c (init_return_column_size): Take a scalar_int_mode > rather than a machine_mode. > * dwarf2out.c (convert_descriptor_to_mode): Likewise. > (scompare_loc_descriptor_wide): Likewise. > (scompare_loc_descriptor_narrow): Likewise. > * emit-rtl.c (adjust_address_1): Use scalar_int_mode for local > variables. > * except.c (sjlj_emit_dispatch_table): Likewise. > (expand_builtin_eh_copy_values): Likewise. > * explow.c (convert_memory_address_addr_space_1): Likewise. > Take a scalar_int_mode rather than a machine_mode. > (convert_memory_address_addr_space): Take a scalar_int_mode rather > than a machine_mode. > (memory_address_addr_space): Use scalar_int_mode for local variables. > * expmed.h (expand_mult_highpart_adjust): Take a scalar_int_mode > rather than a machine_mode. > * expmed.c (mask_rtx): Likewise. > (init_expmed_one_conv): Likewise. > (expand_mult_highpart_adjust): Likewise. > (extract_high_half): Likewise. > (expmed_mult_highpart_optab): Likewise. > (expmed_mult_highpart): Likewise. > (expand_smod_pow2): Likewise. > (expand_sdiv_pow2): Likewise. > (emit_store_flag_int): Likewise. > (adjust_bit_field_mem_for_reg): Use scalar_int_mode for local > variables. > (extract_low_bits): Likewise. > * expr.h (by_pieces_constfn): Take a scalar_int_mode rather than > a machine_mode. > * expr.c (pieces_addr::adjust): Likewise. > (can_store_by_pieces): Likewise. > (store_by_pieces): Likewise. > (clear_by_pieces_1): Likewise. > (expand_expr_addr_expr_1): Likewise. > (expand_expr_addr_expr): Use scalar_int_mode for local variables. > (expand_expr_real_1): Likewise. > (try_casesi): Likewise. > * final.c (shorten_branches): Likewise. > * fold-const.c (fold_convert_const_int_from_fixed): Change the > type of "mode" to machine_mode. > * internal-fn.c (expand_arith_overflow_result_store): Take a > scalar_int_mode rather than a machine_mode. > (expand_mul_overflow): Use scalar_int_mode for local variables. > * loop-doloop.c (doloop_modify): Likewise. > (doloop_optimize): Likewise. > * optabs.c (expand_subword_shift): Take a scalar_int_mode rather > than a machine_mode. > (expand_doubleword_shift_condmove): Likewise. > (expand_doubleword_shift): Likewise. > (expand_doubleword_clz): Likewise. > (expand_doubleword_popcount): Likewise. > (expand_doubleword_parity): Likewise. > (expand_absneg_bit): Use scalar_int_mode for local variables. > (prepare_float_lib_cmp): Likewise. > * rtl.h (convert_memory_address_addr_space_1): Take a scalar_int_mode > rather than a machine_mode. > (convert_memory_address_addr_space): Likewise. > (get_mode_bounds): Likewise. > (get_address_mode): Return a scalar_int_mode rather than a > machine_mode. > * rtlanal.c (get_address_mode): Likewise. > * stor-layout.c (get_mode_bounds): Take a scalar_int_mode rather > than a machine_mode. > * targhooks.c (default_mode_rep_extended): Likewise. > (default_valid_pointer_mode): Likewise. > (default_addr_space_valid_pointer_mode): Likewise. > (default_eh_return_filter_mode): Return a scalar_int_mode rather > than a machine_mode. > (default_libgcc_cmp_return_mode): Likewise. > (default_libgcc_shift_count_mode): Likewise. > (default_unwind_word_mode): Likewise. > (default_addr_space_pointer_mode): Likewise. > (default_addr_space_address_mode): Likewise. > * targhooks.h (default_eh_return_filter_mode): Likewise. > (default_libgcc_cmp_return_mode): Likewise. > (default_libgcc_shift_count_mode): Likewise. > (default_unwind_word_mode): Likewise. > (default_addr_space_pointer_mode): Likewise. > (default_addr_space_address_mode): Likewise. > (default_mode_rep_extended): Take a scalar_int_mode rather than > a machine_mode. > (default_valid_pointer_mode): Likewise. > (default_addr_space_valid_pointer_mode): Likewise. > * tree-ssa-address.c (addr_for_mem_ref): Use scalar_int_mode for > local variables. > * tree-ssa-loop-ivopts.c (get_shiftadd_cost): Take a scalar_int_mode > rather than a machine_mode. > * tree-switch-conversion.c (array_value_type): Use scalar_int_mode > for local variables. > * tree-vrp.c (simplify_float_conversion_using_ranges): Likewise. > * var-tracking.c (use_narrower_mode): Take a scalar_int_mode rather > than a machine_mode. > Going to ACK as obvious. Jeff