Re: [PATCH, i386, middle-end, tessuite] Intel TSX's HLE.

2012-04-23 Thread Kirill Yukhin
Otherwise, OK as far as x86 is concerned, but you will need separate approval for middle-end part. Hi guys, this is a ping Could anybody from middle-end please have a look? Thanks, K

Ping #1: [Patch,AVR]: Auto-generate all -mmcu= options in documentation.

2012-04-23 Thread Georg-Johann Lay
Denis Chertykov schrieb: This patch adds a new file ./gcc/doc/avr-mmcu.texi that lists all valid -mmcu= settings and replaces the respective text in invoke.texi by @include avr-mmcu.texi Up to now, there is no complete list of -mmcu= options, and a list is hard to maintain by hand because it

Re: [google/integration] Extend C++11 UDLs to be compatible with inttypes.h macros (issue6104051)

2012-04-23 Thread Jonathan Wakely
On 23 April 2012 04:10, Jeffrey Yasskin wrote: Could you try to get this into mainline instead of just the google branches? In http://gcc.gnu.org/PR52538, Jonathan sounded like he'd consider accepting it. I think it's useful, but I can't approve front-end patches.

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-23 Thread Uros Bizjak
On Tue, Apr 3, 2012 at 9:47 PM, Sriraman Tallam tmsri...@google.com wrote: i386 maintainers -  Is this patch ok? Has the community reached the consensus on how this kind of functionality has to be implemented? I have followed the discussion a bit, but IIRC, there was no clear decision. Without

Re: [PATCH, i386]: FIX PR 52698, reload failure with complex address

2012-04-23 Thread Uros Bizjak
On Tue, Mar 27, 2012 at 7:20 PM, Richard Henderson r...@redhat.com wrote: On 03/27/12 09:37, Uros Bizjak wrote: Now, in this particular case, there might be another option to avoid this hassle completely:  I understand that this UNSPEC is simply a magic marker to make the address use the

Re: [PATCH] Dissociate store_expr's temp from exp so that it is not marked as addressable

2012-04-23 Thread Eric Botcazou
2012-04-16 Martin Jambor mjam...@suse.cz * expr.c (expand_expr_real_1): Remove setting parent's alias set for temporaries created for a bitfield (reverting revision 122014). OK, thanks. -- Eric Botcazou

Re: [PATCH] Fix PR44214

2012-04-23 Thread Richard Guenther
On Fri, 20 Apr 2012, William J. Schmidt wrote: On Fri, 2012-04-20 at 11:32 -0700, H.J. Lu wrote: On Thu, Apr 19, 2012 at 6:58 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: This enhances constant folding for division by complex and vector constants. When -freciprocal-math

Re: Fix vectorizer ICE when building Mozilla

2012-04-23 Thread Richard Guenther
On Mon, 23 Apr 2012, Jan Hubicka wrote: Hi, when building Mozilla we ICE in expr_invariant_in_loop_p walking ARRAY_REF, because one of the fields is NULL (that is allowed) Bootstrapped/regtested x86_64-linux, OK? Ok with breaking the like before - lines need to be less than 80 columns.

Re: [PATCH] Fix ICE in adjust_bool_pattern (PR tree-optimizations/52891)

2012-04-23 Thread Richard Guenther
On Fri, 20 Apr 2012, Jakub Jelinek wrote: Hi! This patch fixes ICE in adjust_bool_pattern, trueval in that case has the vector type's element type, which is full precision of mode, so when we see a smaller precision, we need to use build_nonstandard_type. Bootstrapped/regtested on

Re: PING: [PATCH] Fix PRs c/52283/37985

2012-04-23 Thread Richard Guenther
On Fri, Apr 20, 2012 at 5:23 PM, Greta Yorsh greta.yo...@arm.com wrote: Here is a patch to fix the failing test gcc.dg/pr52283.c. Adding the missing dg-warning and dg-options. OK? Ok. Thanks, Richard. gcc/testsuite/ChangeLog 2012-04-20  Greta Yorsh  greta.yo...@arm.com        *

Re: Patch to fix compiler ICE due to bad PRE

2012-04-23 Thread Richard Guenther
On Fri, Apr 20, 2012 at 6:16 PM, Xinliang David Li davi...@google.com wrote: On Fri, Apr 20, 2012 at 4:50 AM, Richard Guenther richard.guent...@gmail.com wrote: On Fri, Apr 20, 2012 at 12:07 PM, Richard Guenther richard.guent...@gmail.com wrote: On Fri, Apr 20, 2012 at 10:41 AM, Richard

Re: [PATCH, LTO, 4.6] Backport Honza's PR48246 patch

2012-04-23 Thread Richard Guenther
On Fri, Apr 20, 2012 at 10:15 PM, Peter Bergner berg...@vnet.ibm.com wrote: We're still hitting the overzealous assert identified in PR49246 that is looking for empty LTO partitions, even after the change to a checking assert. How about we backport Honza's mainline patch that stops the creation

Re: [PATCH] Implement (a B) OP0 (c B) into (a OP0 c) B simplifying into forwprop

2012-04-23 Thread Richard Guenther
On Sat, Apr 21, 2012 at 4:26 AM, Andrew Pinski pins...@gmail.com wrote: Hi,  I noticed a missed simple optimization on the tree level where the and expression could be commoned out. This patch implements the optimization in tree-ssa-forwprop.c.   I thought it would be good to get it in even

Re: [PATCH] Fix PR 33512 Folding of x ((~x) | y) into x y on the tree level

2012-04-23 Thread Richard Guenther
On Sat, Apr 21, 2012 at 6:06 AM, Andrew Pinski andrew.pin...@caviumnetworks.com wrote: This time with the patch and describing what the bug was.  The problem was defcodefor_name does not always set arg1 and arg2.  This fixes it so it is always set to NULL if they don't exist. Ok with ... +

Re: [PATCH] genattrtab: avoid NULL-deref on error

2012-04-23 Thread Richard Guenther
On Sat, Apr 21, 2012 at 5:53 PM, Jim Meyering j...@meyering.net wrote: I see that no one responded when I posted this in February. Ok to commit, now? Ok. Thanks, Richard. 2012-02-24  Jim Meyering  meyer...@redhat.com        * genattrtab.c (gen_attr): Avoid NULL-deref after diagnosing      

Re: Remove SETJMP_VIA_SAVE_AREA support

2012-04-23 Thread Eric Botcazou
You do know that at least rtl hoisting is dependent upon calls_setjmp being set, right? :-( Sure, clearly a straightforward dependency. ;-) This part breaks my port. I think you read the comment and thought it was exhaustive, I don't believe it is. I think it should, though.

[PATCH] Fix PR53060

2012-04-23 Thread Richard Guenther
This fixes a typo. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2012-04-23 Richard Guenther rguent...@suse.de PR c/53060 * c-typeck.c (build_binary_op): Fix typo. * gcc.dg/pr53060.c: New testcase. Index: gcc/c-typeck.c

[testsuite] PR 53046

2012-04-23 Thread Paolo Carlini
Hi, tested x86_64-linux, committed. Paolo. /// 2012-04-23 Chris Jefferson ch...@bubblescope.net PR testsuite/53046 * testsuite/25_algorithms/stable_partition/mem_check.cc: Fix size of array A. *

Re: [PATCH] Support for known unknown alignment

2012-04-23 Thread Richard Guenther
On Fri, Apr 20, 2012 at 8:41 PM, Jay Foad jay.f...@gmail.com wrote: On 20 April 2012 16:54, Martin Jambor mjam...@suse.cz wrote: two days ago I talked to Richi on IRC about the functions to determine the expected alignment of objects and pointers we have and he suggested that

Re: [tpf] update for latest build system

2012-04-23 Thread Andreas Krebbel
On Thu, Mar 22, 2012 at 08:38:41PM -0400, DJ Delorie wrote: Per request from IBM... * config/s390/s390.h (LINK_SPEC): Remove, no longer needed. (LIBSTDCXX): Change to CPP2. Ok. Bye, -Andreas-

[PATCH] Fix PR53070

2012-04-23 Thread Richard Guenther
if-conversion fails to free a basic-block predicates (notably that of the latch). Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2012-04-23 Richard Guenther rguent...@suse.de PR tree-optimization/53070 * tree-if-conv.c (combine_blocks): Free

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-23 Thread Gabriel Dos Reis
On Sun, Apr 22, 2012 at 10:59 PM, Ollie Wild a...@google.com wrote: Add new option, -Wreserved-user-defined-literal. Just shorten it to -Wliteral-suffix. This option, which is enabled by default, causes the preprocessor to warn when a string or character literal is followed by a ud-suffix

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-23 Thread Ollie Wild
On Mon, Apr 23, 2012 at 7:10 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Sun, Apr 22, 2012 at 10:59 PM, Ollie Wild a...@google.com wrote: Add new option, -Wreserved-user-defined-literal. Just shorten it to -Wliteral-suffix. I chose -Wreserved-user-defined-literal because

[PING] reload: Try alternative with swapped operands before going to the next

2012-04-23 Thread Andreas Krebbel
Hi, I've re-tested the patch from: http://gcc.gnu.org/ml/gcc-patches/2011-11/msg01819.html on s390x and x86_64. Ok for mainline? Bye, -Andreas-

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-23 Thread Gabriel Dos Reis
On Mon, Apr 23, 2012 at 7:30 AM, Ollie Wild a...@google.com wrote: On Mon, Apr 23, 2012 at 7:10 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Sun, Apr 22, 2012 at 10:59 PM, Ollie Wild a...@google.com wrote: Add new option, -Wreserved-user-defined-literal. Just shorten it to

Re: [PATCH, LTO, 4.6] Backport Honza's PR48246 patch

2012-04-23 Thread Peter Bergner
On Mon, 2012-04-23 at 11:33 +0200, Richard Guenther wrote: On Fri, Apr 20, 2012 at 10:15 PM, Peter Bergner berg...@vnet.ibm.com wrote: We're still hitting the overzealous assert identified in PR49246 that is looking for empty LTO partitions, even after the change to a checking assert.

Re: [PATCH] Support for known unknown alignment

2012-04-23 Thread Martin Jambor
Hi, On Mon, Apr 23, 2012 at 12:50:51PM +0200, Richard Guenther wrote: On Fri, 20 Apr 2012, Martin Jambor wrote: Hi, two days ago I talked to Richi on IRC about the functions to determine the expected alignment of objects and pointers we have and he suggested that

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-23 Thread Ollie Wild
On Mon, Apr 23, 2012 at 7:53 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Mon, Apr 23, 2012 at 7:30 AM, Ollie Wild a...@google.com wrote: Do you still want me to shorten in? yes. Done. Updated patch attached. Ollie commit 3f53671fb7fc7811277f047e7914f78e127031a6 Author:

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-23 Thread Gabriel Dos Reis
On Mon, Apr 23, 2012 at 8:26 AM, Ollie Wild a...@google.com wrote: On Mon, Apr 23, 2012 at 7:53 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Mon, Apr 23, 2012 at 7:30 AM, Ollie Wild a...@google.com wrote: Do you still want me to shorten in? yes. Done.  Updated patch

Re: [PATCH] Support for known unknown alignment

2012-04-23 Thread Richard Guenther
On Mon, 23 Apr 2012, Martin Jambor wrote: Hi, On Mon, Apr 23, 2012 at 12:50:51PM +0200, Richard Guenther wrote: On Fri, 20 Apr 2012, Martin Jambor wrote: Hi, two days ago I talked to Richi on IRC about the functions to determine the expected alignment of objects and pointers

Re: [PATCH] reload: Try alternative with swapped operands before going to the next

2012-04-23 Thread Uros Bizjak
Hello! 2011-11-17 Andreas Krebbel andreas.kreb...@de.ibm.com * reload.c (find_reloads): Change the loop nesting when trying an alternative with swapped operands. I would just like to point out that constran_operands will have problems to re-recognize alternative with swapped

Re: [PATCH] Support for known unknown alignment

2012-04-23 Thread Jay Foad
On 23 April 2012 14:30, Richard Guenther rguent...@suse.de wrote: Well, CCP simply tracks known-bits and derives the alignment value from that.  If tem -tem computes as zero that means val-mask.low is all zeros. Doesn't that mean that all bits are known? So you could set: pi-align = 1 32;

Re: [PATCH] Fix PR44214

2012-04-23 Thread William J. Schmidt
On Mon, 2012-04-23 at 11:09 +0200, Richard Guenther wrote: On Fri, 20 Apr 2012, William J. Schmidt wrote: On Fri, 2012-04-20 at 11:32 -0700, H.J. Lu wrote: On Thu, Apr 19, 2012 at 6:58 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: This enhances constant folding for

Re: [PATCH] Support for known unknown alignment

2012-04-23 Thread Richard Guenther
On Mon, 23 Apr 2012, Jay Foad wrote: On 23 April 2012 14:30, Richard Guenther rguent...@suse.de wrote: Well, CCP simply tracks known-bits and derives the alignment value from that.  If tem -tem computes as zero that means val-mask.low is all zeros. Doesn't that mean that all bits are

[PATCH] Cleanup vectorizer condition generation

2012-04-23 Thread Richard Guenther
The vectorizer ends up emitting tons of D.1234 = 0; if (D.1234 != 0) tests. Those do not get cleaned up by the cfg-cleanup run after it but only during the next constant propagation pass. Easily fixed by avoiding not necessary temporary stmts as follows. Bootstrapped on

Re: divide 64-bit by constant for 32-bit target machines

2012-04-23 Thread Andrew Haley
On 04/20/2012 01:57 PM, Dinar Temirbulatov wrote: Here is the patch that adds support for divide 64-bit by constant for 32-bit target machines, this patch was tested on arm-7a with no new regressions, also I am not sure on how to avoid for example i686 targets since div operation there is fast

Re: RFA: consolidate DWARF strings into libiberty

2012-04-23 Thread Tom Tromey
Tom Here is a new patch for gcc. Tom I still haven't updated the src side, but there's little to do there Tom that isn't already done in this patch. Tom Ok? Tom Ping. Tom Ping. This is the third ping. Please review the patch. There are two choices: 1. Approve the original patch, adding a

Re: [PATCH] reload: Try alternative with swapped operands before going to the next

2012-04-23 Thread Ulrich Weigand
Uros Bizjak wrote: 2011-11-17 Andreas Krebbel andreas.kreb...@de.ibm.com * reload.c (find_reloads): Change the loop nesting when trying an alternative with swapped operands. I would just like to point out that constran_operands will have problems to re-recognize alternative

Re: [PATCH, i386]: FIX PR 52698, reload failure with complex address

2012-04-23 Thread Ulrich Weigand
Uros Bizjak wrote: On Tue, Mar 27, 2012 at 7:20 PM, Richard Henderson r...@redhat.com wrote: On 03/27/12 09:37, Uros Bizjak wrote: Now, in this particular case, there might be another option to avoid this hassle completely: =A0I understand that this UNSPEC is simply a magic marker to

Re: [PATCH v2] ARM: Use different linker path for hardfloat ABI

2012-04-23 Thread Richard Earnshaw
On 22/04/12 23:20, Michael Hope wrote: Change the dynamic linker path for ARM hard float executables. Matches the path discussed and agreed on last week[1]. Carlos will follow up with the matching patch to GLIBC[2]. I'm happy to if he's busy. OK for trunk? -- Michael [1]

Re: RFC reminder: an alternative -fsched-pressure implementation

2012-04-23 Thread Ulrich Weigand
Vladimir Makarov wrote: I have a mixed feeling with the patch. I've tried it on SPEC2000 on x86/x86-64 and ARM. Model algorithm generates bigger code up to 3.5% (SPECFP on x86), 2% (SPECFP on 86-64), and 0.23% (SPECFP on ARM) in comparison with the current algorithm. It is slower too.

Re: [H8300] Use braced strings in MD

2012-04-23 Thread Jeff Law
On 04/23/2012 12:17 AM, Naveen H. S wrote: Some of my colleagues here do have checkin rights but not me. I have now requested for checkin after approval rights. Is this ok? Great. I approved it this morning and it looks like your account was created soon thereafter. Please add yourself to the

Re: [PATCH] recog: Record also added clobbers in insn_invalid_p

2012-04-23 Thread Eric Botcazou
2012-04-20 Andreas Krebbel andreas.kreb...@de.ibm.com * recog.c (insn_invalid_p): Add IN_GROUP parameter and use validate_change to add clobbers if IN_GROUP is nonzero. (verify_changes): Set IN_GROUP parameter to true. * recog.h (insn_invalid_p): Add IN_GROUP

Re: [PATCH] reload: Try alternative with swapped operands before going to the next

2012-04-23 Thread Uros Bizjak
On Mon, Apr 23, 2012 at 5:14 PM, Ulrich Weigand uweig...@de.ibm.com wrote: 2011-11-17  Andreas Krebbel  andreas.kreb...@de.ibm.com     * reload.c (find_reloads): Change the loop nesting when trying an     alternative with swapped operands. I would just like to point out that

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-23 Thread Sriraman Tallam
Hi, On Mon, Apr 23, 2012 at 1:19 AM, Uros Bizjak ubiz...@gmail.com wrote: On Tue, Apr 3, 2012 at 9:47 PM, Sriraman Tallam tmsri...@google.com wrote: i386 maintainers -  Is this patch ok? Has the community reached the consensus on how this kind of functionality has to be implemented? I have

libgo patch committed: Make sure directory exists

2012-04-23 Thread Ian Lance Taylor
This patch to the libgo Makefile makes sure that the directory where we place a .o file exists before we try to use it. Bootstrapped on x86_64-unknown-linux-gnu. Committed to mainline and 4.7 branch. Ian diff -r e746e3770f15 libgo/Makefile.am --- a/libgo/Makefile.am Sun Apr 22 13:07:04 2012

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-23 Thread Sriraman Tallam
On Mon, Apr 23, 2012 at 1:19 AM, Uros Bizjak ubiz...@gmail.com wrote: On Tue, Apr 3, 2012 at 9:47 PM, Sriraman Tallam tmsri...@google.com wrote: i386 maintainers -  Is this patch ok? Has the community reached the consensus on how this kind of functionality has to be implemented? I have

Re: [PATCH] reload: Try alternative with swapped operands before going to the next

2012-04-23 Thread Ulrich Weigand
Uros Bizjak wrote: I don't have any problems with proposed order, but in my failed attempt to convert x86 to post-reload compare elimination, constrain_operands failed to recognize generated combined add+compare pattern, when commutative matched operands were swapped (e.g. the perfectly

Re: Symbol table 13/many: reachability code rewrite

2012-04-23 Thread H.J. Lu
On Sun, Apr 22, 2012 at 2:16 PM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this is second part of cleanup of the callgraph/varpool reachability code. As I wrote in previous email, the callgraph was originally written with reachability code built in. This code was used by non-unit-at-a-time (to

Re: RFC reminder: an alternative -fsched-pressure implementation

2012-04-23 Thread Vladimir Makarov
On 04/23/2012 11:42 AM, Ulrich Weigand wrote: Vladimir Makarov wrote: I have a mixed feeling with the patch. I've tried it on SPEC2000 on x86/x86-64 and ARM. Model algorithm generates bigger code up to 3.5% (SPECFP on x86), 2% (SPECFP on 86-64), and 0.23% (SPECFP on ARM) in comparison with the

Re: [RFC] improve caret diagnostics for overload failures

2012-04-23 Thread Manuel López-Ibáñez
So, apart from the type of the flag, are there any other comments on the patch? Is the approach acceptable? On 21 April 2012 17:51, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Sat, Apr 21, 2012 at 9:42 AM, Jakub Jelinek ja...@redhat.com wrote: On Sat, Apr 21, 2012 at 04:26:32PM

[PATCH][Cilkplus] Fix a 2.21 assembler bug

2012-04-23 Thread Iyer, Balaji V
Hello Everyone, This patch is for the Cilkplus branch affecting the C and C++ compilers. This patch fixes a bug that occurs when low-cost annotations is enabled. The bug only occurs when the user is using an assembler later than 2.20. Thanks, Balaji V. Iyer. Index: gcc/final.c

compare_tests does not work with symlinked directories

2012-04-23 Thread Manuel López-Ibáñez
The version of 'find' in the GCC farm does not traverse symlinked directories unless they have a trailing slash. OK? 2012-04-23 Manuel López-Ibáñez m...@gcc.gnu.org * contrib/compare_tests: Append '/' to make 'find' traverse symlinked directories. compare_tests.diff

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-23 Thread Uros Bizjak
On Mon, Apr 23, 2012 at 6:59 PM, Sriraman Tallam tmsri...@google.com wrote: i386 maintainers -  Is this patch ok? Has the community reached the consensus on how this kind of functionality has to be implemented? I have followed the discussion a bit, but IIRC, there was no clear decision.

Re: thoughts on libatomic

2012-04-23 Thread Richard Henderson
On 04/16/12 11:15, Torvald Riegel wrote: Richard, Andrew, I had a look at libatomic yesterday, focusing primarily on synchronization issues in it. Here are some comments. And I think there is a bug in it too. Notes are in no particular order. Let me know what you think. - seq_cst

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-23 Thread Uros Bizjak
On Mon, Apr 23, 2012 at 6:59 PM, Sriraman Tallam tmsri...@google.com wrote:       * config/i386/i386.c (build_processor_features_struct): New function.       (build_processor_model_struct): New function.       (make_var_decl): New function.       (get_field_from_struct): New function.      

Re: [google/integration] Extend C++11 UDLs to be compatible with inttypes.h macros (issue6104051)

2012-04-23 Thread Diego Novillo
On Sun, Apr 22, 2012 at 22:54, Ollie Wild a...@google.com wrote: Add new option, -Wreserved-user-defined-literal. This option, which is enabled by default, causes the preprocessor to warn when a string or character literal is followed by a ud-suffix which does not begin with an underscore.  

Re: [PATCH] reload: Try alternative with swapped operands before going to the next

2012-04-23 Thread Uros Bizjak
On Mon, Apr 23, 2012 at 7:04 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Uros Bizjak wrote: I don't have any problems with proposed order, but in my failed attempt to convert x86 to post-reload compare elimination, constrain_operands failed to recognize generated combined add+compare

Re: [google/integration] Extend C++11 UDLs to be compatible with inttypes.h macros (issue6104051)

2012-04-23 Thread Ollie Wild
On Mon, Apr 23, 2012 at 2:39 PM, Diego Novillo dnovi...@google.com wrote: This would be for google/main, right?  It does not seem fit for google/integration. It needs to be in google/integration because it blocks nearly all of our code from compiling with -std=gnu++11, and there's no way to

Re: [PATCH v2] ARM: Use different linker path for hardfloat ABI

2012-04-23 Thread Carlos O'Donell
On Sun, Apr 22, 2012 at 6:20 PM, Michael Hope michael.h...@linaro.org wrote: Change the dynamic linker path for ARM hard float executables. Matches the path discussed and agreed on last week[1].  Carlos will follow up with the matching patch to GLIBC[2].  I'm happy to if he's busy. I'm

Re: [PATCH, i386]: FIX PR 52698, reload failure with complex address

2012-04-23 Thread Richard Henderson
On 04/23/12 08:27, Ulrich Weigand wrote: Ulrich, can you please provide some guidelines on how you think this proposed functionality should be implemented? Well, you do not *have* to have a keyword for a special address space. One possible implementation might be: - You define an

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-23 Thread H.J. Lu
On Mon, Apr 23, 2012 at 9:59 AM, Sriraman Tallam tmsri...@google.com wrote: Hi, On Mon, Apr 23, 2012 at 1:19 AM, Uros Bizjak ubiz...@gmail.com wrote: On Tue, Apr 3, 2012 at 9:47 PM, Sriraman Tallam tmsri...@google.com wrote: i386 maintainers -  Is this patch ok? Has the community reached

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-23 Thread Sriraman Tallam
On Mon, Apr 23, 2012 at 12:16 PM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Apr 23, 2012 at 6:59 PM, Sriraman Tallam tmsri...@google.com wrote: i386 maintainers -  Is this patch ok? Has the community reached the consensus on how this kind of functionality has to be implemented? I have

Go patch committed: Fix negative float zero constant

2012-04-23 Thread Ian Lance Taylor
This patch to the Go frontend corrects the handling of negative zero when using untyped constants. Go's untyped constants are ideal floating point numbers, and don't have negative zero or infinity or NaN. The Go frontend was incorrectly representing the negative of zero as a negative zero, since

Re: [google/integration] Extend C++11 UDLs to be compatible with inttypes.h macros (issue6104051)

2012-04-23 Thread Diego Novillo
On 4/23/12 3:53 PM, Ollie Wild wrote: On Mon, Apr 23, 2012 at 2:39 PM, Diego Novillodnovi...@google.com wrote: This would be for google/main, right? It does not seem fit for google/integration. It needs to be in google/integration because it blocks nearly all of our code from compiling

Re: thoughts on libatomic

2012-04-23 Thread Andrew MacLeod
On 04/23/2012 03:29 PM, Richard Henderson wrote: - load_n.c: - I'm concerned about the CAS on read-only mprotected pages? Why again do we think this is safe? Does the standard explicitly allow this? Or should we just use a lock in this case? Andrew, you had a bit of

[lra] patch to bootstrap GCC with LRA on PA-RISC

2012-04-23 Thread Vladimir Makarov
The following patch makes GCC bootstrap on PA-RISC successful. Committed as rev. 186724. 2012-04-23 Vladimir Makarov vmaka...@redhat.com * targhooks.h (default_different_addr_displacement_p): Declare. * targhooks.c (default_different_addr_displacement_p): New

[PATCH, i386]: Add additional r/r/0 alternative to all add patterns

2012-04-23 Thread Uros Bizjak
Hello! Based on discussion at [1], there was an oversight when r/r/0 alternative was added to add*_1 patterns. This alternative should also be added to other add*_X patterns, otherwise combined add+compare patterns won't be matched post-reload. 2012-04-23 Uros Bizjak ubiz...@gmail.com

Re: [PATCH] Fix PR 33512 Folding of x ((~x) | y) into x y on the tree level

2012-04-23 Thread Andrew Pinski
On Mon, Apr 23, 2012 at 2:41 AM, Richard Guenther richard.guent...@gmail.com wrote: On Sat, Apr 21, 2012 at 6:06 AM, Andrew Pinski andrew.pin...@caviumnetworks.com wrote: This time with the patch and describing what the bug was.  The problem was defcodefor_name does not always set arg1 and

[v3] Add std::is_nothrow_destructible, implement LWG 2049

2012-04-23 Thread Paolo Carlini
Hi, this work from Daniel implements the missing std::is_nothrow_destructible trait and implements LWG 2049. Tested x86_64-linux, committed to mainline. Thanks, Paolo. 2012-04-23 Daniel Krugler daniel.krueg...@googlemail.com * include/std/type_traits

Re: [PATCH v2] ARM: Use different linker path for hardfloat ABI

2012-04-23 Thread Michael Hope
On 24 April 2012 03:35, Richard Earnshaw rearn...@arm.com wrote: On 22/04/12 23:20, Michael Hope wrote: Change the dynamic linker path for ARM hard float executables. Matches the path discussed and agreed on last week[1].  Carlos will follow up with the matching patch to GLIBC[2].  I'm happy

Go patch committed: Change floating point precision

2012-04-23 Thread Ian Lance Taylor
The Go spec calls for the precision of untyped floating point constants to use 256 bits, but gccgo was only using 128 bits. This patch fixes that. This required adjusting one of the tests so that a floating point expression remains an integer as required. Bootstrapped and ran Go testsuite on

Re: thoughts on libatomic

2012-04-23 Thread Lawrence Crowl
On 4/23/12, Andrew MacLeod amacl...@redhat.com wrote: On 04/23/2012 03:29 PM, Richard Henderson wrote: - load_n.c: - I'm concerned about the CAS on read-only mprotected pages? Why again do we think this is safe? Does the standard explicitly allow this? Or should we just

Re: [google/integration] Extend C++11 UDLs to be compatible with inttypes.h macros (issue6104051)

2012-04-23 Thread Ollie Wild
On Mon, Apr 23, 2012 at 3:51 PM, Diego Novillo dnovi...@google.com wrote: Great, thanks.  Patch is OK with the ChangeLog entries filled-in.  Not sure if you'd rather wait for the trunk commit to go in, though.  It may be better to put this version in google/integration and deal with the

[PATCH] Fix ICE during diagnosing of section conflict (PR middle-end/52999)

2012-04-23 Thread Jakub Jelinek
Hi! If a user decl conflicts with a use of a section without a decl (usually rtx constant), then we can ICE in get_section. This patch fixes it, to print the error that has been printed in older gcc versions. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk/4.7? 2012-04-23

[PATCH] Fix VRP ICE on x cst1; if (x cmp cst2) (PR tree-optimization/53058)

2012-04-23 Thread Jakub Jelinek
Hi! My PR52533 patch fixed unsigned comparisons, but not signed ones where the maximum is different. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2012-04-23 Jakub Jelinek ja...@redhat.com PR tree-optimization/53058 * tree-vrp.c

[PATCH] Fix wrong-code at -O0 with MEM_REF in initializer (PR middle-end/53084)

2012-04-23 Thread Jakub Jelinek
Hi! output_addressed_constants isn't able to output constants addressed with MEM_REFsomething, X and similarly compute_reloc_for_constant doesn't handle it. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk/4.7? 2012-04-23 Jakub Jelinek ja...@redhat.com

Re: [PATCH] Fix wrong-code at -O0 with MEM_REF in initializer (PR middle-end/53084)

2012-04-23 Thread Richard Henderson
On 04/23/12 16:03, Jakub Jelinek wrote: PR middle-end/53084 * varasm.c (compute_reloc_for_constant): Handle ADDR_EXPR of MEM_REF. (output_addressed_constants): Likewise. * gcc.c-torture/execute/pr53084.c: New test. Ok. r~

Re: [PATCH] Fix ICE during diagnosing of section conflict (PR middle-end/52999)

2012-04-23 Thread Richard Henderson
On 04/23/12 15:57, Jakub Jelinek wrote: 2012-04-23 Jakub Jelinek ja...@redhat.com PR middle-end/52999 * varasm.c (get_section): Don't ICE for section conflicts with built-in section kinds. Ok everywhere. r~

Re: compare_tests does not work with symlinked directories

2012-04-23 Thread Mike Stump
On Apr 23, 2012, at 12:12 PM, Manuel López-Ibáñez wrote: The version of 'find' in the GCC farm does not traverse symlinked directories unless they have a trailing slash. OK? Ok.

Re: [v3] Add std::is_nothrow_destructible, implement LWG 2049

2012-04-23 Thread Paolo Carlini
On 04/23/2012 11:36 PM, Paolo Carlini wrote: Hi, this work from Daniel implements the missing std::is_nothrow_destructible trait and implements LWG 2049. Tested x86_64-linux, committed to mainline. Oops. These improvements leads to tidier diagnostics for noexcept15.C and I have to adjust it

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-23 Thread H.J. Lu
On Mon, Apr 23, 2012 at 1:43 PM, Sriraman Tallam tmsri...@google.com wrote: On Mon, Apr 23, 2012 at 12:16 PM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Apr 23, 2012 at 6:59 PM, Sriraman Tallam tmsri...@google.com wrote: i386 maintainers -  Is this patch ok? Has the community reached the

RE: MIPS Android patch

2012-04-23 Thread Fu, Chao-Ying
Richard Sandiford wrote: Index: gcc/config/mips/gnu-user.h === --- gcc/config/mips/gnu-user.h (revision 186580) +++ gcc/config/mips/gnu-user.h (working copy) @@ -45,8 +45,10 @@ /* A standard GNU/Linux mapping.

[wwwdocs] Use generic bug references in GCC 3.4 release notes

2012-04-23 Thread Gerald Pfeifer
This fixes two more occurrences per Rainer's earlier observation. Installed. Gerald Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-3.4/changes.html,v retrieving revision 1.158 diff -u -3 -p -r1.158 changes.html ---

Re: [PATCH, PR38785] Throttle PRE at -O3

2012-04-23 Thread Gerald Pfeifer
On Wed, 18 Apr 2012, Richard Guenther wrote: The addition of -ftree-pre-partial-partial is ok if you change its name to -ftree-partial-pre and add documentation to invoke.texi. ...and the GCC 4.8 release notes (gcc-4.8/changes.html). ;-) Gerald

Re: divide 64-bit by constant for 32-bit target machines

2012-04-23 Thread Michael Hope
On 21 April 2012 00:57, Dinar Temirbulatov dtemirbula...@gmail.com wrote: Hi, Here is the patch that adds support for divide 64-bit by constant for 32-bit target machines, this patch was tested on arm-7a with no new regressions, also I am not sure on how to avoid for example i686 targets

Re: [tpf] update for latest build system

2012-04-23 Thread DJ Delorie
Per request from IBM... * config/s390/s390.h (LINK_SPEC): Remove, no longer needed. (LIBSTDCXX): Change to CPP2. Ok. Bye, -Andreas- Thanks! Committed.

Go patch committed: Error for invalid use of ...

2012-04-23 Thread Ian Lance Taylor
This patch to the Go compiler issues error messages for some invalid uses of ... in function calls. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline and 4.7 branch. Ian diff -r a69e43eba4d7 go/expressions.cc --- a/go/expressions.cc Mon Apr 23 14:35:45 2012

Re: [PATCH] Fix wrong-code at -O0 with MEM_REF in initializer (PR middle-end/53084)

2012-04-23 Thread Richard Guenther
On Tue, Apr 24, 2012 at 1:03 AM, Jakub Jelinek ja...@redhat.com wrote: Hi! output_addressed_constants isn't able to output constants addressed with MEM_REFsomething, X and similarly compute_reloc_for_constant doesn't handle it.  Fixed thusly, bootstrapped/regtested on x86_64-linux and

Re: [PATCH] Fix wrong-code at -O0 with MEM_REF in initializer (PR middle-end/53084)

2012-04-23 Thread Jakub Jelinek
On Tue, Apr 24, 2012 at 07:56:11AM +0200, Richard Guenther wrote: On Tue, Apr 24, 2012 at 1:03 AM, Jakub Jelinek ja...@redhat.com wrote: Hi! output_addressed_constants isn't able to output constants addressed with MEM_REFsomething, X and similarly compute_reloc_for_constant doesn't