Re: patch to fix PR56999

On 13-04-18 11:58 PM, Vladimir Makarov wrote: 2013-04-18 Vladimir Makarov PR rtl-optimization/56992 * lra-coalesce.c (coalescable_pseudo_p): Remove 2nd parameter and related code. (lra_coalesce): Remove split_origin_bitmap and related code. * lra.c (lr

patch to fix PR56999

The following patch fixes http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56999 The problem was in complicated interactions of coalescing and doing/undoing inheritance and assignment sub-passes through several iterations. One solution would be in modifying coalescing in order to take future

Re: [PATCH, PR 10474] Shedule pass_cprop_hardreg before pass_thread_prologue_and_epilogue

Hi, On Fri, Apr 19, 2013 at 12:37:58AM +0200, Steven Bosscher wrote: > On Fri, Apr 19, 2013 at 12:09 AM, Martin Jambor wrote: > > I also have not tried scheduling the hard register copy propagation > > pass twice and measuring the impact on compile times. Any suggestion > > what might be a good t

Re: [PATCH, PR 10474] Shedule pass_cprop_hardreg before pass_thread_prologue_and_epilogue

On Fri, Apr 19, 2013 at 12:09 AM, Martin Jambor wrote: > I also have not tried scheduling the hard register copy propagation > pass twice and measuring the impact on compile times. Any suggestion > what might be a good testcase for that? I think a better question is when this would be useful in t

[google] Fix testsuite failures with -fdebug-types-section.

This patch for the google/gcc-4_8 branch. It backports the following two patches from trunk to fix testsuite failures when -fdebug-types-section is enabled. Tested with crosstool-validate.py. http://gcc.gnu.org/ml/gcc-cvs/2013-04/msg00768.html 2013-04-18 Cary Coutant gcc/testsuite/

Re: [PATCH, PR 10474] Shedule pass_cprop_hardreg before pass_thread_prologue_and_epilogue

Hi, On Wed, Apr 17, 2013 at 12:43:59PM -0600, Jeff Law wrote: > On 04/17/2013 09:49 AM, Martin Jambor wrote: > > > >The reason why it helps so much is that before register allocation > >there are instructions moving the value of actual arguments from > >"originally hard" register (e.g. SI, DI, etc

[patch rfa] Fix PCH test failure when -fdebug-types-section is enabled

If I turn on -fdebug-types-section, I get a failure in the PCH tests: FAIL: g++.dg/pch/system-2.C -O2 -g assembly comparison This appears to be the result of differences in hash table traversal during debug output (same output, slightly different order), so I'm thinking it's just luck that it wor

[GOOGLE] Fix the LIPO support in AutoFDO

This patch fixes LIPO support in AutoFDO. Bootstrapped and passed regression tests. OK for google-4_7 branch? Thanks, Dehao Index: gcc/coverage.c === --- gcc/coverage.c (revision 198056) +++ gcc/coverage.c (working copy) @@ -2857,7

[PATCH, AArch64] Update definitions of _FP_W_TYPE and _FP_WS_TYPE in libgcc to be based on 'long long'

Hi, The patch changes _FP_W_TYPE and _FP_WS_TYPE in AArch64 libgcc to be based on 'long long', making them ILP32-friendly. x86_64 has the definitions. OK for the trunk? Thanks, Yufeng libgcc/ 2013-04-18 Yufeng Zhang * config/aarch64/sfp-machine.h (_FP_W_TYPE): Change to define

Re: RFA: enable LRA for rs6000 [patch for WRF]

On Thu, Apr 18, 2013 at 04:44:21PM -0400, Vladimir Makarov wrote: > LRA can fix the wrong address but secondary reload was done before > processing addresses. It could be fixed in rs6000.c code too but it > is complicated and I found a better (and i think more right) > solution by moving secondar

Re: RFA: enable LRA for rs6000 [patch for WRF]

On 04/17/2013 12:10 PM, Michael Meissner wrote: On Wed, Apr 17, 2013 at 10:14:53AM -0400, Vladimir Makarov wrote: Mike, thanks for the patch and all the SPEC2006 data (which are very useful as I have no access to power machine which can be used for benchmarking). I guess that may be some bench

[patch][google/gcc-4_7] Suppress failure in gcc.dg/lto/20100430-1 on powerpc and powerpc64

Greetings, I've commited the patch below on google/gcc-4_7 branch (r198071) to XFAILs gcc.dg/lto/20100430-1 on powerpc*. Index: contrib/testsuite-management/powerpc64-grtev3-linux-gnu.xfail === --- contrib/testsuite-management/pow

[PATCH, i386]: Simplify *call_value_rex64_ms_sysv and *call_rex64_ms_sysv.

Hello! Attached patch avoids explicit clobbers in *call_value_rex64_ms_sysv and *call_rex64_ms_sysv by introducing call_rex64_ms_sysv_operation predicate. In addition to simplified pattern, the insn RTX pattern is now constructed from the same array as it is generated from. 2013-04-18 Uros Bizja

[Patch, Fortran, committed] Plug some memory leaks

Another batch of issues found by Coverity - this time only memory leaks. Committed as Rev. 198068 after build + regtesting on x86-64-gnu-linux. Tobias 2013-04-18 Tobias Burnus * expr.c (find_array_element): Don't copy expr. * data.c (create_character_initializer): Free expr. * frontend-pa

Re: [patch] Fix test failure with -fdebug-types-section.

> 2013-04-18 Cary Coutant > > gcc/ > * dwarf2out.c (output_pubnames ): > > gcc/testsuite/ > * g++.dg/debug/dwarf2/pubnames-2.C: Add -fno-debug-types-section. > * g++.dg/debug/dwarf2/pubnames-3.C: New test case. Sorry, I didn't get the complete ChangeLog entry into the e

[patch] Fix test failure with -fdebug-types-section.

I'm committing the following patch to fix a debug test failure when -fdebug-types-section is on. With -fdebug-types-section, GCC fails to emit enumerators into the pubnames table when the enumerator type has been moved to a separate type unit. In output_pubnames, we're checking to see if the type

Re: [Patch] [MIPS] Fix Many warnings in MIPS port (Was: [PATCH] [MIPS] microMIPS gcc support)

On 04/14/2013 01:27 PM, Moore, Catherine wrote: -Original Message- From: David Daney [mailto:ddaney.c...@gmail.com] Sent: Friday, April 12, 2013 7:29 PM To: Moore, Catherine Cc: Rozycki, Maciej; gcc-patches@gcc.gnu.org; Richard Sandiford Subject: Re: [Patch] [MIPS] Fix Many warnings in

[contrib]: Loosen check of build directory in validate_failures.py

Some cross targets have different build directory names that those expected by the script. I loosened the check by simply making it check for the actual values we care to get out of the Makefile. This should remove some false positives we've been getting. Committed to trunk. Will backport to

[patch cygwin]: Add missing 64-bit cygwin triplet to config.gcc

Hi, an ChangeLog 2013-04-18 Kai Tietz * config.gcc: Add x86_64-*-cygwin triplet. Tested for x86_64-pc-cygwin. I will apply it tomorrow, if there are no objections. Kai Index: config.gcc === --- config.gcc (Revision 1

[patch] Fix DWARF test cases that don't work with -fdebug-types-sections

Several test cases in g++.dg/debug/dwarf2 don't work when -fdebug-types-section is enabled. In all but one case, the test cases are fine and -fdebug-types-section works fine -- it's just that the patterns in the assembly code don't match when the option is on. I'm committing the following patch to

Re: [PATCH 4/5] Altera Nios II: dwarf generation fix

On Thu, Apr 18, 2013 at 6:49 AM, Chung-Lin Tang wrote: > This patch was a fix by Julian which corrected a HOST_BITS_PER_WIDE_INT > host dependency in dwarf generation. Nios II does not have > need_64bit_hwint switched on during configuring, and ran into GDB test > FAILs originating from this probl

[patch] [4.7] revisit PR middle-end/56848 for 4.7.4

revisit PR middle-end/56848 for 4.7.4. as outlined in the bug report, this reapplies the patch backed out for the 4.7.3 release and backports r190733. No regressions in the testsuite on x86_64-linux-gnu compared to the 4.7.3 release. Ok to apply? Matthias gcc/ 2013-04-18 Matthias Klose

Re: [Ping, Patch, testsuite] Add -gdwarf to dg-options in debug/dwarf2 testcases

On Thu, Apr 18, 2013 at 04:18:31PM +0200, Jakub Jelinek wrote: > On Thu, Apr 18, 2013 at 07:46:30PM +0530, Senthil Kumar Selvaraj wrote: > > Ping. > > > > http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00665.html > > Shouldn't dwarf2.exp just prepend that to options instead? It does it if the test

Re: [PATCH, SH] PR target/56995

Christian Bruel wrote: > So, Just removing DF_HI_REGS seems to fix the issue with strictly same > performance results for SH4. > > No regressions in the testsuite for > sh-sim//-m2/ > sh-sim//-m2a/ > sh-sim//-m2a-nofpu/ > sh-sim//-m2a-single/ > sh-sim//-m2a-single-only/ >

Re: [Ping, Patch, testsuite] Add -gdwarf to dg-options in debug/dwarf2 testcases

On Thu, Apr 18, 2013 at 07:46:30PM +0530, Senthil Kumar Selvaraj wrote: > Ping. > > http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00665.html Shouldn't dwarf2.exp just prepend that to options instead? If all tests in dwarf2/ directory are supposed to have it, then it just should be done in one plac

[Ping, Patch, testsuite] Add -gdwarf to dg-options in debug/dwarf2 testcases

Ping. http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00665.html Regards Senthil On Thu, Apr 11, 2013 at 03:36:03PM +0530, Senthil Kumar Selvaraj wrote: > Hi, > > This patch adds the new -gdwarf option to dg-options for testcases in > gcc.dg/debug/dwarf2 that don't already explicitly request DWARF.

[Ping, Patch, c-common] Emit error for negative _Alignas alignment values

Ping again! Regards Senthil On Wed, Apr 03, 2013 at 07:18:20PM +0530, Senthil Kumar Selvaraj wrote: > This patch detects and emits an error if the value provided in _Alignas > is negative. The fix was approved pending full regression testing in a > previous discussion (http://gcc.gnu.org/ml/gcc/2

Re: [testsuite, i386] Reimplementing array comparison in avx2-vpop-check.h

> OK with that change. Hi, Checked into trunk: http://gcc.gnu.org/ml/gcc-cvs/2013-04/msg00763.html Thanks, K

Re: [PATCH 3/5] Altera Nios II: testsuite patches

On 2013/4/18 09:49 PM, Chung-Lin Tang wrote: > Testsuite changes, including gcc.target tests, plus some selected > filtering of some other tests. > > 2013-04-18 Sandra Loosemore > Chung-Lin Tang > Based on patches from Altera Corporation Missed filling in one ChangeLo

[PATCH 5/5] Altera Nios II: hexadecimal numbers in options

Altera has defined an FPU instruction set on top of the custom instruction space provided in Nios II, but without specifying the exact binary opcode mapping within the [0,255] space; that mapping is specified by the user, through compiler options or target pragmas. For ease of use in specifying th

[PATCH 4/5] Altera Nios II: dwarf generation fix

This patch was a fix by Julian which corrected a HOST_BITS_PER_WIDE_INT host dependency in dwarf generation. Nios II does not have need_64bit_hwint switched on during configuring, and ran into GDB test FAILs originating from this problem. 2013-04-18 Julian Brown * dwarf2out.c (gen_enum

[PATCH 3/5] Altera Nios II: testsuite patches

Testsuite changes, including gcc.target tests, plus some selected filtering of some other tests. 2013-04-18 Sandra Loosemore Chung-Lin Tang Based on patches from Altera Corporation * gcc.dg/stack-usage-1.c (SIZE): Define case for __nios2__. * gcc.dg/20

[PATCH 2/5] Altera Nios II: libgcc

These are patches for libgcc. 2013-04-18 Sandra Loosemore Chung-Lin Tang Based on patches from Altera Corporation * config.host (nios2-*-*,nios2-*-linux*): Add nios2 host cases. * config/nios2/lib2-nios2.h: New file. * config/nios2/lib2-divmod-

[PATCH 0/5] Submission of Altera Nios II port

Hi, Mentor Graphics is submitting on behalf of Altera, a GCC port for the Nios II architecture. Attached are the series of patches, including a few small additions to machine-independent parts. We are proposing Sandra Loosemoore and myself (Chung-Lin Tang), both of Mentor Graphics, as port mainta

[PING] RE: [PING]RE: [patch] cilkplus: Array notation for C patch

Hi Joseph et al., Did you all get a chance to look at this patch? Thanks, Balaji V. Iyer. > -Original Message- > From: Iyer, Balaji V > Sent: Monday, April 08, 2013 9:56 AM > To: 'Joseph Myers'; 'Aldy Hernandez' > Cc: 'gcc-patches' > Subject: [PING]RE: [patch] cilkplus: Array not

[PATCH] More vectorizer TLC

While trying to remove some restrictions I came along more TLC opportunities. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2013-04-18 Richard Biener * tree-vect-data-refs.c (vect_analyze_group_access): Properly handle negative step. Remove redundan

Re: [testsuite] Adding target nonpic to g++.dg/tm/pr47746.C

Hi Alexander, On Thu, Apr 18, 2013 at 12:49 PM, Alexander Ivchenko wrote: > I'm trying it on linux/x86_64 on trunk. Testing just by adding -fpic > to the dg-options: > > --- a/gcc/testsuite/g++.dg/tm/pr47746.C > +++ b/gcc/testsuite/g++.dg/tm/pr47746.C > @@ -1,5 +1,5 @@ > // { dg-do compile } > -

New German PO file for 'gcc' (version 4.8.0)

Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the German team of translators. The file is available at: http://translationproject.org/latest/gcc/de.po (This file, 'gcc-4.8.0.de.po', has just

Re: [PATCH, x86] Use vector moves in memmove expanding

Forgot to add Uros - adding now. On 18 April 2013 15:53, Michael Zolotukhin wrote: > Hi, > Jan, thanks for the review, I hope to prepare an updated version of the patch > shortly. Please see my answers to your comments below. > > Uros, there is a question of a better approach for generation of w

Re: [PATCH, x86] Use vector moves in memmove expanding

Hi, Jan, thanks for the review, I hope to prepare an updated version of the patch shortly. Please see my answers to your comments below. Uros, there is a question of a better approach for generation of wide moves. Could you please comment it (see details in bullets 3 and 5)? 1. > +static int sma

Re: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions

> I haven't looked at compare-elim, but probably not. Thumb1 doesn't use > cc[0] and doesn't expose flag setting instructions in a MI manner (since > add instructions clobber the flags). compare-elim is supposed to be designed for this kind of targets, but of course you need to split the instruc

Re: [testsuite] Adding target nonpic to g++.dg/tm/pr47746.C

Hi Patrick, I'm trying it on linux/x86_64 on trunk. Testing just by adding -fpic to the dg-options: --- a/gcc/testsuite/g++.dg/tm/pr47746.C +++ b/gcc/testsuite/g++.dg/tm/pr47746.C @@ -1,5 +1,5 @@ // { dg-do compile } -// { dg-options "-fgnu-tm" } +// { dg-options "-fgnu-tm -fpic" } Here is the

Re: [testsuite, i386] Reimplementing array comparison in avx2-vpop-check.h

On Thu, Apr 18, 2013 at 12:10 PM, Alexander Ivchenko wrote: > Yep, that also works. > > diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog > index 64ffe8f..7efc3f1 100644 > --- a/gcc/testsuite/ChangeLog > +++ b/gcc/testsuite/ChangeLog > @@ -1,3 +1,8 @@ > +2013-04-18 Grigoriy Kraynov

Re: [PATCH, tree-ssa] Avoid -Wuninitialized warning in try_unroll_loop_completely()

2013/4/18 Jeff Law : > On 04/15/2013 08:35 PM, Chung-Ju Wu wrote: >> >> I was using gcc-4.6.3, which is provided by Ubuntu 12.04, >> and the warning is displayed during the compilation process. >> As I tried to build another native gcc by myself with >> current main trunk and used it to compile tre

Re: [testsuite, i386] Reimplementing array comparison in avx2-vpop-check.h

Yep, that also works. diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 64ffe8f..7efc3f1 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2013-04-18 Grigoriy Kraynov + + * gcc.target/i386/avx2-vpop-check.h: volatility is casted away in

RE: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions

> -Original Message- > From: Richard Earnshaw > Sent: Thursday, April 18, 2013 5:25 PM > To: Steven Bosscher > Cc: Bin Cheng; gcc-patches@gcc.gnu.org > Subject: Re: [PATCH ARM]Extend thumb1_reorg to save more comparison > instructions > > On 18/04/13 09:20, Steven Bosscher wrote: > > On

Re: [Patch, ARM] Enable libsanitizer

On 4 April 2013 14:19, Christophe Lyon wrote: > ~/src/qemu/qemu-git/arm-linux-user/qemu-arm -cpu cortex-a9 -R 0 -L > /home/lyon/src/GCC/builds/gcc-fsf-asan-arm-none-linux-gnueabihf/sysroot > ./heap-overflow-1.arm > > On 28 March 2013 15:33, Christophe Lyon wrote: - libsanitizer detects if i

Re: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions

On 18/04/13 09:20, Steven Bosscher wrote: On Thu, Apr 18, 2013 at 7:34 AM, Bin Cheng wrote: Hi, Before thumb1_reorg, ARM backend uses peephole to save comparison instructions when a flag setting move is found before branch instruction. Since we are using thumb1_reog now, it can be extended to ca

[PATCH, SH] PR target/56995

Hello, While checking the register classes definitions to fix this ICE, I noticed that DF_HI_REGS seems to be always strictly equivalent to DF_REGS... Indeed, we have: /* DF_HI_REGS: Initialized TARGET_CONDITIONAL_REGISTER_USAGE.*/ { 0x, 0x, 0x, 0x, 0x0

Unreviewed build/driver patch

The following patch has remained unreviewed for a week: [build, driver] Support compressed debug sections http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00679.html It needs build and driver maintainers for review. Thanks. Rainer --

Re: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions

On Thu, Apr 18, 2013 at 7:34 AM, Bin Cheng wrote: > Hi, > Before thumb1_reorg, ARM backend uses peephole to save comparison > instructions when a flag setting move is found before branch instruction. > Since we are using thumb1_reog now, it can be extended to catch more > opportunities by searching

[wwwdocs, committed] gcc-4.9/changes.html - Update Fortran section (add NO_ARGS_CHECK)

I have committed the attached patch, see http://gcc.gnu.org/gcc-4.9/changes.html Tobias Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.9/changes.html,v retrieving revision 1.4 diff -u -r1.4 changes.html --- changes.

Re: [testsuite] Adding target nonpic to g++.dg/tm/pr47746.C

Hi Alexander, On Thu, Apr 11, 2013 at 11:37 AM, Alexander Ivchenko wrote: > The same motivation as for: > http://gcc.gnu.org/ml/gcc-cvs/2013-03/msg00786.html > > "Since -fpic option is turned on by default in Android we have certain test > fails. The reason for that is that those tests rely on th

[Patch, Fortran, committed] PR56994 - invoke.texi: NEAREST's second argument isn't optional

Committed as obvious to 4.7, 4.8 and the trunk. (Rev. 198048-198050). Tobias Index: gcc/fortran/ChangeLog === --- gcc/fortran/ChangeLog (Revision 198047) +++ gcc/fortran/ChangeLog (Arbeitskopie) @@ -1,3 +1,8 @@ +2013-04-18 Tobias Bu

Re: [PATCH] VRP (x << 31) < -1 fix (PR tree-optimization/56984)

On Thu, 18 Apr 2013, Jakub Jelinek wrote: > Hi! > > Like in the case a few lines below, if new_val is a minimum resp. a few > lines below maximum value, we end up with always true resp. always false > assertion, which VRP asserts that those aren't created. So, we should punt > on those, and inst

[Patch Ping] Re: [C++/C Patch] Have -Wpointer-arith enable by -Wpedantic, as documented

Hi, On 4/5/13 12:07 AM, Paolo Carlini wrote: Hi, in the audit trail of c++/56815 Manuel noticed that, inconsistently with the documentation, a LangEnabledBy was missing for -Wpointer-arith vs -Wpedantic. Then I noticed that a clean up was possible in the actual pedwarn calls, which, in fac