Re: Please accept this commit for the trunk

On Feb 5, 2018, at 8:42 AM, Douglas Mencken wrote: > > I’m about > > “ [PATCH 2/4] [Darwin,PPC] Remove uses of LR in > restore_world ” https://gcc.gnu.org/bugzilla/attachment.cgi?id=42304 > > look at bug #84113 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84113 for > more info > > “ One import

Please accept this commit for the trunk

I’m about “ [PATCH 2/4] [Darwin,PPC] Remove uses of LR in restore_world ” https://gcc.gnu.org/bugzilla/attachment.cgi?id=42304 look at bug #84113 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84113 for more info “ One important question ’s yet: Why this patch has been ignored despite it’s been ma

[spu, commit] Fix PR target/82960

Hello, the ICE with --enable-checking=rtl reported in PR 82960 was caused by spu.c:pad_bb using INSN_CODE on RTX with INSN_P false (specifically, on jump_table_data). Add checks to handle this case. Tested on spu-elf, committed to mainline. Bye, Ulrich ChangeLog: PR target/82960

Re: [PATCH, rs6000 V4] Fixes for commit 254464

Hi Carl, On Thu, Nov 09, 2017 at 08:39:10AM -0800, Carl Love wrote: > The recent commit 254464 to add Power 8 support for the vec_revb() > builtin had a couple of issues that I missed on my testing of the > original patch. First, on Power 8 BE the le_swap1 permute vector value >

[PATCH, rs6000 V4] Fixes for commit 254464

GCC Maintainers: The recent commit 254464 to add Power 8 support for the vec_revb() builtin had a couple of issues that I missed on my testing of the original patch. First, on Power 8 BE the le_swap1 permute vector value was wrong in function swap_endian_selector_for_mode(). This issue caused

[gcc commit] [gcc patch] DWARF-5: Define DW_IDX_GNU_static and DW_IDX_GNU_external

On Sun, 02 Jul 2017 18:22:45 +0200, Jason Merrill wrote: > I'd suggest "internal" rather than "static". Otherwise the patch looks good. Checked in as r249883 with: DW_IDX_GNU_internal, DW_IDX_GNU_external Jan

Re: [PATCH rs6000] Fix for commit 249311

On Fri, Jun 16, 2017 at 09:08:50PM -0700, Carl Love wrote: > Commit r249311 had an error. During the patch review the define expand > for VFC_inst was changed to VF_sxddp. I compiled and tested the source > after making the change and it seemed fine. However, I missed a couple > of

[PATCH rs6000] Fix for commit 249311

GCC maintainers: Commit r249311 had an error. During the patch review the define expand for VFC_inst was changed to VF_sxddp. I compiled and tested the source after making the change and it seemed fine. However, I missed a couple of changes. It seems that since I didn't remove al

Re: [PATCH, Fortran, pr70696/68887, gcc-6, v1] Forgot to commit runtime part

Hi all, during the backport of pr70696 I forgot to backport the runtime part in libgfortran/caf/single.c. This is fixed by commit r245016. Sorry for the noise, regards, Andre On Sun, 29 Jan 2017 14:51:03 +0100 Andre Vehreschild wrote: > Hi all, > > just applied the backport

Re: [C++ PATCH] P0490R0 GB 20: decomposition declaration should commit to tuple interpretation early (take 2)

an option), > bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > 2016-12-15 Jakub Jelinek > > P0490R0 GB 20: decomposition declaration should commit to tuple > interpretation early > * decl.c (get_tuple_size): Make static. If in

[C++ PATCH] P0490R0 GB 20: decomposition declaration should commit to tuple interpretation early (take 2)

> 'something goes horribly wrong' Ok, here is the updated patch (and also moving the diagnostics to get_tuple_size caller as written in the other mail as an option), bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2016-12-15 Jakub Jelinek P0490R0 G

[C++ PATCH] P0490R0 GB 20: decomposition declaration should commit to tuple interpretation early

Hi! Issaquah papers have been recently published and there are two decomp changes. One I've sent a partially working patch privately and am lost with, the other one is that whenever std::tuple_size is a complete type, we should commit to the std::tuple* way - so if it is missing value s

[PATCH] simplify-rtx: Add missing line for previous commit (PR78583)

ed on powerpc64-linux, committing to trunk as obvious. Segher 2016-11-30 Segher Boessenkool PR rtl-optimization/78583 * simplify-rtx.c (simplify_truncation): Add check missing from the previous commit. --- gcc/simplify-rtx.c | 1 + 1 file changed, 1 insertion(+)

Re: [PATCH] Revert commit r235318

On 2016-10-08, at 12:53 PM, John David Anglin wrote: > > It doesn't look as if the change was reviewed. For reference, the submission is here: https://gcc.gnu.org/ml/gcc-patches/2016-04/msg00796.html Dave -- John David Anglin dave.ang...@bell.net

[PATCH] Revert commit r235318

The attached patch reverts commit r235318. It caused an ICE in binds_to_current_def_p building java. Although java is now gone, the change broke bootstrap and the handling of _GLOBAL constructors on 32-bit hpux targets. See PR 70795: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70795 In as

Re: [Committed] PR fortran/77420 -- revert previous commit

On Tue, Sep 13, 2016 at 10:05:56AM -0700, Steve Kargl wrote: > --- gcc/fortran/module.c (revision 240117) > +++ gcc/fortran/module.c (working copy) > @@ -4647,7 +4647,7 @@ load_equiv (void) >} > > /* Check for duplicate equivalences being loaded from different modules > */

[Committed] PR fortran/77420 -- revert previous commit

I've reverted my previous fix for PR fortran/77420. It was causing a regression that could, in principle, effect a significant nubmer of gfortran users. 2016-09-13 Steven G. Kargl PR fortran/77420 * module.c (load_equiv): Revert revision 240063. 2016-09-13 Steven G. Kargl

Commit: M32R: Build crtinit.o and crtfini.o

Hi Guys, I am applying the patch below to fix a long standing snafu for the m32r target where the files crtinit.o and crtfini.o were not being built along with the rest of libgcc. Tested with no regressions and a lot of test case fixes using an m32r-elf toolchain. Cheers Nick libgcc

Commit: MSP430: Rename entries in option enums

Hi Guys, I recently noticed that the MSP430 backend uses some pretty generic names for the enum values of its hardware multiply and memory region options. This could possibly cause problems if these names are used elsewhere, so I have decided to check in the patch below to fix this. Te

Commit: MSP430: Three small fixes

Hi Guys, I am checking in the patch below to fix three small problems with the MSP430 backend: Firstly interrupt handlers for the MSP430 cannot be static. Static interrupt handlers can be optimized away since no control flow path can be found to use them. Secondly there no longer is

Commit: MSP430: Optimize 1 bit shifts

Hi Guys, I am applying this patch, developed by DJ, to improve the code generated for the MSP430 when performing a shift by a single bit. Normally a helper function is used to perform N-bit shifts, but for one bit we can save time, and not use up any more space, by performing the shift i

Commit: MSP430: Update prototypes in libgcc

Hi Guys, I am applying the following patch to update the prototypes in the MSP430 specific part of libgcc. It adds missing prototypes for exported ABI functions, and it changes the prototypes for the arithmetic shift functions so that they explicitly take a signed char parameter. Cheer

libgo patch committed: Commit final version of pkg-config support

to mainline. I will commit this to the GCC 6 branch when that is open for bug fixes. Ian Index: gcc/go/gofrontend/MERGE === --- gcc/go/gofrontend/MERGE (revision 234958) +++ gcc/go/gofrontend/MERGE (working copy) @@ -1,4 +1,4

Commit: ARM: Extend fix for PR 62254

Hi Guys, I am applying this patch as a further fix for PR 62254. In the long run we will hopefully be dropping support for ARM v3 (and earlier) so this is more in the nature of a plaster than a real fix. Cheers Nick gcc/ChangeLog 2016-03-30 Nick Clifton PR target/62254

[commit] Sync include/plugin-api.h with binutils

I'm committing the attached patch to sync include/plugin-api.h with binutils. -cary 2016-03-03 Than McIntosh * plugin-api.h: Add new hooks to the plugin transfer vector to to support querying section alignment and section size. (ld_plugin_get_input_section_alignment):

Commit: CR16: Add newlib-stdint.h to tm_file.

Hi Guys, I am applying the patch below as an obvious fix for a problem with the CR16 target - the lack of a definition of __INTPTR_TYPE__. This definition is needed by the newlib C library's _intsup.h header in order to correctly calculate the size of integers and pointers. Cheers Nick

Commit: MSP430: Update devices list

Hi Guys, I am checking in this patch to update the list of MSP430 devices built in to the msp430 backend. Cheers Nick gcc/ChangeLog 2016-02-17 Nick Clifton * config/msp430/msp430.c (msp430_mcu_data): Sync with data from TI's devices.csv file as of March 2016. Index: gc

Re: Commit: Try harder to run ARM neon testcases

Hi Christian, That works in my local tests. Are you OK with it ? yeah, I checked that it tests the attribute in my configuration as well. just to make things less obfuscated, What about /* { dg-additional-options "-mfpu=fp-armv8 } */ instead, eventually with a selector for armv7-a. just an id

Re: Commit: Try harder to run ARM neon testcases

Hi Christian, * gcc.target/arm/attr-neon.c: Use dg-add-options to add the command line options necessary to enable Neon support. is this last one needed ? the __attribute__ ((target("fpu=neon"))) is here to test without -mfpu=neon. So we are losing something here. Ah, good point. We

Commit: Try harder to run ARM neon testcases

Hi Guys Whilst checking PR 69194 I noticed that the new testcase (gcc.target/arm/pr69194.c) was not being run with a Linux hosted toolchain configured as --target=arm-eabi. Investigating further I found that the check_effective_target_arm_neon_ok_nocache proc in target_supports.exp was

Commit: MSP430: Reduce number of multilibs

Hi Guys, I am applying the attached patch to reduce the number of multilibs for the MSP430 target. This is at the request of TI, on behalf of their customers, who complained that the toolchain was too large. The patch only affects MSP430 specific files, and parts of files. It does not

[commit] [patch] Python Pretty Printers get disabled on libstdc++ reload by GDB (PR libstdc++/68448)

On Fri, 20 Nov 2015 18:40:46 +0100, Jonathan Wakely wrote: > The patch is OK for trunk and gcc-5-branch. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68448 trunk: https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=230669 5.x: https://gcc.gnu.org/viewcvs/gcc?view=revision&revisi

[PATCH, applied] Fixup PowerPC error from my last commit

In my last commit, I had a case where the mode iterator (VSX_L, FMOVE128_GPR) used IFmode (IBM extended double) in code that was only meant for 128-bit types that fit in a single vector. In the case of VSX_L, it had a condition on the iterator, that would never be true for IFmode. But it didn&#

remove inadvertent testcase commit

I've removed a testcase I inadvertently committed along with the routine testcases: 2015-11-09 Nathan Sidwell * testsuite/libgomp.oacc-c-c++-common/firstprivate-1.c: Remove inadvertent commit. nathan

Commit: Fix spelling mistake in RL78 documentation

Hi Guys, I am checking in the patch below to fix a small spelling mistake that I recently introduced to the RL78 documentation: gcc/ChangeLog 2015-10-26 Nick Clifton * doc/invoke.texi (RL78 Options): Fix spelling mistake. Index: gcc/doc/invoke.texi ===

Commit: MSP430: Pass silicon errata options on to the assembler

Hi Guys, I am checking in the patch below to allow gcc to pass the new MSP430 -msilicon-errata and -msilicon-errata-warn option on to the assembler. Cheers Nick gcc/ChangeLog 2015-10-22 Nick Clifton * config/msp430/msp430.opt: Add -msilicon-errata and -msilicon-errata

Commit: MSP430: Add support for persistent data

Hi Guys, I am checking in the attached patch to add support for persistent data to the MSP430 port. Persistent data retains its current value across processor resets (because it is held in flash), so its value must be set once - when the program is loaded - and then never changed by the

Commit: RL78: Improve multiplication cost estimate

Hi DJ, This is the patch that I have checked in. Cheers Nick gcc/ChangeLog 2015-10-06 Nick Clifton * config/rl78/rl78.c (rl78_rtx_costs): Improve cost estimates for multiplication. Index: gcc/config/rl78/rl78.c

[commit][spu] Support atomic builtins

Hello, Fortran tests have been failing on SPU since libgfortran now assumes the atomic builtins are always available. On the SPU, execution is always single-threaded, so we have not provided atomic builtins so far. As suggested by Ian here: https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01818.htm

[commit, spu] Re: [BUILDROBOT] spu: left shift of negative value

Jan-Benedict Glaw wrote: > I just noticed that (for config_list.mk builds), current GCC errors > out at spu.c, see eg. build > http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=3D469639 : > > g++ -fno-PIE -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-excep= > tions -fno-rt

Commit: MSP430: Fix typo in --target-help text

Hi DJ, I am applying the patch below as an obvious fix for a typo in the msp430's --target-help output. Cheers Nick gcc/ChangeLog 2015-09-01 Nick Clifton * config/msp430/msp430.opt (mcpu): Fix typo. Index: gcc/config/msp430/msp430.opt ==

[patch commit SH] Fix PR target/66609

I've commit the attached patch to fix PR target/66609. SH doesn't take into account weak symbol references for the pc relative calls and sibcalls. See PR target/66609 for details. The patch introduces a new unspec enum UNSPEC_PCREL so to generate pc relative symbols like foo@PCREL an

Commit: M32R: Fix handling of the __model__ attribute

Hi Guys, I am checking in the patch below to fix the M32R port's handling of its __model__ attribute. This attribute takes a parameter identifying the memory model to use, but gcc was not being told to expect this identifier. Cheers Nick gcc/ChangeLog 2015-07-31 Nick Clifton

libgo patch committed: Add missing spaces to last mksysinfo commit

The last change to libgo/mksysinfo.sh was missing some spaces, which apparently causaed mksysinfo to hang on some systems. This is https://golang.org/issue/11924. This patch from Lynn Boger fixes the problem. Bootstrapped on x86_64-unknown-linux-gnu, where it made no difference. Committed to ma

Commit: MSP430: Fix zero-extending a 16-bit value to a 20-bit value, in memory

Hi Guys, I am applying the patch below to fix a problem with the MSP430's zero_extendhipsi2 pattern. When the destination is memory, four bytes must be written, even though only a 20-bit value is being stored, as otherwise the top nibble will be lost. Cheers Nick gcc/ChangeLog 2015-07

[patch commit SH] Fix failures for gcc.target/sh/pr49263-2.c with -mlra

Hi, Some scan-assembler-times tests fail with -mlra for gcc.target/sh/pr49263-2.c and gcc.target/sh/pr64345-2.c. A reduced test case int foo (int x) { return (x & 255) == 0;} gives a code like mov r4,r0 tst #255,r0 rts movtr0 with -mno-lra and

Commit: RX: Do not promote vector types

Hi Guys, I am applying the patch below to the RX's handling of vector functions. The RX ABI specifies that small integer return values should always be promoted to 32-bit values, but the code that performs this promotion was also affecting vector types. This results in internal compile

Re: [commit#2] [patch#2] PR other/65366: Fix gdbhooks.py for GDB with Python3

On Mon, Jun 8, 2015 at 3:37 PM, Jan Kratochvil wrote: > On Mon, 08 Jun 2015 09:46:59 +0200, Richard Biener wrote: >> adding a >> >> import sys >> >> makes it work fine though. > > I do not see the sys error with either FSF GDB HEAD or Fedora 22 GDB. > I agree it probably should be there. Yeah, I

[commit#2] [patch#2] PR other/65366: Fix gdbhooks.py for GDB with Python3

On Mon, 08 Jun 2015 09:46:59 +0200, Richard Biener wrote: > adding a > > import sys > > makes it work fine though. I do not see the sys error with either FSF GDB HEAD or Fedora 22 GDB. I agree it probably should be there. > Thus, ok with also adding a imoprt sys. Done and checked in: r224223

[commit] [patch] PR other/65366: Fix gdbhooks.py for GDB with Python3

On Mon, 01 Jun 2015 22:59:03 +0200, Jason Merrill wrote: > OK, thanks. Checked in: r224012 Jan

Commit: MSP430: Add multilibs for different types of hardware multiply

Hi Guys, I am applying the patch below to the MSP430 backend to add new multilibs based upon the type of hardware multiply support used. Cheers Nick gcc/ChangeLog 2015-05-29 Nick Clifton * config/msp430/t-msp430 (MULTILIB_OPTIONS): Add multilibs for the different types

Commit: RX: Better use of PUSHM and POPM

Hi Guys, I am applying the patch below to enhance the RX backend so that it will push and pop multiple groups of registers using the PUSHM and POPM instructions, thus reducing code size and increasing performance. Cheers Nick gcc/ChangeLog 2015-05-28 Nick Clifton * config/r

[patch commit SH] Fix PR target/65979

Hi, I've committed the attached patches to fix PR target/65979 which is a 5/6 regression in behalf of Oleg. The problematic peephole can clobber a register before its use. See PR65979 for details. The first patch is for trunk and the second one is a slightly different version against gcc-5-bran

Commit: MSP430: Enhance the zero_extendhisi2 pattern

Hi Guys, I am applying the patch below to enhance the zero_extendhisi2 pattern in the MSP430 backend so that it can cope with separate source and destination registers. This makes zero extending into another register more efficient and it also helps to work around a reload bug reported

Commit: MSP430: Add -mcode-region= and -mdata-region= options

Hi Guys, I am applying the attached patch to add two new command line options to the MSP430 backend. The -mcode-region= and -mdata-region= options allow the user to specify whether functions and data should be placed into low memory (below 64K) or high memory. This only applies to the

Commit: RL78: Fix compile time warnings

Hi Guys, I am applying the patch below as an obvious fix for come compile time warning messages building the rl78 toolchain. Cheers Nick gcc/ChangeLog 2015-04-23 Nick Clifton * config/rl78/rl78.c (rl78_preferred_reload_class): Add ATTRIBUTE_UNUSED to x parameter.

Commit: RX: Add option to disable string instructions.

Hi Guys, I am applying the attached patch to the RX backend. It adds a new command line option -mno-allow-string-insns which stops the compiler from using any of the RX string instructions (SMOVF, SUNTIL, etc). These instructions are problematic because they are unsafe if used in the RX

Commit: Avoid calling convert_to_mode with invalid rtl.

Hi Guys, Now that the sources are unfrozen I am applying the patch discussed on this thread: https://gcc.gnu.org/ml/gcc-patches/2015-03/msg00736.html It fixes the places where an address offset is computed in the wrong mode and needs to be converted to the correct mode. Since we can

Commit: V850: Pass -msoft-float/-mhard-float on to the assembler

Hi Guys, I am applying the patch below to update the V850 backend so that it will pass on the -msoft-float and -mhard-float options to the assembler. This matches up with a recent update to the assembler that accepts these options. Cheers Nick gcc/ChangeLog 2015-02-24 Nick Clifton

Commit: FR30: Fix di_operand and nonimmediate_di_operand predicates

Hi Guys, I am checking in the patch below to add SUBREG to the list of rtx codes accepted by the di_operand and nonimmediate_di_operand predicates in the FR30 backend. This should resolve PR 64408. Cheers Nick gcc/ChangeLog 2015-02-04 Nick Clifton PR target/64408 * c

Commit: MSP430: Extend msp430_use_f5_series_hwmult()

Hi Guys, I am applying the patch below to extend the MSP430 port of gcc's auto-recognition of F5 multiply hardware enabled MCUs. Cheers Nick gcc/ChangeLog 2015-02-04 Nick Clifton * config/msp430/msp430.c (msp430_use_f5_series_hwmult): Add more prefixes of known F5 using

[commit] [patch] DW_AT_producer: Ignore -fpreprocessed

On Sat, 17 Jan 2015 07:41:51 +0100, Jakub Jelinek wrote: > > gcc/ChangeLog > > * dwarf2out.c (gen_producer_string): Ignore also OPT_fpreprocessed. > > Ok for trunk, thanks. Checked in: 219797 Jan

Re: [PATCH] config/h8300/h8300.c: Regress part of the original commit for fixing issue

On 1/13/15 01:32, Jeff Law wrote: > On 01/12/15 10:01, Jeff Law wrote: >> This indicates a violation of the type safety invariants we're adding to >> GCC. Simply changing the code to use rtx rather than rtx_insn is >> probably a step in the wrong direction. >> >> Part of the problem here is that R

Re: [PATCH] config/h8300/h8300.c: Regress part of the original commit for fixing issue

On 01/12/15 10:01, Jeff Law wrote: This indicates a violation of the type safety invariants we're adding to GCC. Simply changing the code to use rtx rather than rtx_insn is probably a step in the wrong direction. Part of the problem here is that RTX_FRAME_RELATED_P is valid on both rtx_insn and

Re: [PATCH] config/h8300/h8300.c: Regress part of the original commit for fixing issue

On 01/11/15 07:02, Chen Gang S wrote: The related commit "1a1ed14 config/h8300: Use rtx_insn" gives an extra check for rtx, which will cause building libgcc break, after regress it, it can still generate the correct assemble code. The related information is below: [root@localh

Re: [PATCH] ipa-icf.c: Fix issues generated by original latest commit

On 01/12/2015 09:51 AM, Richard Biener wrote: On Sat, Jan 10, 2015 at 10:03 AM, Chen Gang S wrote: The related commit is "275e275 IPA ICF: target and optimization flags comparison.". For sem_function::equals_private(), fix the typo issue, and for target_opts_for_fn(), fix access

Re: [PATCH] ipa-icf.c: Fix issues generated by original latest commit

On Sat, Jan 10, 2015 at 10:03 AM, Chen Gang S wrote: > The related commit is "275e275 IPA ICF: target and optimization flags > comparison.". For sem_function::equals_private(), fix the typo issue, > and for target_opts_for_fn(), fix access NULL issue. > > For cross compi

[PATCH] config/h8300/h8300.c: Regress part of the original commit for fixing issue

The related commit "1a1ed14 config/h8300: Use rtx_insn" gives an extra check for rtx, which will cause building libgcc break, after regress it, it can still generate the correct assemble code. The related information is below: [root@localhost libgcc]# cat libgcc2.i typedef

Re: [PATCH] ipa-icf.c: Fix issues generated by original latest commit

On 01/10/2015 10:03 AM, Chen Gang S wrote: > The related commit is "275e275 IPA ICF: target and optimization flags > comparison.". For sem_function::equals_private(), fix the typo issue, > and for target_opts_for_fn(), fix access NULL issue. > > For cross compiling h8300

Re: [PATCH] ipa-icf.c: Fix issues generated by original latest commit

Rainer Orth writes: > Chen Gang S writes: > >> The related commit is "275e275 IPA ICF: target and optimization flags >> comparison.". For sem_function::equals_private(), fix the typo issue, >> and for target_opts_for_fn(), fix access NULL issue. > > The N

Re: [PATCH] ipa-icf.c: Fix issues generated by original latest commit

Chen Gang S writes: > The related commit is "275e275 IPA ICF: target and optimization flags > comparison.". For sem_function::equals_private(), fix the typo issue, > and for target_opts_for_fn(), fix access NULL issue. The NULL pointer dereference issue also broke Sola

[PATCH] ipa-icf.c: Fix issues generated by original latest commit

The related commit is "275e275 IPA ICF: target and optimization flags comparison.". For sem_function::equals_private(), fix the typo issue, and for target_opts_for_fn(), fix access NULL issue. For cross compiling h8300, it will cause the issue below: [root@localhost h8300]# ca

Commit: MSP430: Fix addsi splitter (PR 64160)

Hi Guys. I am applying the patch below to fix PR 64160 for the MSP430. Cheers Nick gcc/ChangeLog 2014-12-24 Nick Clifton PR target/64160 * config/msp430/msp430.md (addsi splitter): Do not split when the destination partially overlaps the source. Index: config/msp

Commit: 4.9 branch: Backport RX 'R' operator fix

Hi Guys, I am backporting an RX-specific patch from the mainline to the 4.9 branch as it fixes an ICE running the gcc test gcc.c-torture/compile/pr39423-2.c. Cheers Nick gcc/ChangeLog 2014-12-24 Nick Clifton Backport from mainline: 2014-03-25 Nick Clifton

Commit: 4.9 branch: Backport RX alignment fix

Hi Guys, I am backporting an RX-specific patch from the mainline to the 4.9 branch as this fixes an internal error in the assembler (!) when running the gcc test gcc.c-torture/compile/pr53748.c. Cheers Nick gcc/ChangeLog 2014-12-24 Nick Clifton Backport from mainline:

[commit] reload: Fix PR 64010

Hello, as discussed in the PR, this fixes a wrong-code reload bug in the following situation in push_reload: /* If this is an input reload and the operand contains a register that dies in this insn and is used nowhere else, see if it is the right class to be used for this reload. Use

Commit: Rl78: Save ES register in interrupt handlers

Hi Guys, I am applying the patch below to fix the RL78 backend so that it will preserve the ES register if an interrupt handler uses it. The ES register can be altered if a __far variable is addressed inside the handler. Tested without any regressions on an rl78-elf toolchain. Cheers

[patch commit] [SH] Use define_c_enum in sh.md

I've noticed that config/sh/sh.md uses define_constants to define unspec and unspecv numbers, though config/sh/sync.md uses define_c_enum for them. This causes collisions of some numbers. The attached patch would be an obvious fix. Tested on sh4-unknown-linux. Applied to trunk. I'll backport i

Commit: MSP430: Improve -mhwmult= opton and fix prologues and epilogues for naked functions

Hi Guys, I am applying the patch below to the MSP430 backend. It improves the -mhwmult=auto command line option so that MCUs without any hardware support will be recognised if the -mmcu= option has also been given. The patch also fixes a small problem with the prologue and epilogue gen

Re: Commit: SH: Fix typo in atomic fetch sequence

On Thu, 2014-09-25 at 17:23 +0200, Oleg Endo wrote: > On Thu, 2014-09-25 at 16:04 +0100, Nick Clifton wrote: > > Hi Alex, Hi Kaz, Hi Oleg, > > > > I am applying the patch below as an obvious fix for a typo in the > > instruction sequence of the atomic_fetch_nand_soft_imask > > pattern. I ho

Re: Commit: SH: Fix typo in atomic fetch sequence

On Thu, 2014-09-25 at 16:04 +0100, Nick Clifton wrote: > Hi Alex, Hi Kaz, Hi Oleg, > > I am applying the patch below as an obvious fix for a typo in the > instruction sequence of the atomic_fetch_nand_soft_imask > pattern. I hope that this is OK with you. Yes, ouch. Thanks! I'll backport

Commit: SH: Fix typo in atomic fetch sequence

Hi Alex, Hi Kaz, Hi Oleg, I am applying the patch below as an obvious fix for a typo in the instruction sequence of the atomic_fetch_nand_soft_imask pattern. I hope that this is OK with you. Cheers Nick gcc/ChangeLog 2014-09-25 Nick Clifton PR target/62218 * config/s

Commit: RL78: Fix poping of registers in G10 mode

Hi Guys, I am applying the patch below to fix a problem with the RL78's popping of pushed registers in G10 mode. The problem was that the pop uses a two instruction sequence and dead code elimination was deleting the second instruction (the move of the popped value from register A to th

[PATCH][match-and-simplify] Fix last commit

Committed. Richard. 2014-09-10 Richard Biener * match-conversions.pd: Fix unsigned type used for shortened multiplication. Index: gcc/match-conversions.pd === --- gcc/match-conversions.pd(revision 215057) ++

Commit:

Hi Guys, I am applying the patch below as an obvious fix. It adds a missing @gol to the end of one of the option list lines and it removes a superfluous second "functions" from the description of the -mhotpatch option. Cheers Nick gcc/ChangeLog 2014-09-09 Nick Clifton * do

[PATCH COMMIT]Handle REG addressing mode in output_move_neon explicitly

Hi, This patch is posted/approved before at https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00311.html It's one prerequisite patch for memset inlining patch on ARM. Unfortunately, it was reverted along with it because the whole patch set revealed a latent bug causing glibc build failure and I didn't

[commit, s390] Fix TPF unwinder

Hello, I've checked in a fix provided by Jim Johnston on the TPF team to fix a case where the special TPF unwinder didn't work correctly. Tested on TPF by Jim, committed to mainline. Bye, Ulrich ChangeLog: gcc/ 2014-07-30 Ulrich Weigand * config/s390/s390.c (s390_emit_tpf_eh_retur

[commit, 4.8/4.9, libobjc] Match branch ABI (Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields)

> Matthias Klose wrote: > > > #define rs6000_special_adjust_field_align_p(FIELD, COMPUTED) 0 > > > > is what succeeds for me. false is not defined for ObjC. Checked in on the > > trunk > > and the branches. > > Since we didn't backport the actual ABI change to the branches, only the > warning,

[COMMIT] Add a .gitattributes file for use with git-merge-changelog

Individual users will still have to: 1. Install git-merge-changelog 2. Set up the merge driver in their git config See gnulib's lib/git-merge-changelog.c [1] for details. For example, I: 1. Patched Debian's gnulib package to build git-merge-changelog, and sent the patch to the Debian ma

Commit: RX:

Hi Guys, I am checking in the patch below to fix a small DWARF generation problem with the RX backend. The stack_push pattern contains two separate operations that act in parallel, but they were written as if they happened in sequence. Which meant that the DWARF generated to show where

Re: [commit] Fix ABI fallout (Re: wide-int, rs6000)

Mike Stump wrote: > On Jun 28, 2014, at 3:31 AM, Ulrich Weigand wrote: > > Mike Stump wrote: > > > >>(rs6000_aggregate_candidate): Use wide-int interfaces. > > [snip] > >> - /* Can't handle incomplete types. */ > >> - if (!COMPLETE_TYPE_P (type)) > >> -return -1; > >> + /* Can't handl

Re: [commit] Fix ABI fallout (Re: wide-int, rs6000)

On Jun 28, 2014, at 3:31 AM, Ulrich Weigand wrote: > Mike Stump wrote: > >> (rs6000_aggregate_candidate): Use wide-int interfaces. > [snip] >> -/* Can't handle incomplete types. */ >> -if (!COMPLETE_TYPE_P (type)) >> - return -1; >> +/* Can't handle incomplete types nor siz

[commit] Fix ABI fallout (Re: wide-int, rs6000)

Mike Stump wrote: > (rs6000_aggregate_candidate): Use wide-int interfaces. [snip] > - /* Can't handle incomplete types. */ > - if (!COMPLETE_TYPE_P (type)) > - return -1; > + /* Can't handle incomplete types nor sizes that are not > +fixed. */ > + if (!COMPLET

Commit: FRV: Remove redundant assert

Hi Guys, I am checking in the patch below to remove a redundant assert, now that DECL_SECTION_NAME returns a string rather than a tree. Cheers Nick gcc/ChangeLog 2014-06-26 Nick Clifton * config/frv/frv.c (frv_in_small_data_p): Remove redundant assert. Index: config/frv/frv.c

Commit: Testsuite: Fix typo in proc check_effective_target_trapping

Hi Guys, I am applying the patch below as an obvious fix for a typo in the check_effective_target_trapping proc in the testsuite's target-supports.exp file. Cheers Nick gcc/testsuite/ChangeLog 2014-06-26 Nick Clifton * lib/target-supports.exp (check_effective_target_trapping)

Commit: RX: Fix typo in description of RX vector attribute

Hi Guys, I am checking in the patch below to fix a small typo in the description of the RX vector function attribute. Cheers Nick gcc/ChangeLog 2014-06-26 Nick Clifton * doc/extend.texi (Function Attributes): Fix typo in description of RX vector attribute. Index: doc/e

Commit: MSP430: Add NOP after DINT in hardware multiply patterns

Hi Guys, I am checking in the patch below to update the hardware multiply patterns for the MSP430 so that there is a NOP instruction after disabling interrupts with the DINT instruction. Timing issues mean that it is possible for the instruction following the DINT to be interrupted, so

Commit: RX: Correctly honour user specified alignment of jumps, loops and labels

Hi Guys, I am applying the patch below to fix a small problem with the RX port - it was using non-log based alignment values for jumps, loops and labels when user specified alignment was enabled. Cheers Nick gcc/ChangeLog 2014-06-13 Nick Clifton * config/rx/rx.h (JUMP_ALIGN):

Re: [Patch] Change URL in commit emails to https

On Mon, 12 May 2014, Tobias Burnus wrote: The patch changes the URL shown in the release message to HTTPS. (Cf. https://gcc.gnu.org/viewcvs/gcc/hooks/svnmailer.conf and gcc-cvs mailing list.) Yes, please. Thanks! Gerald

Re: Commit policy? Re: [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags

On Tue, Jun 10, 2014 at 8:47 AM, Thomas Schwinge wrote: > Hi! > > On Tue, 3 Jun 2014 11:55:44 +0200, I wrote: >> Ping -- OK to commit to trunk? > > Even though several of those who I'd consider regular GCC developers do > agree with this patch (see also <https:

<    1   2   3   4   5   6   >