[PATCH] fixincludes/Makefile for Interix Rev 3

2011-06-08 Thread Douglas B Rupp
Reformulated as per your suggestion, and retested. OK to commit? --Douglas Rupp 2011-06-07 Douglas B Rupp r...@gnat.com * fixincludes/configure.ac (AC_USE_SYSTEM_EXTENSIONS): Add. * fixincludes/configure: Regenerate. * fixincludes/config.h.in: Regenerate. ---

Re: [Patch, Fortran] (Coarray) Add parse support for LOCK/UNLOCK (part 1 of 2)

2011-06-08 Thread Tobias Burnus
Dear Daniel, thank you for the review. Daniel Kraft wrote: + fputs (lock-variable=, dumpfile); + if (c-expr1 != NULL) + show_expr (c-expr1); Why do you dump lock-variable= in any case, while you only print the names for the other arguments only if present? The lock variable

Re: [PATCH] fixincludes/Makefile for Interix Rev 3

2011-06-08 Thread Paolo Bonzini
On 06/08/2011 08:02 AM, Douglas B Rupp wrote: Reformulated as per your suggestion, and retested. OK to commit? Yes, thanks! Paolo

Re: libobjc: Remove Traditional Objective-C runtime API

2011-06-08 Thread Nicola Pero
This patch completes the removal of the public part of the Traditional Objective-C runtime API from libobjc. From now on, the only supported API is the Modern API. :-) If you are removing exported functions from libobjc.so, you should bump VERSION= in libobjc/configure.ac. Yes, sure

Re: libobjc: Remove Traditional Objective-C runtime API

2011-06-08 Thread Nicola Pero
This patch completes the removal of the public part of the Traditional Objective-C runtime API from libobjc. From now on, the only supported API is the Modern API. :-) If you are removing exported functions from libobjc.so, you should bump VERSION= in libobjc/configure.ac. Yes, sure

Re: libobjc: Remove Traditional Objective-C runtime API

2011-06-08 Thread Nicola Pero
-innovation.com + * objc/objc.h (__GNU_LIBOBJC__): Bumped to 20110608. + +2011-06-08 Nicola Pero nicola.p...@meta-innovation.com + * configure.ac (VERSION): Bumped to 4:0:0. * configure (VERSION): Likewise. Index: objc/objc.h

Re: libobjc: Remove Traditional Objective-C runtime API

2011-06-08 Thread Jakub Jelinek
On Wed, Jun 08, 2011 at 09:25:09AM +0100, Nicola Pero wrote: This patch completes the removal of the public part of the Traditional Objective-C runtime API from libobjc. From now on, the only supported API is the Modern API. :-) If you are removing exported functions from

Re: -fdump-passes -fenable-xxx=func_name_list

2011-06-08 Thread Richard Guenther
On Tue, Jun 7, 2011 at 8:54 PM, Xinliang David Li davi...@google.com wrote: Please review the attached two patches. In the first patch, gate functions are cleaned up. All the per function legality checks are moved into the executor and the optimization heuristic checks (optimize for size)

Re: -fdump-passes -fenable-xxx=func_name_list

2011-06-08 Thread Richard Guenther
On Tue, Jun 7, 2011 at 10:20 PM, Xinliang David Li davi...@google.com wrote: The dump-pass patch with test case. Ok. Thanks, Richard. David On Tue, Jun 7, 2011 at 11:54 AM, Xinliang David Li davi...@google.com wrote: Please review the attached two patches. In the first patch, gate

Re: [google] pessimize stack accounting during inlining

2011-06-08 Thread Richard Guenther
On Wed, Jun 8, 2011 at 1:36 AM, Xinliang David Li davi...@google.com wrote: Ok for google/main.   A good candidate patch for trunk too. Well, it's still not a hard limit as we can't tell how many spill slots or extra call argument or return value slots we need. Richard. Thanks, David On

Re: Dump before flag

2011-06-08 Thread Richard Guenther
On Wed, Jun 8, 2011 at 1:08 AM, Xinliang David Li davi...@google.com wrote: The following is the patch that does the job. Most of the changes are just  removing TODO_dump_func. The major change is in passes.c and tree-pass.h. -fdump-xxx-yyy-start       -- dump before TODO_start

Re: [PATH] PR/49139 fix always_inline failures diagnostics

2011-06-08 Thread Richard Guenther
On Wed, Jun 8, 2011 at 9:46 AM, Christian Bruel christian.br...@st.com wrote: Hello Richard, On 06/06/2011 11:55 AM, Richard Guenther wrote: On Mon, Jun 6, 2011 at 10:58 AM, Christian Bruelchristian.br...@st.com  wrote: OK, the only difference is that we don't have the node analyzed here,

Re: [build] Define HAVE_GAS_HIDDEN on Darwin

2011-06-08 Thread Rainer Orth
Jason Merrill ja...@redhat.com writes: On 05/19/2011 01:33 PM, Rainer Orth wrote: The cleanest way to account for this seems to allow overriding USE_LINKONCE_INDIRECT in target headers (darwin.h in this case). If Jason or Richard consider this appropriate, I'll modify the patch accordingly

[PATCH, PR43864] Gimple level duplicate block cleanup.

2011-06-08 Thread Tom de Vries
Hi Richard, I have a patch for PR43864. The patch adds a gimple level duplicate block cleanup. The patch has been bootstrapped and reg-tested on x86_64, and reg-tested on ARM. The size impact on ARM for spec2000 is shown in the following table (%, lower is better). none

[PATCH, PR43864] Gimple level duplicate block cleanup - test cases.

2011-06-08 Thread Tom de Vries
On 06/08/2011 11:42 AM, Tom de Vries wrote: I'll send the patch with the testcases in a separate email. OK for trunk? Thanks, - Tom 2011-06-08 Tom de Vries t...@codesourcery.com PR middle-end/43864 * gcc.dg/pr43864.c: New test. * gcc.dg/pr43864-2.c: New test. Index:

Re: [PATCH, PR43864] Gimple level duplicate block cleanup.

2011-06-08 Thread Richard Guenther
On Wed, Jun 8, 2011 at 11:42 AM, Tom de Vries vr...@codesourcery.com wrote: Hi Richard, I have a patch for PR43864. The patch adds a gimple level duplicate block cleanup. The patch has been bootstrapped and reg-tested on x86_64, and reg-tested on ARM. The size impact on ARM for spec2000 is

[PATCH] better warning for section conflict

2011-06-08 Thread Andi Kleen
When erroring on a section type conflict print both locations. This makes it a lot easier to track them down, especially in LTO. This is still not quite good before it prints only one conflict pair currently instead of all, but at least all the problems can be fixed iteratively now. Better would

[PATCH] add another sandy bridge model number

2011-06-08 Thread Andi Kleen
Add another sandy bridge model number. Passes bootstrap test suite on x86_64-linux. Ok for trunk and 4.6? 2011-06-6 Andi Kleen a...@linux.intel.com * config/i386/driver-i386.c (host_detect_local_cpu): Add model 0x2d Intel CPU. --- gcc/config/i386/driver-i386.c |1 + 1

[PATCH] Print location for conflicting global regs in warning

2011-06-08 Thread Andi Kleen
Print location for conflicting global regs when warning about them. Otherwise it's hard to track down where they are. To review: I hope the way I use the garbage collector for the global variable is ok? Passes bootstrap test suite on x86_64-linux. Ok to commit? 2011-06-06 Andi Kleen

Re: [testsuite] skip ARM neon-fp16 tests for other -mcpu values

2011-06-08 Thread Joseph S. Myers
On Tue, 7 Jun 2011, Janis Johnson wrote: These tests fail when multilib options use -mfpu= and override the -mfpu=neon-fp16 used for the test: g++.dg/ext/arm-fp16/arm-fp16-ops-5.C g++.dg/ext/arm-fp16/arm-fp16-ops-6.C gcc.dg/torture/arm-fp16-ops-5.c

Re: [PATCH] better warning for section conflict

2011-06-08 Thread Richard Guenther
On Wed, Jun 8, 2011 at 12:07 PM, Andi Kleen a...@firstfloor.org wrote: When erroring on a section type conflict print both locations. This makes it a lot easier to track them down, especially in LTO. This is still not quite good before it prints only one conflict pair currently instead of

Re: [PATCH] Print location for conflicting global regs in warning

2011-06-08 Thread Richard Guenther
On Wed, Jun 8, 2011 at 12:11 PM, Andi Kleen a...@firstfloor.org wrote: Print location for conflicting global regs when warning about them. Otherwise it's hard to track down where they are. To review: I hope the way I use the garbage collector for the global variable is ok? Passes bootstrap

Re: [PATCH, PR43864] Gimple level duplicate block cleanup.

2011-06-08 Thread Steven Bosscher
On Wed, Jun 8, 2011 at 11:55 AM, Richard Guenther richard.guent...@gmail.com wrote: On Wed, Jun 8, 2011 at 11:42 AM, Tom de Vries vr...@codesourcery.com wrote: Hi Richard, I have a patch for PR43864. The patch adds a gimple level duplicate block cleanup. The patch has been bootstrapped and

Re: [pph] Stream TREE_TYPE for identifier node (issue4550121)

2011-06-08 Thread Steven Bosscher
On Tue, Jun 7, 2011 at 11:50 PM, Diego Novillo dnovi...@google.com wrote: On Tue, Jun 7, 2011 at 14:12, Steven Bosscher stevenb@gmail.com wrote: On Tue, Jun 7, 2011 at 8:44 PM, Gabriel Charette gch...@google.com wrote: We need to stream TREE_TYPE for identifier node. That seems unlikely,

Re: [testsuite] skip ARM tests with conflicting options

2011-06-08 Thread Richard Earnshaw
On 08/06/11 03:14, Janis Johnson wrote: On 06/07/2011 06:25 PM, Mike Stump wrote: On Jun 7, 2011, at 4:24 PM, Janis Johnson wrote: On 06/07/2011 02:07 PM, Joseph S. Myers wrote: On Tue, 7 Jun 2011, Janis Johnson wrote: Several tests in gcc.target/arm use dg-options with -mcpu=, which

Re: [PATH] PR/49139 fix always_inline failures diagnostics

2011-06-08 Thread Christian Bruel
On 06/08/2011 11:11 AM, Richard Guenther wrote: On Wed, Jun 8, 2011 at 9:46 AM, Christian Bruelchristian.br...@st.com wrote: Hello Richard, On 06/06/2011 11:55 AM, Richard Guenther wrote: On Mon, Jun 6, 2011 at 10:58 AM, Christian Bruelchristian.br...@st.com wrote: OK, the only

[testsuite] Build tmpdir-gcc.dg-struct-layout-1 tests with -fno-common on Tru64 UNIX (PR middle-end/21953)

2011-06-08 Thread Rainer Orth
Looking at the remaining Tru64 UNIX testsuite failures, I noticed that the gcc.dg/compat/struct-layout-1.exp tests are built with -fno-common on PA HP-UX. Since Alpha Tru64 UNIX is often similar, I tried this and in fact most of the failures are gone. The single one in

Re: Libiberty: POSIXify psignal definition

2011-06-08 Thread Hans-Peter Nilsson
On Thu, 26 May 2011, DJ Delorie wrote: Yes please. I've been arguing for that for some time. Since you feel so strongly about it and nobody objects, go ahead. Backport to open branches? (And note PR48825.) brgds, H-P

[v3] Avoid including the whole limits in a couple of places + small fix to std::duration_values

2011-06-08 Thread Paolo Carlini
Hi, tested x86_64-linux, committed. Paolo. /// 2011-06-08 Paolo Carlini paolo.carl...@oracle.com * include/ext/extptr_allocator.h: Include ext/numeric_traits.h instead of limits. * include/bits/allocator.h: Likewise. * include/std/chrono

Re: Libiberty: POSIXify psignal definition

2011-06-08 Thread Richard Earnshaw
On 08/06/11 12:56, Hans-Peter Nilsson wrote: On Thu, 26 May 2011, DJ Delorie wrote: Yes please. I've been arguing for that for some time. Since you feel so strongly about it and nobody objects, go ahead. Backport to open branches? (And note PR48825.) brgds, H-P I think we have

Re: [PATCH] add another sandy bridge model number

2011-06-08 Thread Uros Bizjak
Hello! 2011-06-6 Andi Kleen a...@linux.intel.com 2011-06-08 ... * config/i386/driver-i386.c (host_detect_local_cpu): Add model 0x2d Intel CPU. OK with updated ChangeLog entry. Thanks, Uros.

C++ PATCH for c++/49322 (libstdc++ testsuite failures)

2011-06-08 Thread Jason Merrill
In my last patch I ggc_freed a tree that was used in tinst levels, leading to GC corruption when a tinst level is saved in a pending template. This patch avoids that by not freeing if we've allocated a pending template. Tested x86_64-pc-linux-gnu, applying to trunk. commit

[libffi] Fix libffi.call/huge_struct.c on Tru64 UNIX

2011-06-08 Thread Rainer Orth
The last libffi failure on Tru64 UNIX is libffi.call/huge_struct.c: FAIL: libffi.call/huge_struct.c -O0 -W -Wall execution test FAIL: libffi.call/huge_struct.c -O2 execution test FAIL: libffi.call/huge_struct.c -O3 execution test FAIL: libffi.call/huge_struct.c -Os execution test FAIL:

PATCH: Support unknown Intel family 0x6 CPUs

2011-06-08 Thread H.J. Lu
Hi, This patch supports unknown Intel family 0x6 CPUs. OK for trunk and 4.6 branch? Thanks. H.J. -- 2011-06-08 H.J. Lu hongjiu...@intel.com * config/i386/driver-i386.c (host_detect_local_cpu): Support unknown Intel family 0x6 CPUs. diff --git a/gcc/config/i386/driver-i386.c

Re: PATCH: Support unknown Intel family 0x6 CPUs

2011-06-08 Thread Uros Bizjak
On Wed, Jun 8, 2011 at 3:20 PM, H.J. Lu hongjiu...@intel.com wrote: Hi, This patch supports unknown Intel family 0x6 CPUs.  OK for trunk and 4.6 branch? Thanks. H.J. -- 2011-06-08  H.J. Lu  hongjiu...@intel.com        * config/i386/driver-i386.c (host_detect_local_cpu): Support        

Re: Libiberty: POSIXify psignal definition

2011-06-08 Thread DJ Delorie
Since you feel so strongly about it and nobody objects, go ahead. Backport to open branches? (And note PR48825.) I've already given my OK, it's up to the branch managers to apply as they see fit.

Cgraph alias reorg 0/14

2011-06-08 Thread Jan Hubicka
Hi, in the following series of patches I want to reorganize handling of alises in cgraph infastructure. The longer term plan is to introduce symbol table into GCC that will be common base for cgraph, varpool and aliases (i.e. all kinds of symbols we have in GCC). Original design of

[PATCH] Improve debug info for IPA-SRA optimized code - add DW_OP_GNU_parameter_ref support (PR debug/47858)

2011-06-08 Thread Jakub Jelinek
Hi! This is something I've already briefly talked about in my GCC Summit talk. E.g. given a simple testcase like: volatile int vv; static __attribute__((noinline)) int foo (int x, int y, int z) { int a = x * 2; int b = y * 2; int c = z * 2; vv++; return x + z; } int bar (int x) {

Re: [pph] Stream TREE_TYPE for identifier node (issue4550121)

2011-06-08 Thread Diego Novillo
On Wed, Jun 8, 2011 at 03:38, Steven Bosscher stevenb@gmail.com wrote: On Tue, Jun 7, 2011 at 11:50 PM, Diego Novillo dnovi...@google.com wrote: On Tue, Jun 7, 2011 at 14:12, Steven Bosscher stevenb@gmail.com wrote: On Tue, Jun 7, 2011 at 8:44 PM, Gabriel Charette gch...@google.com

[Patch ARM] Unify movdi patterns and fix case for mvn in const_ok_for_op.

2011-06-08 Thread Ramana Radhakrishnan
Hi, This fixes a testsuite failure with 20041011-1.c where we are generating out of range adr instructions because a splitter goes awry because we didn't consider the mvn case in const_ok_for_op. So we ended up splitting this movdi reg:r1 0x to (insn 370 21 371 (set (reg:SI 2 r2)

Re: [testsuite] skip ARM tests with conflicting options

2011-06-08 Thread Janis Johnson
On 06/08/2011 03:39 AM, Richard Earnshaw wrote: On 08/06/11 03:14, Janis Johnson wrote: On 06/07/2011 06:25 PM, Mike Stump wrote: On Jun 7, 2011, at 4:24 PM, Janis Johnson wrote: On 06/07/2011 02:07 PM, Joseph S. Myers wrote: On Tue, 7 Jun 2011, Janis Johnson wrote: Several tests in

Re: [testsuite] skip ARM neon-fp16 tests for other -mcpu values

2011-06-08 Thread Janis Johnson
On 06/08/2011 03:17 AM, Joseph S. Myers wrote: On Tue, 7 Jun 2011, Janis Johnson wrote: These tests fail when multilib options use -mfpu= and override the -mfpu=neon-fp16 used for the test: g++.dg/ext/arm-fp16/arm-fp16-ops-5.C g++.dg/ext/arm-fp16/arm-fp16-ops-6.C

Re: [google] Patch to add compiler flag to dump callgraph edge profiles in special .note sections (issue4591045)

2011-06-08 Thread Sriraman Tallam
+davidxl On Tue, Jun 7, 2011 at 7:05 PM, Sriraman Tallam tmsri...@google.com wrote: Patch Description: = I am working on a project to do global function layout in the linker where the linker reads the callgraph edge profile information, generated by FDO, and uses that to

Re: [google] Patch to add compiler flag to dump callgraph edge profiles in special .note sections (issue4591045)

2011-06-08 Thread Sriraman Tallam
On Wed, Jun 8, 2011 at 9:16 AM, Xinliang David Li davi...@google.com wrote: ok for google/main. Thanks, the patch is now committed. David On Wed, Jun 8, 2011 at 9:13 AM, Sriraman Tallam tmsri...@google.com wrote: +davidxl On Tue, Jun 7, 2011 at 7:05 PM, Sriraman Tallam

FORBIDDEN_INC_DEC_CLASSES in ira-costs.c

2011-06-08 Thread Hans-Peter Nilsson
There's a lot of dead code inside the obsolete (removed in 2009) and nowhere else set #ifdef FORBIDDEN_INC_DEC_CLASSES. Remove and poison? I'd say borderline obvious, but maybe there's instead reason to reinstate it, if that code is tested and supposed to be live. brgds, H-P

PATCH: PR other/49325: Incorrect target HAVE_INITFINI_ARRAY check

2011-06-08 Thread H.J. Lu
Hi, Target HAVE_INITFINI_ARRAY support was added by: http://gcc.gnu.org/ml/gcc-patches/2002-11/msg00387.html Unfortunately, it checks if host supports init_array/fini_array sections, not target. It will generate wrong result for cross compiler. This patch uses the target compiler instead of

Re: Dump before flag

2011-06-08 Thread Xinliang David Li
On Wed, Jun 8, 2011 at 2:06 AM, Richard Guenther richard.guent...@gmail.com wrote: On Wed, Jun 8, 2011 at 1:08 AM, Xinliang David Li davi...@google.com wrote: The following is the patch that does the job. Most of the changes are just  removing TODO_dump_func. The major change is in passes.c and

[PATCH] Fix PR lto/49302

2011-06-08 Thread William J. Schmidt
This patch adds code to the cse_sincos pass to expand builtin cabs (x) into sqrt (r*r + i*i), where r = realpart(x) and i = imagpart(x). This is usually handled by fold_builtin_cabs; however, PR49302 shows a case where compile flags of -O0 -flto combined with link flags of -O2 -ffast-math allow

Re: libobjc: Remove Traditional Objective-C runtime API

2011-06-08 Thread Mike Stump
On Jun 8, 2011, at 1:25 AM, Nicola Pero wrote: But - you're right that we should bump it, so I bumped it :-) I suppose we could bump it again when we get into phase 3, when we're finished with the changes. No. The next number is for the next release, and ideally, we want to sit on it for

Re: libobjc: Remove Traditional Objective-C runtime API

2011-06-08 Thread Nicola Pero
No. The next number is for the next release, and ideally, we want to sit on it for another 10 years or so. We offer no compatibility before the RM says, done with the release. Up until then, you can twiddle, after that, no more changes to the abi, though, you can bug fix it. Sounds

[C++ PATCH] Add chain_next GTY markup to struct tinst_level

2011-06-08 Thread Jakub Jelinek
Hi! We shouldn't recurse in GC to mark tinst_level chain, it is IMHO better to iterate through the chain instead. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2011-06-08 Jakub Jelinek ja...@redhat.com * cp-tree.h (struct tinst_level): Add chain_next GTY

[committed] Fix integer-typedef.C testcase (PR testsuite/49323)

2011-06-08 Thread Jakub Jelinek
Hi! This testcase suffered from the same issues as cdtor-1.C testcase recently. Fixed thusly, regtested on x86_64-linux and i686-linux, committed to trunk as obvious. 2011-06-08 Jakub Jelinek ja...@redhat.com PR testsuite/49323 * g++.dg/debug/dwarf2/integer-typedef.C: Add

Go patch committed: Fix type comparison

2011-06-08 Thread Ian Lance Taylor
This patch to the Go frontend fixes a dumb mistake I made: I compared the TYPE_SIZE pointers rather than the actual type size. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 292aa99c6436 go/expressions.cc --- a/go/expressions.cc Tue May 31

Re: [testsuite] skip ARM tests with conflicting options

2011-06-08 Thread Mike Stump
On Jun 8, 2011, at 8:28 AM, Janis Johnson wrote: The big question is whether such a test should be run for all multilibs that might possibly pass the test, or only for default and for mulitlibs that provide the same options. Here, reasonable people may disagree. I suspect in the end, we'll

Re: [pph] Stream TREE_TYPE for identifier node (issue4550121)

2011-06-08 Thread Steven Bosscher
On Wed, Jun 8, 2011 at 4:12 PM, Diego Novillo wrote: That seems unlikely, as identifiers do not have a type. There is some TREE_TYPE abuse in cp-tree.h, perhaps you should find out what you're streaming. It's used by the C++ parser, so it needs to be streamed in pph. Yes, but you should not

Re: [testsuite] skip ARM tests with conflicting options

2011-06-08 Thread Mike Stump
On Jun 8, 2011, at 3:39 AM, Richard Earnshaw wrote: I'm worried by this whole approach of command-line checking. Right, and this was essentially my point originally. Luckily there is enough beef I think behind the curtains to do everything that needs doing without worrying about adding yet

Re: [build] Define HAVE_GAS_HIDDEN on Darwin

2011-06-08 Thread Mike Stump
On Jun 8, 2011, at 2:29 AM, Rainer Orth wrote: The Darwin parts of the previous versions have already been approved, but I'd appreciate if Mike could have a look at the changes. Ok for mainline? Ok.

Re: [C++ PATCH] Add chain_next GTY markup to struct tinst_level

2011-06-08 Thread Jason Merrill
OK. Jason

Re: [build] Define HAVE_GAS_HIDDEN on Darwin

2011-06-08 Thread Mike Stump
On Jun 8, 2011, at 2:29 AM, Rainer Orth wrote: Bootstrapped without regressions on i386-apple-darwin9.8.0. powerpc-apple-darwin9.8.0 bootstrap is still running. Unfortunately, Darwin seems to be exceedingly unreliable under load: tests randomly time out and work again the next time. This

[patch, testsuite] Fix PR 49191 with test for STRICT_ALIGNMENT

2011-06-08 Thread Steve Ellcey
Here is a patch to skip gcc.dg/memcpy-3.c on platforms that define STRICT_ALIGNMENT. It adds a new check_effective routine, check_effective_target_strict_align that looks for a warning that will only be produced on platforms that define STRICT_ALIGNMENT. Technically, the test may not work if the

Re: [patch, testsuite] Fix PR 49191 with test for STRICT_ALIGNMENT

2011-06-08 Thread Jakub Jelinek
On Wed, Jun 08, 2011 at 01:03:53PM -0700, Steve Ellcey wrote: 2011-06-08 Steve Ellcey s...@cup.hp.com PR middle-end/49191 * lib/target-supports.exp (check_effective_target_strict_align): New. * gcc.dg/memcpy-3.c: Add dg-require-effective-target strict_align. Isn't the

Re: [patch, testsuite] Fix PR 49191 with test for STRICT_ALIGNMENT

2011-06-08 Thread Steve Ellcey
On Wed, 2011-06-08 at 22:08 +0200, Jakub Jelinek wrote: On Wed, Jun 08, 2011 at 01:03:53PM -0700, Steve Ellcey wrote: 2011-06-08 Steve Ellcey s...@cup.hp.com PR middle-end/49191 * lib/target-supports.exp (check_effective_target_strict_align): New. * gcc.dg/memcpy-3.c: Add

Re: [patch, testsuite] Fix PR 49191 with test for STRICT_ALIGNMENT

2011-06-08 Thread Jakub Jelinek
On Wed, Jun 08, 2011 at 01:16:26PM -0700, Steve Ellcey wrote: On Wed, 2011-06-08 at 22:08 +0200, Jakub Jelinek wrote: On Wed, Jun 08, 2011 at 01:03:53PM -0700, Steve Ellcey wrote: 2011-06-08 Steve Ellcey s...@cup.hp.com PR middle-end/49191 * lib/target-supports.exp

Re: [testsuite] skip ARM tests with conflicting options

2011-06-08 Thread Janis Johnson
On 06/08/2011 12:30 PM, Mike Stump wrote: On Jun 8, 2011, at 8:28 AM, Janis Johnson wrote: The big question is whether such a test should be run for all multilibs that might possibly pass the test, or only for default and for mulitlibs that provide the same options. Here, reasonable people

[pph] Test cleanup (issue4572050)

2011-06-08 Thread Lawrence Crowl
Update the test suite to reflect recent fixes and merges from trunk. The following changes happened in this patch. Conversion of a compilation failure to an assembly diff failure: remove the dg comments and add the comment // pph asm xdiff Conversion of a ICE to a BOGUS change the

Re: [patch, testsuite] Fix PR 49191 with test for STRICT_ALIGNMENT

2011-06-08 Thread Steve Ellcey
On Wed, 2011-06-08 at 22:19 +0200, Jakub Jelinek wrote: On Wed, Jun 08, 2011 at 01:16:26PM -0700, Steve Ellcey wrote: On Wed, 2011-06-08 at 22:08 +0200, Jakub Jelinek wrote: On Wed, Jun 08, 2011 at 01:03:53PM -0700, Steve Ellcey wrote: 2011-06-08 Steve Ellcey s...@cup.hp.com

Re: [pph] Clean up PPH tests (issue4572042)

2011-06-08 Thread Lawrence Crowl
On 6/7/11, Diego Novillo dnovi...@google.com wrote: After getting new failures due to an unrelated fix, I think this will be more trouble than it's worth. First, we can't get rid of the XPASSes, so those will always be noisy. Remove all the noise with: sed -e ' /-fpph-map=pph.map/

Re: [PATCH] c-pragma: adding a data field to pragma_handler

2011-06-08 Thread Pierre Vittet
I have written a test for this patch and run it (it works correctly). I guess there is no reason why it should not be accepted now. To recap, this patch add a void * data field to the pragma handler, allowing to pass extra data. If we want to use this field, we need to use the function

C++ PATCH for c++/49107 (excessive instantiation due to noexcept)

2011-06-08 Thread Jason Merrill
49107 is an issue with the addition of noexcept to the standard library causing extra instantiations, to the point that code that worked before doesn't work now due to circular dependency. Someone suggested that to avoid this problem we could defer instantiation of noexcept-specifiers until

C++ PATCH for C++0x failure on template/dtor6.C

2011-06-08 Thread Jason Merrill
Running the testsuite in C++0x mode caught this issue. Tested x86_64-pc-linux-gnu, applied to trunk. commit b531be395b85f7872c51e79ffb5db9a30a2a4f66 Author: Jason Merrill ja...@redhat.com Date: Thu Jun 2 14:35:59 2011 -0400 * semantics.c (potential_constant_expression_1): Handle

[pph] Front-end builtin macros (issue4575055)

2011-06-08 Thread Lawrence Crowl
Some builtin macros are more builtin than others. In particular, front-end builtin macros are weaker than libcpp builtin macros, and should not be processed with the libcpp builtin tools. Index: gcc/testsuite/ChangeLog.pph 2011-06-08 Lawrence Crowl cr...@google.com *

ObjC/ObjC++/testsuite: Remove unused method_get_* functions from next-encode-assist files

2011-06-08 Thread Nicola Pero
This patch removes a number of functions from the ObjC/ObjC++ testsuite next-encode-assist files which are never used in the testsuite ... and will never be, since they have been removed from the GNU runtime. ;-) The next-encode-assist provides a compatible implementation for the NeXT runtime

Re: [pph] Front-end builtin macros (issue4575055)

2011-06-08 Thread Diego Novillo
On Wed, Jun 8, 2011 at 15:09, Lawrence Crowl cr...@google.com wrote: Some builtin macros are more builtin than others.  In particular, front-end builtin macros are weaker than libcpp builtin macros, and should not be processed with the libcpp builtin tools. Nice! This fixes a whole chunk of C

Re: Dump before flag

2011-06-08 Thread Xinliang David Li
this is the patch that just removes the TODO_dump flag and forces it to dump. The original code cfun-last_verified = flags TODO_verify_all looks weird -- depending on TODO_dump is set or not, the behavior of the update is different (when no other todo flags is set). Ok for trunk? David On Wed,

libobjc: updated documentation for removal of traditional API

2011-06-08 Thread Nicola Pero
This patch (trivially) updates the libobjc documentation to state that the Traditional Objective-C runtime API is no longer available. Committed to trunk. Thanks Index: ChangeLog === --- ChangeLog (revision 174821) +++ ChangeLog

[patch committed] Fix PR target/49305

2011-06-08 Thread Kaz Kojima
Hi, My patch r174586 on trunk was a bit incomplete and causes PR49305 on sh2a target. The attached patch is to fix it. Tested on sh-elf and sh4-unknown-linux-gnu and applied on trunk. BTW, when looking into PR, I noticed that we forgot to utilize 32-bit mov.w instructions on sh2a while 32-bit

Re: ObjC/ObjC++/testsuite: Remove unused method_get_* functions from next-encode-assist files

2011-06-08 Thread Mike Stump
On Jun 8, 2011, at 3:21 PM, Nicola Pero wrote: This patch removes a number of functions from the ObjC/ObjC++ testsuite next-encode-assist files which are never used in the testsuite ... and will never be, since they have been removed from the GNU runtime. ;-) OK to commit ? Ok.

toplevel libgcc md-unwind-support.h change breaks build

2011-06-08 Thread David Miller
The bootstrap breaks in stage1 when trying to build libgcc for multilib (this is sparc-unknown-linux with --enable-targets=all): make[3]: Leaving directory `/home/davem/src/GIT/GCC/build-sparc-linux/gcc' mkdir -p -- sparc-unknown-linux-gnu/libgcc Checking multilib configuration for libgcc...

[pph] Remove some additional cruft we no longer use (issue4547091)

2011-06-08 Thread Diego Novillo
No functional changes in this patch. Additional cleanups after the various trunk merges and old code removal patches. Tested on x86_64. Committed to branch. Diego. Fix spurious differences with trunk. * c-parser.c (c_parser_declaration_or_fndef): Remove call to

[pph] Fix cp_debug_parser (issue4571058)

2011-06-08 Thread Diego Novillo
This fixes the ICE that Gab ran into while calling cp_debug_parser on a test case. We were not handling the case of empty token buffers and when the caller set the starting token to NULL. Additionally, the window of tokens to print was too small to be of any real use. * parser.c

Re-factor TS_* marking code (issue4572051)

2011-06-08 Thread Diego Novillo
This is a fairly simple re-factoring that simplifies the TS_* marking code, but it changes how the C++ front end marks the tree codes, so I'm looking for an explicit approval. There are no functional changes. The code in initialize_tree_contains_struct moves to mark_ts_structures_for and

Re: Re-factor TS_* marking code (issue4572051)

2011-06-08 Thread Jason Merrill
It seems like you're changing to mark a lot of things as TS_COMMON that previously were only TS_TYPED or even only TS_BASE. Jason

Fix PR49154-related SEGV in IRA. And ping.

2011-06-08 Thread Hans-Peter Nilsson
First, a ping for http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00136.html; updated regclass documentation. Ok? Second, I updated the CRIS port to fit the proposed documentation update (adding a class as the patch you sent, but more complete), with regtest results clean for revisions before the

Re: [pph] Stream TREE_TYPE for identifier node (issue4550121)

2011-06-08 Thread Jason Merrill
On 06/08/2011 03:31 PM, Steven Bosscher wrote: The rest are all in cp/. It looks like g++ uses TREE_TYPE as a cache for name lookups. Perhaps Jason can comment. Obviously not a front end I know very well, but let's look at them one at a time: cp/cp-tree.h: (DECL_CONV_FN_P (FN) ? TREE_TYPE

[patch] Committed: Fix the printable name typo in treestruct.def

2011-06-08 Thread Mingjie Xing
Hi, This small patch fixes the printable name typo in treestruct.def. I've committed as it is obvious. (http://gcc.gnu.org/ml/gcc/2011-06/msg00083.html) 2011-06-09 Mingjie Xing mingjie.x...@gmail.com * treestruct.def (TS_TYPE_DECL): Fix the printable name typo. Index: treestruct.def

Re: Remove SETJMP_VIA_SAVE_AREA support

2011-06-08 Thread Hans-Peter Nilsson
On Thu, 2 Jun 2011, Eric Botcazou wrote: This removes the (undocumented) support for SETJMP_VIA_SAVE_AREA from the compiler. Poison it (in system.h)? brgds, H-P