Tcwg-commits July 2015

tcwg-commits@lists.linaro.org

2 participants
1355 discussions

[toolchain/abe] 04/08: Add some comments

by git

This is an automated email from the git hooks/post-receive script. bernie.ogden pushed a commit to branch bernie/fs in repository toolchain/abe. commit 637ead6b7afefaf8a519075ae77f77c5250e5bbd Author: Bernard Ogden <bernie.ogden(a)linaro.org> Date: Wed Jul 29 16:21:13 2015 +0200 Add some comments Change-Id: Ib141f404240910da8ebf9829a8680f8ee93c4dc1 --- config/bench/fs/makefs.sh | 3 +++ 1 file changed, 3 insertions(+) diff --git a/config/bench/fs/makefs.sh b/config/bench/fs/makefs.sh index 8e00c91..d8c4298 100755 --- a/config/bench/fs/makefs.sh +++ b/config/bench/fs/makefs.sh @@ -41,6 +41,8 @@ function derivevars { echo "${tarball} already exists" >&2 exit 1 fi + + #Assumes host machine is x86_64 if test x"${arch}" = "xx86-64"; then foreign= else @@ -71,6 +73,7 @@ function genrootfs { #This depends on the relevant kernel magic being in place to execute #arm/aarch64 code on qemu, and the relevant qemu being installed + #Debian: apt-get install binfmt-support qemu-user-static if test x"${foreign:-}" != x; then sudo /usr/sbin/debootstrap --second-stage --second-stage-target="${outdir}" fi -- To stop receiving notification emails like this one, please contact the administrator of this repository.

9 years, 3 months

[toolchain/abe] 02/08: Add check that HEAD maps to something meaningful upstream

by git

This is an automated email from the git hooks/post-receive script. bernie.ogden pushed a commit to branch bernie/fs in repository toolchain/abe. commit bff8b87ef05b62f223e26bbd965d040698cdd224 Author: Bernard Ogden <bernie.ogden(a)linaro.org> Date: Wed Jul 29 13:41:54 2015 +0200 Add check that HEAD maps to something meaningful upstream Change-Id: I2b82e9698e2e0fbc06990ca1d7608ecd3b0e93da --- config/bench/fs/makefs.sh | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/config/bench/fs/makefs.sh b/config/bench/fs/makefs.sh index 2c27503..2e87bc0 100755 --- a/config/bench/fs/makefs.sh +++ b/config/bench/fs/makefs.sh @@ -3,6 +3,16 @@ set -e set -u set -o pipefail +#Check that we are in a clean state w.r.t. source control +#Required to guarantee that the hash is valid +if git status -sb --porcelain | head -n1 | grep -q '\]$'; then + echo "Committed state out of sync with upstream" >&2 + exit 1 +elif test x"`git status -sb --porcelain | sed 1d`" != x; then + echo "Repository has local changes" >&2 + exit 1 +fi + #All of these variables MUST be defined and non-empty export stamp=20150607T041354Z export suite=jessie -- To stop receiving notification emails like this one, please contact the administrator of this repository.

9 years, 3 months

[toolchain/abe] branch bernie/fs updated (3e6103b -> 81667ca)

by git

This is an automated email from the git hooks/post-receive script. bernie.ogden pushed a change to branch bernie/fs in repository toolchain/abe. discards 3e6103b Add inotify-tools to the default package set discards 076a7aa Use full paths for tarballs discards 01a90fc Allow user to override 'local changes' check discards b9d5673 Allow selection of what to produce discards 9413ca9 Add some comments discards fdfdd76 Run second-stage for foreign chroots discards 8ed05f3 Add check that HEAD maps to something meaningful upstream discards b22f7a7 Introduce script to build filesystems omits 1a9237c Reduce target-session to minimal patches new f7327ee Introduce script to build filesystems new bff8b87 Add check that HEAD maps to something meaningful upstream new d8e109e Run second-stage for foreign chroots new 637ead6 Add some comments new e5372c3 Allow selection of what to produce new bb8b862 Allow user to override 'local changes' check new 38818f9 Use full paths for tarballs new 81667ca Add inotify-tools to the default package set This update added new revisions after undoing existing revisions. That is to say, some revisions that were in the old version of the branch are not in the new version. This situation occurs when a user --force pushes a change and generates a repository containing something like this: * -- * -- B -- O -- O -- O (3e6103b) \ N -- N -- N refs/heads/bernie/fs (81667ca) You should already have received notification emails for all of the O revisions, and so the following emails describe only the N revisions from the common base, B. Any revisions marked "omits" are not gone; other references still refer to them. Any revisions marked "discards" are gone forever. The 8 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "adds" were already present in the repository and have only been added to this reference. Summary of changes: config/bench/fs/makefs.sh | 2 +- config/bench/lava/target-session-kvm.yaml | 23 ----------------------- config/bench/lava/target-session.yaml | 5 +++++ 3 files changed, 6 insertions(+), 24 deletions(-) delete mode 100644 config/bench/lava/target-session-kvm.yaml -- To stop receiving notification emails like this one, please contact the administrator of this repository.

9 years, 3 months

[toolchain/abe] 03/08: Run second-stage for foreign chroots

by git

This is an automated email from the git hooks/post-receive script. bernie.ogden pushed a commit to branch bernie/fs in repository toolchain/abe. commit d8e109ebc58ae81c71388ef57f114fd24ff37f16 Author: Bernard Ogden <bernie.ogden(a)linaro.org> Date: Wed Jul 29 14:03:12 2015 +0200 Run second-stage for foreign chroots Change-Id: I4b00aae017377562a3d50d2ce4d17094eb93d8f0 --- config/bench/fs/makefs.sh | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/config/bench/fs/makefs.sh b/config/bench/fs/makefs.sh index 2e87bc0..8e00c91 100755 --- a/config/bench/fs/makefs.sh +++ b/config/bench/fs/makefs.sh @@ -68,6 +68,12 @@ function genrootfs { ${foreign:-} \ "${suite}" "${outdir}" \ http://snapshot.debian.org/archive/debian/${stamp} + + #This depends on the relevant kernel magic being in place to execute + #arm/aarch64 code on qemu, and the relevant qemu being installed + if test x"${foreign:-}" != x; then + sudo /usr/sbin/debootstrap --second-stage --second-stage-target="${outdir}" + fi } -- To stop receiving notification emails like this one, please contact the administrator of this repository.

9 years, 3 months

[toolchain/abe] 01/08: Introduce script to build filesystems

by git

This is an automated email from the git hooks/post-receive script. bernie.ogden pushed a commit to branch bernie/fs in repository toolchain/abe. commit f7327ee9c8da4a86718a86001f798c59b63f23d6 Author: Bernard Ogden <bernie.ogden(a)linaro.org> Date: Wed Jul 29 12:00:24 2015 +0200 Introduce script to build filesystems Change-Id: If013a3507396c219ff8a023bad8a68651391c468 --- config/bench/fs/makefs.sh | 101 ++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 101 insertions(+) diff --git a/config/bench/fs/makefs.sh b/config/bench/fs/makefs.sh new file mode 100755 index 0000000..2c27503 --- /dev/null +++ b/config/bench/fs/makefs.sh @@ -0,0 +1,101 @@ +#!/bin/bash +set -e +set -u +set -o pipefail + +#All of these variables MUST be defined and non-empty +export stamp=20150607T041354Z +export suite=jessie +export variant=minbase +export include=openssh-server,wget,make,cpufrequtils +export label=target +export hash="`git rev-parse HEAD`" +if test $? -ne 0; then + echo "Could not determine local rev" >&2 + exit 1 +fi + +export nosudo=${1:-} + +function derivevars { + echo ${arch:?Must give arch} > /dev/null + echo ${label:?Must give label} > /dev/null + extras="${include}${extras:+,${extras}}" + outdir="${label}-${suite}-${variant:+${variant}-}${arch}-${stamp}-${hash}" + tarball="${outdir}.tar.gz" + if test -e "${outdir}"; then + echo "${outdir} already exists" >&2 + exit 1 + fi + if test -e "${tarball}"; then + echo "${tarball} already exists" >&2 + exit 1 + fi + if test x"${arch}" = "xx86-64"; then + foreign= + else + foreign="--foreign --arch=${arch}" + fi +} + +function gentar { + /usr/sbin/debootstrap --make-tarball="${tarball}" \ + ${variant:+--variant="${variant}"} \ + --include="${extras}" \ + "${suite}" "${outdir}" \ + http://snapshot.debian.org/archive/debian/${stamp} +} + +function genrootfs { + if test x"${nosudo}" != x; then return 0; fi + + #This command sets a mirror just in case we need to get anything + #from the net - but, IIUC, it should not be used. + sudo /usr/sbin/debootstrap --unpack-tarball="${tarball}" \ + ${variant:+--variant="${variant}"} \ + --include="${extras}" \ + --keep-debootstrap-dir \ + ${foreign:-} \ + "${suite}" "${outdir}" \ + http://snapshot.debian.org/archive/debian/${stamp} +} + + +( #armhf - full target + arch=armhf + variant= + derivevars + gentar + genrootfs +) + +( #armhf - minimal target + arch=armhf + derivevars + gentar + genrootfs +) + +( #aarch64 - full target + arch=aarch64 + variant= + derivevars + gentar + genrootfs +) + +( #aarch64 - minimal target + arch=aarch64 + derivevars + gentar + genrootfs +) + +( #kvm host - many of the extras are abe dependencies, it will not configure witho [...] + arch=x86_64 + label=host + extras=autogen,binutils,bison,dejagnu,flex,gawk,gcc,gcc-multilib,g++-multilib,gi [...] + derivevars + gentar + genrootfs +) -- To stop receiving notification emails like this one, please contact the administrator of this repository.

9 years, 3 months

[gcc] branch trunk updated (18b3866 -> f58dd84)

by git

This is an automated email from the git hooks/post-receive script. unknown user pushed a change to branch trunk in repository gcc. from 18b3866 2015-07-30 Richard Biener <rguenther(a)suse.de> new f58dd84 * c-common.c (warn_tautological_cmp): Bail for float types. The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "adds" were already present in the repository and have only been added to this reference. Summary of changes: gcc/c-family/ChangeLog | 4 ++++ gcc/c-family/c-common.c | 6 ++++++ gcc/testsuite/ChangeLog | 4 ++++ gcc/testsuite/c-c++-common/Wtautological-compare-3.c | 20 ++++++++++++++++++++ 4 files changed, 34 insertions(+) create mode 100644 gcc/testsuite/c-c++-common/Wtautological-compare-3.c -- To stop receiving notification emails like this one, please contact the administrator of this repository.

9 years, 3 months

[gcc] branch trunk updated (c4699ff -> 18b3866)

by git

This is an automated email from the git hooks/post-receive script. unknown user pushed a change to branch trunk in repository gcc. from c4699ff 2015-07-30 Richard Biener <rguenther(a)suse.de> new 18b3866 2015-07-30 Richard Biener <rguenther(a)suse.de> The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "adds" were already present in the repository and have only been added to this reference. Summary of changes: gcc/ChangeLog | 16 ++++++++++ gcc/genmatch.c | 92 ++++++++++++++++++++++++++++++++++++++++++++++------------ gcc/match.pd | 5 ++-- 3 files changed, 92 insertions(+), 21 deletions(-) -- To stop receiving notification emails like this one, please contact the administrator of this repository.

9 years, 3 months

[gcc] branch trunk updated (081da05 -> c4699ff)

by git

This is an automated email from the git hooks/post-receive script. unknown user pushed a change to branch trunk in repository gcc. from 081da05 DR 1558 PR c++/67021 * pt.c (tsubst_decl) [TYPE_DECL]: Cle [...] new c4699ff 2015-07-30 Richard Biener <rguenther(a)suse.de> The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "adds" were already present in the repository and have only been added to this reference. Summary of changes: gcc/ChangeLog | 6 ++++++ gcc/match.pd | 2 +- 2 files changed, 7 insertions(+), 1 deletion(-) -- To stop receiving notification emails like this one, please contact the administrator of this repository.

9 years, 3 months

[gcc] 01/01: incremental fixes

by git

This is an automated email from the git hooks/post-receive script. unknown user pushed a commit to branch aoliva/pr64164 in repository gcc. commit 3004886e489b18e67ad97660b2de6ed14d1b011c Author: Alexandre Oliva <aoliva(a)redhat.com> Date: Wed Jul 29 14:34:34 2015 -0300 incremental fixes PR bootstrap/66978 * function.c (expand_function_start): Convert static chain to Pmode if needed. From H.J. Lu <hongjiu.lu(a)intel.com>. PR middle-end/66983 PR middle-end/67035 * cfgexpand.c (align_local_variable, add_stack_var): Support anonymous SSA names. (defer_stack_allocation): Likewise. Declare earlier. (expand_one_ssa_partition): Record alignment before expanding stack vars. Support deferred allocation. (set_rtl): Do no record deferred-allocation marker in SA.partition_to_pseudo. (expand_stack_vars): Adjust check for the marker in it. (adjust_one_expanded_partition_var): Skip deferred-alloc vars. PR middle-end/67034 * function.c (assign_parm_setup_reg): Copy BLKmode byref args to RTL assigned to their default partition. PR rtl-optimization/67000 * expr.c (read_complex_part): Export. * expr.h (read_complex_part): Declare. * function.c (split_complex_args): Use it. (assign_parms_unsplit_complex): Use the preexisting complex RTL if it matches the components. (assign_parm_setup_reg): Drop assert on from_expand mode. Adjust it to the promoted_mode. --- gcc/cfgexpand.c | 77 +++++++++++++++++++++++++++++++++++++++++---------------- gcc/expr.c | 2 +- gcc/expr.h | 1 + gcc/function.c | 63 ++++++++++++++++++++++++++++++++++------------ 4 files changed, 105 insertions(+), 38 deletions(-) diff --git a/gcc/cfgexpand.c b/gcc/cfgexpand.c index 0b19953..9577ad6d 100644 --- a/gcc/cfgexpand.c +++ b/gcc/cfgexpand.c @@ -97,6 +97,8 @@ gimple currently_expanding_gimple_stmt; static rtx expand_debug_expr (tree); +static bool defer_stack_allocation (tree, bool); + /* Return an expression tree corresponding to the RHS of GIMPLE statement STMT. */ @@ -253,7 +255,7 @@ set_rtl (tree t, rtx x) { if (SA.partition_to_pseudo[part]) gcc_assert (SA.partition_to_pseudo[part] == x); - else + else if (x != pc_rtx) SA.partition_to_pseudo[part] = x; } /* For the benefit of debug information at -O0 (where @@ -348,8 +350,15 @@ static bool has_short_buffer; static unsigned int align_local_variable (tree decl) { - unsigned int align = LOCAL_DECL_ALIGNMENT (decl); - DECL_ALIGN (decl) = align; + unsigned int align; + + if (TREE_CODE (decl) == SSA_NAME) + align = TYPE_ALIGN (TREE_TYPE (decl)); + else + { + align = LOCAL_DECL_ALIGNMENT (decl); + DECL_ALIGN (decl) = align; + } return align / BITS_PER_UNIT; } @@ -415,12 +424,15 @@ add_stack_var (tree decl) decl_to_stack_part->put (decl, stack_vars_num); v->decl = decl; - v->size = tree_to_uhwi (DECL_SIZE_UNIT (SSAVAR (decl))); + tree size = TREE_CODE (decl) == SSA_NAME + ? TYPE_SIZE_UNIT (TREE_TYPE (decl)) + : DECL_SIZE_UNIT (decl); + v->size = tree_to_uhwi (size); /* Ensure that all variables have size, so that &a != &b for any two variables that are simultaneously live. */ if (v->size == 0) v->size = 1; - v->alignb = align_local_variable (SSAVAR (decl)); + v->alignb = align_local_variable (decl); /* An alignment of zero can mightily confuse us later. */ gcc_assert (v->alignb != 0); @@ -1051,9 +1063,9 @@ expand_stack_vars (bool (*pred) (size_t), struct stack_vars_d [...] /* Skip variables that have already had rtl assigned. See also add_stack_var where we perpetrate this pc_rtx hack. */ decl = stack_vars[i].decl; - if ((TREE_CODE (decl) == SSA_NAME - ? SA.partition_to_pseudo[var_to_partition (SA.map, decl)] - : DECL_RTL (decl)) != pc_rtx) + if (TREE_CODE (decl) == SSA_NAME + ? SA.partition_to_pseudo[var_to_partition (SA.map, decl)] != NULL_RTX + : DECL_RTL (decl) != pc_rtx) continue; large_size += alignb - 1; @@ -1082,9 +1094,9 @@ expand_stack_vars (bool (*pred) (size_t), struct stack_vars_d [...] /* Skip variables that have already had rtl assigned. See also add_stack_var where we perpetrate this pc_rtx hack. */ decl = stack_vars[i].decl; - if ((TREE_CODE (decl) == SSA_NAME - ? SA.partition_to_pseudo[var_to_partition (SA.map, decl)] - : DECL_RTL (decl)) != pc_rtx) + if (TREE_CODE (decl) == SSA_NAME + ? SA.partition_to_pseudo[var_to_partition (SA.map, decl)] != NULL_RTX + : DECL_RTL (decl) != pc_rtx) continue; /* Check the predicate to see whether this variable should be @@ -1290,12 +1302,6 @@ expand_one_ssa_partition (tree var) if (SA.partition_to_pseudo[part]) return; - if (!use_register_for_decl (var)) - { - expand_one_stack_var_1 (var); - return; - } - unsigned int align = MINIMUM_ALIGNMENT (TREE_TYPE (var), TYPE_MODE (TREE_TYPE (var)), TYPE_ALIGN (TREE_TYPE (var))); @@ -1307,6 +1313,15 @@ expand_one_ssa_partition (tree var) record_alignment_for_reg_var (align); + if (!use_register_for_decl (var)) + { + if (defer_stack_allocation (var, true)) + add_stack_var (var); + else + expand_one_stack_var_1 (var); + return; + } + machine_mode reg_mode = promote_ssa_mode (var, NULL); rtx x = gen_reg_rtx (reg_mode); @@ -1331,6 +1346,13 @@ adjust_one_expanded_partition_var (tree var) rtx x = SA.partition_to_pseudo[part]; + if (!x) + { + /* This var will get a stack slot later. */ + gcc_assert (defer_stack_allocation (var, true)); + return; + } + set_rtl (var, x); if (!REG_P (x)) @@ -1409,10 +1431,14 @@ expand_one_error_var (tree var) static bool defer_stack_allocation (tree var, bool toplevel) { + tree size_unit = TREE_CODE (var) == SSA_NAME + ? TYPE_SIZE_UNIT (TREE_TYPE (var)) + : DECL_SIZE_UNIT (var); + /* Whether the variable is small enough for immediate allocation not to be a problem with regard to the frame size. */ bool smallish - = ((HOST_WIDE_INT) tree_to_uhwi (DECL_SIZE_UNIT (var)) + = ((HOST_WIDE_INT) tree_to_uhwi (size_unit) < PARAM_VALUE (PARAM_MIN_SIZE_FOR_STACK_SHARING)); /* If stack protection is enabled, *all* stack variables must be deferred, @@ -1421,16 +1447,24 @@ defer_stack_allocation (tree var, bool toplevel) if (flag_stack_protect || ((flag_sanitize & SANITIZE_ADDRESS) && ASAN_STACK)) return true; + unsigned int align = TREE_CODE (var) == SSA_NAME + ? TYPE_ALIGN (TREE_TYPE (var)) + : DECL_ALIGN (var); + /* We handle "large" alignment via dynamic allocation. We want to handle this extra complication in only one place, so defer them. */ - if (DECL_ALIGN (var) > MAX_SUPPORTED_STACK_ALIGNMENT) + if (align > MAX_SUPPORTED_STACK_ALIGNMENT) return true; + bool ignored = TREE_CODE (var) == SSA_NAME + ? !SSAVAR (var) || DECL_IGNORED_P (SSA_NAME_VAR (var)) + : DECL_IGNORED_P (var); + /* When optimization is enabled, DECL_IGNORED_P variables originally scoped might be detached from their block and appear at toplevel when we reach here. We want to coalesce them with variables from other blocks when the immediate contribution to the frame size would be noticeable. */ - if (toplevel && optimize > 0 && DECL_IGNORED_P (var) && !smallish) + if (toplevel && optimize > 0 && ignored && !smallish) return true; /* Variables declared in the outermost scope automatically conflict @@ -6135,7 +6169,8 @@ pass_expand::execute (function *fun) if (part == NO_PARTITION) continue; - gcc_assert (SA.partition_to_pseudo[part]); + gcc_assert (SA.partition_to_pseudo[part] + || defer_stack_allocation (name, true)); /* If this decl was marked as living in multiple places, reset this now to NULL. */ diff --git a/gcc/expr.c b/gcc/expr.c index d601129..fc49f92 100644 --- a/gcc/expr.c +++ b/gcc/expr.c @@ -3022,7 +3022,7 @@ write_complex_part (rtx cplx, rtx val, bool imag_p) /* Extract one of the components of the complex value CPLX. Extract the real part if IMAG_P is false, and the imaginary part if it's true. */ -static rtx +rtx read_complex_part (rtx cplx, bool imag_p) { machine_mode cmode, imode; diff --git a/gcc/expr.h b/gcc/expr.h index 32d1707..a2c8e1d 100644 --- a/gcc/expr.h +++ b/gcc/expr.h @@ -210,6 +210,7 @@ extern rtx_insn *emit_move_insn_1 (rtx, rtx); extern rtx_insn *emit_move_complex_push (machine_mode, rtx, rtx); extern rtx_insn *emit_move_complex_parts (rtx, rtx); +extern rtx read_complex_part (rtx, bool); extern void write_complex_part (rtx, rtx, bool); extern rtx emit_move_resolve_push (machine_mode, rtx); diff --git a/gcc/function.c b/gcc/function.c index c3d00cd..53bd42b 100644 --- a/gcc/function.c +++ b/gcc/function.c @@ -2322,11 +2322,10 @@ split_complex_args (struct assign_parm_data_all *all, vec<t [...] if (currently_expanding_to_rtl) { rtx rtl = rtl_for_parm (all, cparm); - gcc_assert (!rtl || GET_CODE (rtl) == CONCAT); if (rtl) { - SET_DECL_RTL (p, XEXP (rtl, 0)); - SET_DECL_RTL (decl, XEXP (rtl, 1)); + SET_DECL_RTL (p, read_complex_part (rtl, false)); + SET_DECL_RTL (decl, read_complex_part (rtl, true)); DECL_CONTEXT (p) = cparm; DECL_CONTEXT (decl) = cparm; @@ -3104,10 +3103,11 @@ assign_parm_setup_reg (struct assign_parm_data_all *all, tr [...] rtx from_expand = rtl_for_parm (all, parm); - if (from_expand && !data->passed_pointer) + if (from_expand) { parmreg = from_expand; - gcc_assert (GET_MODE (parmreg) == promoted_nominal_mode); + if (GET_MODE (parmreg) != promoted_nominal_mode) + parmreg = gen_lowpart (promoted_nominal_mode, parmreg); } else { @@ -3118,7 +3118,7 @@ assign_parm_setup_reg (struct assign_parm_data_all *all, tree parm, /* If this was an item that we received a pointer to, set DECL_RTL appropriately. */ - if (data->passed_pointer) + if (!from_expand && data->passed_pointer) { rtx x = gen_rtx_MEM (TYPE_MODE (TREE_TYPE (data->passed_type)), parmreg); set_mem_attributes (x, parm, 1); @@ -3139,7 +3139,8 @@ assign_parm_setup_reg (struct assign_parm_data_all *all, tree parm, need_conversion = (data->nominal_mode != data->passed_mode || promoted_nominal_mode != data->promoted_mode); - moved = false; + gcc_assert (!(need_conversion && data->passed_pointer)); + moved = from_expand && data->passed_pointer; if (need_conversion && GET_MODE_CLASS (data->nominal_mode) == MODE_INT @@ -3278,12 +3279,15 @@ assign_parm_setup_reg (struct assign_parm_data_all *all, tr [...] if (data->passed_pointer && (from_expand || TYPE_MODE (TREE_TYPE (parm)) != BLKmode)) { + rtx src = DECL_RTL (parm); + /* We can't use nominal_mode, because it will have been set to Pmode above. We must use the actual mode of the parm. */ if (from_expand) { - parmreg = from_expand; gcc_assert (GET_MODE (parmreg) == TYPE_MODE (TREE_TYPE (parm))); + src = gen_rtx_MEM (GET_MODE (parmreg), validated_mem); + set_mem_attributes (src, parm, 1); } else if (use_register_for_decl (parm)) { @@ -3302,14 +3306,14 @@ assign_parm_setup_reg (struct assign_parm_data_all *all, tr [...] set_mem_attributes (parmreg, parm, 1); } - if (GET_MODE (parmreg) != GET_MODE (DECL_RTL (parm))) + if (GET_MODE (parmreg) != GET_MODE (src)) { - rtx tempreg = gen_reg_rtx (GET_MODE (DECL_RTL (parm))); + rtx tempreg = gen_reg_rtx (GET_MODE (src)); int unsigned_p = TYPE_UNSIGNED (TREE_TYPE (parm)); push_to_sequence2 (all->first_conversion_insn, all->last_conversion_insn); - emit_move_insn (tempreg, DECL_RTL (parm)); + emit_move_insn (tempreg, src); tempreg = convert_to_mode (GET_MODE (parmreg), tempreg, unsigned_p); emit_move_insn (parmreg, tempreg); all->first_conversion_insn = get_insns (); @@ -3318,8 +3322,21 @@ assign_parm_setup_reg (struct assign_parm_data_all *all, tree parm, did_conversion = true; } + else if (GET_MODE (parmreg) == BLKmode) + { + push_to_sequence2 (all->first_conversion_insn, + all->last_conversion_insn); + gcc_assert (TREE_CODE (data->passed_type) == POINTER_TYPE); + gcc_assert (TREE_TYPE (data->passed_type) == TREE_TYPE (parm)); + emit_block_move (parmreg, src, + GEN_INT (int_size_in_bytes (TREE_TYPE (parm))), + BLOCK_OP_NORMAL);; + all->first_conversion_insn = get_insns (); + all->last_conversion_insn = get_last_insn (); + end_sequence (); + } else - emit_move_insn (parmreg, DECL_RTL (parm)); + emit_move_insn (parmreg, src); SET_DECL_RTL (parm, parmreg); @@ -3495,11 +3512,21 @@ assign_parms_unsplit_complex (struct assign_parm_data_all *all, imag = DECL_RTL (fnargs[i + 1]); if (inner != GET_MODE (real)) { - real = gen_lowpart_SUBREG (inner, real); - imag = gen_lowpart_SUBREG (inner, imag); + real = simplify_gen_subreg (inner, real, GET_MODE (real), + subreg_lowpart_offset + (inner, GET_MODE (real))); + imag = simplify_gen_subreg (inner, imag, GET_MODE (imag), + subreg_lowpart_offset + (inner, GET_MODE (imag))); } - if (TREE_ADDRESSABLE (parm)) + if ((tmp = DECL_RTL_IF_SET (parm)) + && rtx_equal_p (real, + read_complex_part (tmp, false)) + && rtx_equal_p (imag, + read_complex_part (tmp, true))) + ; /* We have the right rtl in parm and tmp already. */ + else if (TREE_ADDRESSABLE (parm)) { rtx rmem, imem; HOST_WIDE_INT size = int_size_in_bytes (TREE_TYPE (parm)); @@ -3645,7 +3672,7 @@ assign_bounds (vec<bounds_parm_data> &bndargs, assign_parm_setup_block (&all, pbdata->bounds_parm, &pbdata->parm_data); else if (pbdata->parm_data.passed_pointer - || use_register_for_decl (pbdata->bounds_parm)) + || use_register_for_parm_decl (&all, pbdata->bounds_parm)) assign_parm_setup_reg (&all, pbdata->bounds_parm, &pbdata->parm_data); else @@ -5207,6 +5234,10 @@ expand_function_start (tree subr) SET_DECL_RTL (parm, local); mark_reg_pointer (local, TYPE_ALIGN (TREE_TYPE (TREE_TYPE (parm)))); + if (GET_MODE (local) != Pmode) + local = convert_to_mode (Pmode, local, + TYPE_UNSIGNED (TREE_TYPE (parm))); + insn = emit_move_insn (local, chain); /* Mark the register as eliminable, similar to parameters. */ -- To stop receiving notification emails like this one, please contact the administrator of this repository.

9 years, 3 months

[gcc] branch aoliva/pr64164 updated (d1e62a5 -> 3004886)

by git

This is an automated email from the git hooks/post-receive script. unknown user pushed a change to branch aoliva/pr64164 in repository gcc. discards d1e62a5 incremental fixes new 3004886 incremental fixes This update added new revisions after undoing existing revisions. That is to say, some revisions that were in the old version of the branch are not in the new version. This situation occurs when a user --force pushes a change and generates a repository containing something like this: * -- * -- B -- O -- O -- O (d1e62a5) \ N -- N -- N refs/heads/aoliva/pr64164 (3004886) You should already have received notification emails for all of the O revisions, and so the following emails describe only the N revisions from the common base, B. Any revisions marked "omits" are not gone; other references still refer to them. Any revisions marked "discards" are gone forever. The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "adds" were already present in the repository and have only been added to this reference. Summary of changes: gcc/expr.c | 2 +- gcc/expr.h | 1 + gcc/function.c | 59 ++++++++++++++++++++++++++++++++++++++++++---------------- 3 files changed, 45 insertions(+), 17 deletions(-) -- To stop receiving notification emails like this one, please contact the administrator of this repository.

9 years, 3 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

Tcwg-commits July 2015