[google] Add intermediate text format for gcov (issue4595053)

2011-06-14 Thread Sharad Singhai
This patch adds an intermediate gcov text format which does not require source code. This format can be used by lcov or other tools. I have bootstrapped it on x86 and all tests pass. Okay for main? Thanks, Sharad 2011-06-13 Sharad Singhai sing...@google.com Google Ref 3

[google] Fix a bug leading to inconsistent comdat group in LIPO mode (issue4616041)

2011-06-14 Thread David Li
The patch will be committed to google/main to fix a problem in LIPO model that leads to 'reference to discarded comdat section' ld warning. The problem is caused by inconsistent comdat groups between primary and aux modules because thunks were skipped in aux module. 2011-06-14 David Li

Re: [patch, libgfortran] PR48906 Wrong rounding results with -m32

2011-06-14 Thread Thomas Henlich
On Tue, Jun 14, 2011 at 06:51, jerry DeLisle jvdeli...@charter.net wrote: It should be easy to implement: After the switch between F and E editing, we just need to shift the decimal point and decrement the exponent. No new rounding is required, because we keep the number of significant

Re: [PATCH, SMS] Fix violation of memory dependence

2011-06-14 Thread Ayal Zaks
Revital Eres revital.e...@linaro.org wrote on 13/06/2011 10:29:06 AM: From: Revital Eres revital.e...@linaro.org To: Ayal Zaks/Haifa/IBM@IBMIL Cc: gcc-patches@gcc.gnu.org, Patch Tracking patc...@linaro.org Date: 13/06/2011 10:29 AM Subject: [PATCH, SMS] Fix violation of memory dependence

Re: RFA (fold): PATCH for c++/49290 (folding *(T*)(ar+10))

2011-06-14 Thread Richard Guenther
On Mon, 13 Jun 2011, Jason Merrill wrote: On 06/13/2011 06:51 AM, Richard Guenther wrote: But I suppose you want the array-ref be folded to a constant eventually? Right. I'm not going to keep arguing about VIEW_CONVERT_EXPR, but that brings me back to my original question: is it OK to

Re: RFC: Fix GCSE exp_equiv_p on MEMs with different MEM_ATTRS (PR rtl-optimization/49390)

2011-06-14 Thread Richard Guenther
On Mon, 13 Jun 2011, Jakub Jelinek wrote: Hi! As the testcase shows, the http://gcc.gnu.org/ml/gcc-patches/2010-06/msg02945.html patch looks wrong, MEM_ATTRS matters quite a lot for the alias oracle, so ignoring it leads to miscompilations. Instead of just reverting the patch, this

Re: RFC: Fix GCSE exp_equiv_p on MEMs with different MEM_ATTRS (PR rtl-optimization/49390)

2011-06-14 Thread Bernd Schmidt
On 06/14/2011 10:43 AM, Richard Guenther wrote: The patch that reverted the MEM_ATTR comparison didn't come with a single testcase (ugh, I realize I approved it though ;)). Bernd, do you have any testcases? It was a missed-optimization problem, but I think it only showed up with a modified

Re: [PATCH] Only run pr48377.c testcase on i?86/x86_64

2011-06-14 Thread Jakub Jelinek
On Tue, Jun 14, 2011 at 11:10:13AM +0200, Eric Botcazou wrote: This limits this testcase to i?86/x86_64 (moving to gcc.target/ would be harder because it relies on all the weirdo vectorization options to be passed), because apparently on strict alignment targets we don't handle aligned (1)

Ping^5: Re: Updated^2: RFA: Fix middle-end/46500 (void * encapsulated)

2011-06-14 Thread Joern Rennecke
Except or the fortran/java bits (committed), this patch hasn't been reviewed for five weeks: http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00582.html

Re: RFC: Fix GCSE exp_equiv_p on MEMs with different MEM_ATTRS (PR rtl-optimization/49390)

2011-06-14 Thread Richard Guenther
On Tue, 14 Jun 2011, Bernd Schmidt wrote: On 06/14/2011 10:43 AM, Richard Guenther wrote: The patch that reverted the MEM_ATTR comparison didn't come with a single testcase (ugh, I realize I approved it though ;)). Bernd, do you have any testcases? It was a missed-optimization

Re: Ping: [testsuite]: Skip tests for targets with int 32 bits

2011-06-14 Thread Richard Guenther
On Mon, Jun 13, 2011 at 2:45 PM, Georg-Johann Lay a...@gjlay.de wrote: Ping #1 for: http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00746.html Ok. THanks, Richard. Georg-Johann Lay: This patch fixes testsuite failures because the testcases assume sizeof(int) = 4.        *

Re: Do not stream BINFO_VIRTUALs to ltrans unit

2011-06-14 Thread Richard Guenther
On Mon, Jun 13, 2011 at 2:54 PM, Jan Hubicka hubi...@ucw.cz wrote: Hi, by accident I noticed that BINFO_VIRTUALs streaming is really expensive. It about doubles amount of IL and types streamed by Mozilla. One obvious optimization is to not stream into ltrans unit where it is too late to do

Re: Ping^5: Re: Updated^2: RFA: Fix middle-end/46500 (void * encapsulated)

2011-06-14 Thread Richard Guenther
On Tue, Jun 14, 2011 at 11:40 AM, Joern Rennecke amyl...@spamcop.net wrote: Except or the fortran/java bits (committed), this patch hasn't been reviewed for five weeks: http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00582.html A patch doing s/CUMULATIVE_ARGS*/cumulative_args_t/ only is ok.

Re: [patch] Don't insert pattern statements into the code (was Fix PR tree-optimization/49318)

2011-06-14 Thread Ira Rosen
On 14 June 2011 13:02, Richard Guenther richard.guent...@gmail.com wrote: On Mon, Jun 13, 2011 at 2:43 PM, Ira Rosen ira.ro...@linaro.org wrote: On 10 June 2011 12:14, Richard Guenther richard.guent...@gmail.com wrote: In the end I think we should not generate the pattern stmt during pattern

Re: Ping: The TI C6X port

2011-06-14 Thread Bernd Schmidt
Ping^4 for the C6X port. Additional preliminary scheduler tweaks: http://gcc.gnu.org/ml/gcc-patches/2011-05/msg02408.html Allow alternatives in attr predicable: http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00094.html regrename across basic block boundaries:

Re: Ping^5: Re: Updated^2: RFA: Fix middle-end/46500 (void * encapsulated)

2011-06-14 Thread Joern Rennecke
Quoting Richard Guenther richard.guent...@gmail.com: On Tue, Jun 14, 2011 at 11:40 AM, Joern Rennecke amyl...@spamcop.net wrote: Except or the fortran/java bits (committed), this patch hasn't been reviewed for five weeks: http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00582.html A patch doing

Re: [patch] Don't insert pattern statements into the code (was Fix PR tree-optimization/49318)

2011-06-14 Thread Ira Rosen
On 14 June 2011 14:27, Richard Guenther richard.guent...@gmail.com wrote:   /* Mark the stmts that are involved in the pattern. */ -  gsi_insert_before (si, pattern_stmt, GSI_SAME_STMT);   set_vinfo_for_stmt (pattern_stmt,                      new_stmt_vec_info (pattern_stmt, loop_vinfo,

Re: [patch] Don't insert pattern statements into the code (was Fix PR tree-optimization/49318)

2011-06-14 Thread Richard Guenther
On Tue, Jun 14, 2011 at 1:38 PM, Ira Rosen ira.ro...@linaro.org wrote: On 14 June 2011 14:27, Richard Guenther richard.guent...@gmail.com wrote:   /* Mark the stmts that are involved in the pattern. */ -  gsi_insert_before (si, pattern_stmt, GSI_SAME_STMT);   set_vinfo_for_stmt

Re: Ping^5: Re: Updated^2: RFA: Fix middle-end/46500 (void * encapsulated)

2011-06-14 Thread Bernd Schmidt
On 06/14/2011 01:29 PM, Richard Guenther wrote: On Tue, Jun 14, 2011 at 1:16 PM, Joern Rennecke amyl...@spamcop.net wrote: Quoting Richard Guenther richard.guent...@gmail.com: On Tue, Jun 14, 2011 at 11:40 AM, Joern Rennecke amyl...@spamcop.net wrote: Except or the fortran/java bits

Re: Cgraph alias reorg 8/14 (ipa-cp and ipa-prop update)

2011-06-14 Thread Jan Hubicka
Index: ipa-cp.c === --- ipa-cp.c(revision 174905) +++ ipa-cp.c(working copy) @@ -818,7 +828,7 @@ ipcp_iterate_stage (void) /* Some lattices have changed from IPA_TOP to IPA_BOTTOM. This

Re: [PATCH, PR 49089] Don't split AVX256 unaligned loads by default on bdver1 and generic

2011-06-14 Thread H.J. Lu
On Tue, Jun 14, 2011 at 3:16 AM, Jakub Jelinek ja...@redhat.com wrote: On Tue, Jun 14, 2011 at 12:13:47PM +0200, Richard Guenther wrote: On Tue, Jun 14, 2011 at 1:59 AM, Fang, Changpeng changpeng.f...@amd.com wrote: The patch ( http://gcc.gnu.org/ml/gcc-patches/2011-02/txt00059.txt ) which

PING^4 APPROVED patch for AMD64 targets running GNU/kFreeBSD, anyone?

2011-06-14 Thread Robert Millan
This patch for AMD64 targets running GNU/kFreeBSD has been approved already, would anyone be so kind to commit it? I'm afraid I don't have write perms currently. See: http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00884.html Thank you very much :-) 2011/6/10 Richard Henderson r...@redhat.com:

Re: Ping^5: Re: Updated^2: RFA: Fix middle-end/46500 (void * encapsulated)

2011-06-14 Thread Bernd Schmidt
On 06/14/2011 02:53 PM, Joern Rennecke wrote: Quoting Bernd Schmidt ber...@codesourcery.com: I'm not getting the point of the use of attribute((transparent_union)). Without that attribute, lots of ABIs add a lot of overhead for function argument and return value passing. * These functions

Re: [Design notes, RFC] Address-lowering prototype design (PR46556)

2011-06-14 Thread Richard Guenther
On Fri, Jun 10, 2011 at 5:11 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: On Tue, 2011-06-07 at 16:49 +0200, Richard Guenther wrote: On Tue, Jun 7, 2011 at 4:14 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: snip Loss of aliasing information

[testsuite] Require lto support in g++.dg/torture/pr48954.C

2011-06-14 Thread Rainer Orth
The new g++.dg/torture/pr48954.C testcase FAILs on alpha-dec-osf5.1b: FAIL: g++.dg/torture/pr48954.C -O0 (test for excess errors) Excess errors: cc1plus: error: LTO support has not been enabled in this configuration The following test fixes this, tested with the appropriate runtest invocation,

Re: Dump before flag

2011-06-14 Thread Richard Guenther
On Fri, Jun 10, 2011 at 8:44 PM, Xinliang David Li davi...@google.com wrote: This is the revised patch as suggested. How does it look? } +static void +execute_function_dump (void *data ATTRIBUTE_UNUSED) function needs a comment. Ok with that change. Please always specify how you tested

Re: [Design notes, RFC] Address-lowering prototype design (PR46556)

2011-06-14 Thread William J. Schmidt
On Tue, 2011-06-14 at 15:39 +0200, Richard Guenther wrote: On Fri, Jun 10, 2011 at 5:11 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: On Tue, 2011-06-07 at 16:49 +0200, Richard Guenther wrote: On Tue, Jun 7, 2011 at 4:14 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote:

[build, libgcc] Correctly apply c_flags in shared-object.mk

2011-06-14 Thread Rainer Orth
When I first did a Solaris 11/x86 bootstrap with gld after checking in my ENABLE_EXECUTE_STACK patch, I found that several acats and gnat.dg tests were failing. This hadn't happened with Sun ld. Reghunting revealed that this had been introduced by that patch. Fortunately, not the code itself was

Re: Cgraph alias reorg 13/14 (disable inlining functions called once at -O0

2011-06-14 Thread Eric Botcazou
I think we also suggested at some point that -O1 optimizations shouldn't interfere with debugging too much. But if it is what we did before it's certainly fine. FWIW we have some evidences that -finline-functions-called-once really help at -O1 in terms of performances (with the 4.5 back-end)

Re: [Design notes, RFC] Address-lowering prototype design (PR46556)

2011-06-14 Thread Richard Guenther
On Tue, Jun 14, 2011 at 4:18 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: On Tue, 2011-06-14 at 15:39 +0200, Richard Guenther wrote: On Fri, Jun 10, 2011 at 5:11 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: On Tue, 2011-06-07 at 16:49 +0200, Richard Guenther wrote:

Re: RFC: Fix GCSE exp_equiv_p on MEMs with different MEM_ATTRS (PR rtl-optimization/49390)

2011-06-14 Thread Jakub Jelinek
On Tue, Jun 14, 2011 at 11:49:08AM +0200, Richard Guenther wrote: So I'd say we revert your patch for now and if somebody feels like implementing the above ... Ok, here is what I've bootstrapped/regtested on x86_64-linux and i686-linux and committed to trunk and 4.6 branch: 2011-06-14 Jakub

Unreviewed libffi patch

2011-06-14 Thread Rainer Orth
The following patch has remained unreviewed for a week: [libffi] Fix libffi.call/huge_struct.c on Tru64 UNIX http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00644.html It needs a libffi maintainer or global reviewer. Thanks. Rainer --

Re: PING^4 APPROVED patch for AMD64 targets running GNU/kFreeBSD, anyone?

2011-06-14 Thread Uros Bizjak
Hello! This patch for AMD64 targets running GNU/kFreeBSD has been approved already, would anyone be so kind to commit it? I'm afraid I don't have write perms currently. I have committed your patch to SVN mainline after bootstrapping it on x86_64-pc-linux-gnu. Thanks, Uros.

RFA PR middle-end/48770

2011-06-14 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 This version incorporates suggestions from Bernd. Basically we have reload1.c set reload_completed internally rather than deferring it into ira.c. That allows the call to reload() to return whether or not a DCE pass is desirable at the end of

Re: Unreviewed libffi patch

2011-06-14 Thread Andreas Tobler
On 14.06.11 17:22, Rainer Orth wrote: The following patch has remained unreviewed for a week: [libffi] Fix libffi.call/huge_struct.c on Tru64 UNIX http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00644.html It needs a libffi maintainer or global reviewer. From the test suite

[PATCH] Ensure incoming location is available in debug info for parameters (PR debug/49382)

2011-06-14 Thread Jakub Jelinek
Hi! As detailed in the PR, when gdb attempts to print originally passed values to parameters instead of current values using call site info, if the parameter is modified already before the first real instruction in the function, it will find there already the modified value. E.g. void foo (int x)

Re: [testsuite]: Skip tests for targets with int 32 bits

2011-06-14 Thread Mike Stump
On Jun 14, 2011, at 2:20 AM, Georg-Johann Lay wrote: testsuite/ * gcc.c-torture/execute/cmpsi-2.c: Undo 172757. Please always include the PR number in the changelog entries when there is one. This autolinks the work to the PR. Use the exact formatting found in the changelog file.

Re: [PATCH] Only run pr48377.c testcase on i?86/x86_64

2011-06-14 Thread Jakub Jelinek
On Tue, Jun 14, 2011 at 04:52:18PM +0200, Eric Botcazou wrote: Well, Steve has a patch for non_strict_align effective_target in http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00673.html (with s/strict_align/non_strict_align/g ), I was hoping it would be reviewed and I'd just adjust the

PATCH [6/n]: Prepare x32: PR middle-end/47449: Don't propagate hard register non-local goto save area

2011-06-14 Thread H.J. Lu
Hi, RTL-based forward propagation pass shouldn't propagate hard register. OK for trunk? Thanks. H.J. --- 2011-06-14 H.J. Lu hongjiu...@intel.com PR middle-end/47449 * fwprop.c (forward_propagate_subreg): Don't propagate hard register nor zero/sign extended hard

Re: Unreviewed libffi patch

2011-06-14 Thread Rainer Orth
Andrew Haley a...@redhat.com writes: On 06/14/2011 04:22 PM, Rainer Orth wrote: The following patch has remained unreviewed for a week: I think it wasn't cc'd to libffi-disc...@sourceware.org Right, I hadn't known/had forgotten about that since all my libffi fixes happen in GCC context. I'd

PATCH [7/n]: Prepare x32: Use Use long long builtin for x86-64

2011-06-14 Thread H.J. Lu
Hi, long may be 32bit for x86-64. But long long is always 64bit. This patch uses long long builtin for 64bit. OK for trunk? Thanks. H.J. --- 2011-06-14 H.J. Lu hongjiu...@intel.com * longlong.h (count_leading_zeros): Use long long builtin for x86-64.

RFA minor DF cleanup

2011-06-14 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 As I've noted in prior messages; I'm looking to improve our path isolation to improve code generation and reduce false positives from warnings. The patch that's been in my queue for some time now (and I suspect it's the final patch to our current

Re: [Design notes, RFC] Address-lowering prototype design (PR46556)

2011-06-14 Thread William J. Schmidt
On Tue, 2011-06-14 at 17:21 +0200, Richard Guenther wrote: On Tue, Jun 14, 2011 at 4:18 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: On Tue, 2011-06-14 at 15:39 +0200, Richard Guenther wrote: On Fri, Jun 10, 2011 at 5:11 PM, William J. Schmidt wschm...@linux.vnet.ibm.com

[google] backport r174930 to google/main

2011-06-14 Thread Xinliang David Li
Backported r174930 to google/main. David

Re: [PATCH] Ensure incoming location is available in debug info for parameters (PR debug/49382)

2011-06-14 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/14/11 09:51, Jakub Jelinek wrote: Hi! As detailed in the PR, when gdb attempts to print originally passed values to parameters instead of current values using call site info, if the parameter is modified already before the first real

Re: PATCH [1/n]: Prepare x32: PR middle-end/47364: internal compiler error: in emit_move_insn, at expr.c:3355

2011-06-14 Thread H.J. Lu
On Tue, Jun 14, 2011 at 8:11 AM, Richard Guenther richard.guent...@gmail.com wrote: On Sun, Jun 12, 2011 at 6:28 PM, H.J. Lu hjl.to...@gmail.com wrote: On Sun, Jun 12, 2011 at 7:33 AM, H.J. Lu hjl.to...@gmail.com wrote: On Sun, Jun 12, 2011 at 7:00 AM, H.J. Lu hjl.to...@gmail.com wrote: On

Re: Improve DSE in the presence of calls

2011-06-14 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 05/10/11 13:18, Easwaran Raman wrote: I am not sure I understand the problem here. If there is a wild read from asm, the instruction has the wild_read flag set. The if statement checks if that flag is set and if so it clears the bitmap - which

Re: Create common hooks structure shared between driver and cc1

2011-06-14 Thread Ian Lance Taylor
On Wed, May 25, 2011 at 12:21 PM, Joseph S. Myers jos...@codesourcery.com wrote: Here is a revised version of my patch http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01779.html to create the common hooks structure.  Tested in the same way as the original patch.  OK to commit? 2011-05-25  

Re: Move option-related hooks to common structure

2011-06-14 Thread Ian Lance Taylor
On Fri, May 27, 2011 at 9:13 AM, Joseph S. Myers jos...@codesourcery.com wrote: 2011-05-27  Joseph Myers  jos...@codesourcery.com        * target-def.h (TARGET_HAVE_NAMED_SECTIONS): Move to        common/common-target-def.h.        * target.def (default_target_flags, handle_option,        

C++ PATCH for c++/49290 (ICE regression on *(T*)(ar+10))

2011-06-14 Thread Jason Merrill
In this testcase, we were hitting an assert that I put in to make sure that fold_indirect_ref_1 was doing its job and folding everything that ought to be folded. But fold_indirect_ref_1 doesn't want to mess with type identity, so it can't fold if, say, the array element type has different

C++ PATCH for c++/49369 (wrong cv-quals on base member in unevaluated context)

2011-06-14 Thread Jason Merrill
We were forgetting to propagate cv-quals from 'this' to the result along one code path. Fixed by moving the cv-qual propagation up so it's shared by all code paths. Tested x86_64-pc-linux-gnu, applying to trunk and 4.6. commit a7eeb9dc7b67d159f46e9d8e7976332bd73332ca Author: Jason Merrill

Re: PATCH [7/n]: Prepare x32: Use Use long long builtin for x86-64

2011-06-14 Thread Uros Bizjak
On Tue, Jun 14, 2011 at 6:04 PM, H.J. Lu hongjiu...@intel.com wrote: long may be 32bit for x86-64. But long long is always 64bit.  This patch uses long long builtin for 64bit.  OK for trunk? Thanks. H.J. --- 2011-06-14  H.J. Lu  hongjiu...@intel.com        * longlong.h

C++ PATCH for c++/49117 (error message regression on conversion failure)

2011-06-14 Thread Jason Merrill
PR 49117 complains that the error message given on conversion failure regressed from 4.5 to 4.6 in that it no longer prints the source type. So I've added it back in. While I was at it, I've also tweaked the compiler to also print the typedef-stripped version of a type when appropriate, which

[testsuite] ARM tests should ignore warning about conflicting switches

2011-06-14 Thread Janis Johnson
Many tests in gcc.target/arm that specify -march= fail compilation when multilib flags include -mcpu= due to warnings about conflicts in switches, but then go on to pass the remainder of the test. This patch causes some of those tests to ignore that compiler warning; I'll get to the rest later.

Re: PATCH [1/n]: Add initial -x32 support

2011-06-14 Thread H.J. Lu
On Sun, Jun 05, 2011 at 12:54:41PM -0700, H.J. Lu wrote: Hi, I'd like to start submitting a series of patches to enable x32: https://sites.google.com/site/x32abi/ The GCC x32 branch is very stable. There are no unexpected failures in C, C++, Fortran and Objective C testsuites. SPEC CPU

[v2] Mark noexcept some destructors, add tests

2011-06-14 Thread Paolo Carlini
Hi, tested x86_64-linux, committed to mainline. Paolo. 2011-06-14 Paolo Carlini paolo.carl...@oracle.com * include/std/valarray (~valarray): Use noexcept. * include/bits/unique_ptr.h (~unique_ptr): Likewise. *

Re: PATCH [7/n]: Prepare x32: Use Use long long builtin for x86-64

2011-06-14 Thread H.J. Lu
On Tue, Jun 14, 2011 at 10:37 AM, Uros Bizjak ubiz...@gmail.com wrote: On Tue, Jun 14, 2011 at 6:04 PM, H.J. Lu hongjiu...@intel.com wrote: long may be 32bit for x86-64. But long long is always 64bit.  This patch uses long long builtin for 64bit.  OK for trunk? Thanks. H.J. ---

[testsuite] skip ARM tests if no THUMB support

2011-06-14 Thread Janis Johnson
Fix three ARM tests so they are skipped for multilibs that don't support THUMB. OK for trunk and 4.6? Janis 2011-06-14 Janis Johnson jani...@codesourcery.com * gcc.target/arm/pr45701-1.c: Require thumb support. * gcc.target/arm/pr45701-2.c: Likewise. *

[pph] pph_in_binding_level fixing shadowed_labels read (issue4589054)

2011-06-14 Thread Gabriel Charette
We weren't reading in shadowed labels properly. The local variable *sl also turned out to be useless, the compiler just didn't mention it until now as it was used by the bad VEC_iterate call. This doesn't fix any currently exposed pph bugs, but does help with me with the patch I'm currently

C++ PATCH for c++/49389 (wrong value category for .*)

2011-06-14 Thread Jason Merrill
If the object expression is an rvalue, the result should be as well. Tested x86_64-pc-linux-gnu, applying to trunk. commit 93619457bb3756b091d86a13d1aa72880bb1ac62 Author: Jason Merrill ja...@redhat.com Date: Mon Jun 13 22:19:24 2011 -0400 PR c++/49389 * typeck2.c

[google] Merge r173574 to google/gcc-4_6 to fix an incompatibility between C++98 and C++0x (issue4592057)

2011-06-14 Thread Jeffrey Yasskin
In C++0x mode, without this patch, calls to a user-defined trunc() function with an argument in namespace std and a parameter type that has an implicit conversion from the argument's type, cause infinite recursion in std::trunc(). This patch also includes

Re: [testsuite] ARM tests should ignore warning about conflicting switches

2011-06-14 Thread Mike Stump
On Jun 14, 2011, at 10:47 AM, Janis Johnson wrote: Many tests in gcc.target/arm that specify -march= fail compilation when multilib flags include -mcpu= due to warnings about conflicts in switches, but then go on to pass the remainder of the test. OK for trunk and 4.6? Ok. As usual, please

Re: [google] Merge r173574 to google/gcc-4_6 to fix an incompatibility between C++98 and C++0x (issue4592057)

2011-06-14 Thread Diego Novillo
On Tue, Jun 14, 2011 at 14:45, Jeffrey Yasskin jyass...@google.com wrote: In C++0x mode, without this patch, calls to a user-defined trunc() function with an argument in namespace std and a parameter type that has an implicit conversion from the argument's type, cause infinite recursion in

Re: Dump before flag

2011-06-14 Thread Xinliang David Li
Committed after Bootstrapping and regression testing on x86-64/linux. The follow up patch will come soon. Thanks, David On Tue, Jun 14, 2011 at 8:57 AM, Xinliang David Li davi...@google.com wrote: On Tue, Jun 14, 2011 at 6:58 AM, Richard Guenther richard.guent...@gmail.com wrote: On Fri, Jun

Re: [google] Merge r173574 to google/gcc-4_6 to fix an incompatibility between C++98 and C++0x (issue4592057)

2011-06-14 Thread Jeffrey Yasskin
On Tue, Jun 14, 2011 at 12:38 PM, Diego Novillo dnovi...@google.com wrote: On Tue, Jun 14, 2011 at 14:45, Jeffrey Yasskin jyass...@google.com wrote: In C++0x mode, without this patch, calls to a user-defined trunc() function with an argument in namespace std and a parameter type that has an

Re: [google] Merge r173574 to google/gcc-4_6 to fix an incompatibility between C++98 and C++0x (issue4592057)

2011-06-14 Thread Diego Novillo
On Tue, Jun 14, 2011 at 15:59, Jeffrey Yasskin jyass...@google.com wrote: It's already in trunk, so my impression was that it was going to be automatically merged to google/main. I only need a manual merge to get it into our release branches. Yeah, in this case it's not too different since

Re: RFA (fold): PATCH for c++/49290 (folding *(T*)(ar+10))

2011-06-14 Thread Mike Stump
On Jun 13, 2011, at 3:57 AM, Richard Guenther wrote: That's not exactly an example - I can't think of how you want or need to use VIEW_CONVERT_EXPRs to implement said divmod instruction or why you would need anything special for the _argument_ of said instruction. Oh, I completely

Re: [PATCH] sel-sched: Avoid placing bookkeeping code above a fence (PR49349)

2011-06-14 Thread Vladimir Makarov
On 06/14/2011 07:34 AM, Alexander Monakov wrote: Hello, Quoting myself from the PR audit trail, It's a rare bug in sel-sched: we fail to schedule some code in non-pipelining mode. The root cause is that we put bookkeeping instructions above a fence that is placed on the last insn (uncond.

[testsuite] (committed) let more ARM tests ignore warnings about conflicting switches

2011-06-14 Thread Janis Johnson
I made other changes to these tests earlier today, then the patch to ignore warnings for conflicting options was approved. I've committed this to trunk. Janis 2011-06-14 Janis Johnson jani...@codesourcery.com * gcc.target/arm/pr45701-1.c: Ignore warnings about conflicting switches.

[testsuite] skip ARM tests if no thumb2 support

2011-06-14 Thread Janis Johnson
These tests apparently require thumb2 support (I don't yet know much about ARM). OK for trunk, and later 4.6? Janis 2011-06-14 Janis Johnson jani...@codesourcery.com * gcc.target/arm/pr42879.c: Skip if no thumb2 support, ignore compiler warning about switch conflicts.

[PATCH, PR 48613] Don't stream jump functions if there are none

2011-06-14 Thread Martin Jambor
Hi, the patch below fixes PR 48613 which is an ICE with -O0 -findirect-inlining. Rather than adding optimize here and there, at this place we can easily see whether there is something to do or not by testing ipa_node_params_vector for NULL. And the flag-triggering combinations can -and are -

Re: [google] Add intermediate text format for gcov (issue4595053)

2011-06-14 Thread शरद सिंघई
Sorry, Rietveld didn't send out the updated patch along with my mail. Here it is. Sharad 2011-06-14 Sharad Singhai sing...@google.com Google Ref 3 * doc/gcov.texi: Document gcov intermediate format. * gcov.c (get_gcov_file_intermediate_name): New function.

Re: Ping: The TI C6X port

2011-06-14 Thread Vladimir Makarov
On 06/06/2011 07:26 AM, Bernd Schmidt wrote: Ping^3 for the C6X port. Now with extra patches: Additional preliminary scheduler tweaks: http://gcc.gnu.org/ml/gcc-patches/2011-05/msg02408.html It is ok for me. Thanks, Bernd. Allow alternatives in attr predicable:

Re: [Patch, AVR]: Fix PR46779

2011-06-14 Thread Georg-Johann Lay
Denis Chertykov schrieb: 2011/6/14 Georg-Johann Lay a...@gjlay.de: Denis Chertykov schrieb: 2011/6/13 Georg-Johann Lay a...@gjlay.de: So you think is is pointless/discouraged to give a more realistic description of AVR addressing be means of MODE_CODE_BASE_REG_CLASS (instead of

[RFA/libiberty] Darwin has case-insensitive filesystems

2011-06-14 Thread Joel Brobecker
Hello, HFS+, the FS on Darwin, is case insensitive. So this patch adjusts filename_cmp.c to ignore the casing when comparing filenames on Darwin. This is visible in GDB when trying to break on a file whose name is, say 'Mixed_Case.adb', but was compiled using 'mixed_case.adb' as the filename.

Re: [RFA/libiberty] Darwin has case-insensitive filesystems

2011-06-14 Thread DJ Delorie
Looks OK to me.

Re: [RFA/libiberty] Darwin has case-insensitive filesystems

2011-06-14 Thread Andrew Pinski
On Tue, Jun 14, 2011 at 2:33 PM, Joel Brobecker brobec...@adacore.com wrote: Hello, HFS+, the FS on Darwin, is case insensitive. So this patch adjusts filename_cmp.c to ignore the casing when comparing filenames on Darwin. This is wrong as not all FSs are case insensitive. In fact HFS+ can

PING: PATCH: PR other/49325: Incorrect target HAVE_INITFINI_ARRAY check

2011-06-14 Thread H.J. Lu
On Wed, Jun 8, 2011 at 9:49 AM, H.J. Lu hongjiu...@intel.com wrote: 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

[v3] use noexcept in pointer_traits

2011-06-14 Thread Jonathan Wakely
I missed the noexcept qualifier off this function when I added it recently. 2011-06-14 Jonathan Wakely jwakely@gmail.com * include/bits/ptr_traits.h (pointer_traitsT*::pointer_to): Use noexcept. Tested x86_64-linux and committed to trunk Index: include/bits/ptr_traits.h

Re: [PATCH, PR 48613] Don't stream jump functions if there are none

2011-06-14 Thread Jan Hubicka
Hi, the patch below fixes PR 48613 which is an ICE with -O0 -findirect-inlining. Rather than adding optimize here and there, at this place we can easily see whether there is something to do or not by testing ipa_node_params_vector for NULL. And the flag-triggering combinations can -and

Re: [Patch, AVR]: Fix PR46779

2011-06-14 Thread Richard Henderson
On 06/14/2011 02:29 PM, Georg-Johann Lay wrote: http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01029.html It does look like a step in the right direction. I tested on some handcrafted examples and on the code attached to PR46278. The generated code looked very good and so I started regression

Re: [PATCH, PR 49089] Don't split AVX256 unaligned loads by default on bdver1 and generic

2011-06-14 Thread H.J. Lu
On Tue, Jun 14, 2011 at 3:41 PM, Fang, Changpeng changpeng.f...@amd.com wrote: It probably should go to the 4.6 branch as well. H.J. Lu's original patch that splits unaligned load and store was checked in gcc 4.7 trunk. We found that,  splitting unaligned store is beneficial to bdver1,

Re: Dump only functions with name matching patterns

2011-06-14 Thread Xinliang David Li
Attached the patch. David On Tue, Jun 14, 2011 at 4:21 PM, Xinliang David Li davi...@google.com wrote: This is the second (hopefully the last in the series of dumper changes) follow-up patch. It adds a control so that verbosity of the dump can be greatly reduced (and hopefully simplify gcc

Re: [PATCH, PR 49089] Don't split AVX256 unaligned loads by default on bdver1 and generic

2011-06-14 Thread H.J. Lu
On Tue, Jun 14, 2011 at 4:59 PM, Fang, Changpeng changpeng.f...@amd.com wrote: So, is it OK to commit this patch to trunk, and H.J's original patch + this to 4.6 branch? I have no problems on -mtune=Bulldozer.  But I object -mtune=generic change and did suggest a different approach for