Re: LRA has been merged into trunk.

2012-10-23 Thread Marc Glisse
On Tue, 23 Oct 2012, Vladimir Makarov wrote: Hi, I was going to merge LRA into trunk last Sunday. It did not happen. LRA was actively changed last 4 weeks by implementing reviewer's proposals which resulted in a lot of new LRA regressions on GCC testsuite in comparison with reload. Finally,

[PATCH] Use define_memory_constraint on sparc when necessary.

2012-10-23 Thread David Miller
While playing around with LRA on sparc I noticed that we had some poorly formed target memory constraints on sparc. In particular, they were not using define_memory_constraint, so we would not get a true return from EXTRA_MEMORY_CONSTRAINT for them. Also, these were matching 'reg' objects for sp

libgo patch committed: Set libgo version number

2012-10-23 Thread Ian Lance Taylor
This patch for mainline corresponds to the one I committed earlier on the 4.7 branch. This sets a different version for libgo.so on mainline going forward. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. I actually committed this yesterday but forgot to pre

Doc patch committed: The '+' constraint does not require a register

2012-10-23 Thread Ian Lance Taylor
Back in May 2004 Richard Henderson ensured that using a '+' constraint in an extended asm statement would work with an 'm' constraint. http://gcc.gnu.org/ml/gcc-patches/2004-05/msg00438.html Unfortunately he did not update the documentation. I verified that Richard's code is still in the compile

Re: [RS6000] libffi ppc64 assembly

2012-10-23 Thread David Edelsohn
On Tue, Oct 23, 2012 at 6:34 PM, Alan Modra wrote: > Gold on powerpc64 doesn't support old ABI objects, but libffi contains > old ABI assembly. This patch modifies those files to support both old > and new ABI, and adds a builtin define to powerpc64 gcc that can be > used to select between the AB

ARC port (5/5): rest of gcc/{,common/}config/arc/

2012-10-23 Thread Joern Rennecke
2012-10-22 Saurabh Verma Ramana Radhakrishnan Joern Rennecke Muhammad Khurram Riaz Brendan Kehoe Michael Eager * config/arc, common/config/arc: New directories. arc-rest.tar.xz Description: application/xz

ARC port (4/5): libgcc/config/arc/

2012-10-23 Thread Joern Rennecke
libgcc: 2012-10-18 Joern Rennecke Brendan Kehoe * libgcc/config/arc: New directory. arc-libgcc.tar.xz Description: application/xz

ARC port (3/5): gcc/config/arc/arc.md

2012-10-23 Thread Joern Rennecke
gcc: 2012-10-22 Saurabh Verma Ramana Radhakrishnan Joern Rennecke Muhammad Khurram Riaz Brendan Kehoe Michael Eager * config/arc: New directory. arc.md.tar.xz Description: application/xz

ARC port (1/5): configuration file patches

2012-10-23 Thread Joern Rennecke
Prerequisites to allow the port to build properly: the lock_length attribute: http://gcc.gnu.org/ml/gcc-patches/2012-10/msg01890.html http://gcc.gnu.org/ml/gcc-patches/2012-10/msg02120.html And from Easwaran Raman : PR middle-end/54957 * optabs.c (emit_cmp_and_jump_insn_1): Remo

Re: LRA has been merged into trunk.

2012-10-23 Thread David Miller
From: David Miller Date: Tue, 23 Oct 2012 21:44:05 -0400 (EDT) > The first issue sparc runs into is that it does not define it's > extra constraints properly. In particular 'T' and 'W' must use > define_memory_constraint. > > Otherwise the EXTRA_MEMORY_CONSTRAINT logic in process_alt_operands()

Re: Minimize downward code motion during reassociation

2012-10-23 Thread Easwaran Raman
On Tue, Oct 23, 2012 at 2:52 AM, Richard Biener wrote: > On Mon, Oct 22, 2012 at 8:31 PM, Easwaran Raman wrote: >> On Mon, Oct 22, 2012 at 12:59 AM, Richard Biener >> wrote: >>> On Fri, Oct 19, 2012 at 12:36 AM, Easwaran Raman wrote: Hi, During expression reassociation, statement

Re: LRA has been merged into trunk.

2012-10-23 Thread David Miller
From: Vladimir Makarov Date: Tue, 23 Oct 2012 19:04:03 -0400 > I am not sure that anything except x86/x86-64 will work now on the > branch. There were too many changes on the branch and I tested only > x86/x86-64. I'll start testing the rest of targets on the branch > next week when LRA is sett

RFA: Add lock_lenth attribute to support the ARC port (Was: Re: Ping: RFA: add lock_length attribute to break branch-shortening cycles)

2012-10-23 Thread Joern Rennecke
Quoting Richard Biener : On Tue, Oct 16, 2012 at 9:35 PM, Joern Rennecke wrote: .. Well, we could split it anyway, and give ports without the need for multiple length attributes the benefit of the optimistic algorithm. I have attached a patch that implements this. Looks reasonable to me, t

Restore bootstrap

2012-10-23 Thread Jeff Law
The obvious fix. commit 36d5e6eace0697b2f8613ab5c24bc23ea359d347 Author: law Date: Wed Oct 24 00:43:24 2012 + * tree-ssa-threadedge.c (thread_across_edge): Remove unused parameter in call to cond_arg_set_in_bb. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk

Re: PR c/53063 Handle Wformat with LangEnabledBy

2012-10-23 Thread Joseph S. Myers
On Tue, 23 Oct 2012, Manuel López-Ibáñez wrote: > The problem is how to represent that Wformat-y2k is enabled by > -Wformat=X with X >= 2, while Wformat-zero-length is enabled by X >=1. > > One possiblity is to allow to specify a condition directly: I guess that's reasonable. -- Joseph S. Myer

Re: [Patch] Fix the tests gcc.dg/vect/vect-8[23]_64.c

2012-10-23 Thread Sharad Singhai
Committed in r192750. Thanks, Sharad On Tue, Oct 23, 2012 at 2:46 PM, Mike Stump wrote: > On Oct 23, 2012, at 6:52 AM, Dominique Dhumieres wrote: >> Following the changes in [PATCH] Add option for dumping to stderr >> (issue6190057) >> the tests gcc.dg/vect/vect-8[23]_64.c fails on powerpc*-*-

Re: PR tree-optimization/54985

2012-10-23 Thread Sharad Singhai
The following trivial patch seems to fix it. Index: tree-ssa-threadedge.c === --- tree-ssa-threadedge.c (revision 192749) +++ tree-ssa-threadedge.c (working copy) @@ -743,7 +743,7 @@ safe to thread this edge. */ if (e->flags

Re: PR tree-optimization/54985

2012-10-23 Thread Sharad Singhai
The trunk seems to be broken at r192749 due to this patch. ../../srctrunk/gcc/tree-ssa-threadedge.c: In function ‘void thread_across_edge(gimple_statement_d*, edge_def*, bool, vec_t**, tree_node* (*)(gimple_statement_d*, gimple_statement_d*))’: ../../srctrunk/gcc/tree-ssa-threadedge.c:583: error:

[C++ Patch PING] PR 53761

2012-10-23 Thread Paolo Carlini
Hi, I'm pinging this patchlet: http://gcc.gnu.org/ml/gcc-patches/2012-10/msg01013.html For sure not an high priority issue, neither I can say to fully understand whether in C++ we can and should have the exact same semantics of the __transparent_union__ attribute in C, but I think that i

Re: LRA has been merged into trunk.

2012-10-23 Thread Vladimir Makarov
On 12-10-23 5:28 PM, David Miller wrote: From: Vladimir Makarov Date: Tue, 23 Oct 2012 11:46:34 -0400 Hi, I was going to merge LRA into trunk last Sunday. It did not happen. LRA was actively changed last 4 weeks by implementing reviewer's proposals which resulted in a lot of new LRA

Re: [AARCH64] Update maintainers file

2012-10-23 Thread Richard Earnshaw
On 23/10/12 23:36, Steven Bosscher wrote: On Wed, Oct 24, 2012 at 12:32 AM, Richard Earnshaw wrote: Patch to update the MAINTAINERS file following the merge of the aarch64 port. Congrats on the new port! Will you also add an announcement of this to the news page (home page) and to gcc-4.8/cha

Re: Remove unused debugging arg from last change

2012-10-23 Thread Magnus Fromreide
On Tue, Oct 23, 2012 at 03:28:37PM -0600, Jeff Law wrote: > > Committed as obvious. > >* tree-ssa-threadedge.c (cond_arg_set_in_bb): Remove unused > debugging argument. Could you please remove the third argument in the calls to cond_arg_set_in_bb as well? /MF > Index: tree-ssa-

Re: [AARCH64] Update maintainers file

2012-10-23 Thread Steven Bosscher
On Wed, Oct 24, 2012 at 12:32 AM, Richard Earnshaw wrote: > Patch to update the MAINTAINERS file following the merge of the aarch64 > port. Congrats on the new port! Will you also add an announcement of this to the news page (home page) and to gcc-4.8/changes.html? Ciao! Steven

[RS6000] libffi ppc64 assembly

2012-10-23 Thread Alan Modra
Gold on powerpc64 doesn't support old ABI objects, but libffi contains old ABI assembly. This patch modifies those files to support both old and new ABI, and adds a builtin define to powerpc64 gcc that can be used to select between the ABIs in assembly. I figure a define is generally useful, and

[AARCH64] Update maintainers file

2012-10-23 Thread Richard Earnshaw
Patch to update the MAINTAINERS file following the merge of the aarch64 port. R. 2012-10-23 Richard Earnshaw * MAINTAINERS (aarch64): Add Marcus and myself. Index: MAINTAINERS === --- MAINTAINERS (revision 192746) +++ MA

Re: [Patch] Fix the test libgomp.graphite/force-parallel-6.c

2012-10-23 Thread Mike Stump
On Oct 23, 2012, at 6:33 AM, Dominique Dhumieres wrote: > The test libgomp.graphite/force-parallel-6.c is not valid as it tries > to write Y[2*N] for Y defined as Ok. [ Could someone check it in for Dominique? ] > 2012-10-23 Dominique d'Humieres > > * testsuite/libgomp.graphite/force-

Re: Follow-up to PR bootstrap/54820

2012-10-23 Thread Ian Lance Taylor
On Tue, Oct 23, 2012 at 2:11 PM, Eric Botcazou wrote: > > 2012-10-23 Eric Botcazou > > PR bootstrap/54820 > * configure.ac (have_static_libs): Force 'no' for GCC version < 4.5. > * configure: Regenerate. This is OK. Thanks. Ian

Re: PR c++/54928 infinite ICE when reporting ICE on macro expansion

2012-10-23 Thread Manuel López-Ibáñez
On 18 October 2012 00:24, Gabriel Dos Reis wrote: > On Wed, Oct 17, 2012 at 6:26 AM, Manuel López-Ibáñez > wrote: >> On 17 October 2012 11:55, Dodji Seketeli wrote: >>> Hello Manuel, >>> >>> Let's CC Gaby on this one as well. >>> >>> Manuel López-Ibáñez writes: >>> The problem is that the

Re: [Patch] Fix the tests gcc.dg/vect/vect-8[23]_64.c

2012-10-23 Thread Mike Stump
On Oct 23, 2012, at 6:52 AM, Dominique Dhumieres wrote: > Following the changes in [PATCH] Add option for dumping to stderr > (issue6190057) > the tests gcc.dg/vect/vect-8[23]_64.c fails on powerpc*-*-*. > This patch adjust the dump files and has been tested on powerpc-apple-darwin9. Ok. > gcc/

Re: PR tree-optimization/54985

2012-10-23 Thread Jakub Jelinek
On Tue, Oct 23, 2012 at 03:34:46PM -0600, Jeff Law wrote: > >I think it should be backported to 4.7, perhaps with a few days delay after > >the > >trunk commit. > Do we even have debug statements after control flow statements? They shouldn't be there, so if you just give up the same way for gsi_s

Re: Tidy store_bit_field_1 & co.

2012-10-23 Thread Eric Botcazou
> I should probably have responded to this earlier, sorry. I'm not sure > which part you mean, so here's an attempt at justifying the whole block: > > 1) WORDS_BIG_ENDIAN is deliberately ignored: > > /* The following line once was done only if WORDS_BIG_ENDIAN, >but I think that is

Re: PR tree-optimization/54985

2012-10-23 Thread Jeff Law
On 10/23/2012 03:22 PM, Jakub Jelinek wrote: On Tue, Oct 23, 2012 at 03:21:59PM -0600, Jeff Law wrote: On 10/23/2012 02:50 PM, Jakub Jelinek wrote: +static bool +cond_arg_set_in_bb (edge e, basic_block bb, int n) +{ + ssa_op_iter iter; + use_operand_p use_p; + gimple last = gsi_stmt (gsi_las

Re: [Patch] libitm: Ask dispatch whether it requires serial mode.

2012-10-23 Thread Richard Henderson
On 2012-10-24 01:48, Torvald Riegel wrote: > Ask dispatch whether it requires serial mode. > > * retry.cc (gtm_thread::decide_begin_dispatch): Ask dispatch whether > it requires serial mode instead of assuming that for certain > dispatchs. > * dispatch.h (abi_dispat

Re: patch to fix constant math - 4th patch - the wide-int class.

2012-10-23 Thread Kenneth Zadeck
On 10/23/2012 04:25 PM, Lawrence Crowl wrote: On 10/23/12, Kenneth Zadeck wrote: On 10/23/2012 02:38 PM, Lawrence Crowl wrote: On 10/23/12, Kenneth Zadeck wrote: On 10/23/2012 10:12 AM, Richard Biener wrote: + inline bool minus_one_p () const; + inline bool zero_p () const; + inline boo

Re: LRA has been merged into trunk.

2012-10-23 Thread David Miller
From: Vladimir Makarov Date: Tue, 23 Oct 2012 11:46:34 -0400 > Hi, I was going to merge LRA into trunk last Sunday. It did not > happen. LRA was actively changed last 4 weeks by implementing > reviewer's proposals which resulted in a lot of new LRA regressions on > GCC testsuite in comp

Re: [patch] libitm: Clarify ABI requirements for data-logging functions.

2012-10-23 Thread Richard Henderson
On 2012-10-24 01:50, Torvald Riegel wrote: > Clarify ABI requirements for data-logging functions. > > * libitm.texi: Clarify ABI requirements for data-logging functions. Ok. r~

Remove unused debugging arg from last change

2012-10-23 Thread Jeff Law
Committed as obvious. * tree-ssa-threadedge.c (cond_arg_set_in_bb): Remove unused debugging argument. Index: tree-ssa-threadedge.c === --- tree-ssa-threadedge.c (revision 192745) +++ tree-ssa-threadedge.c

Re: PR tree-optimization/54985

2012-10-23 Thread Jakub Jelinek
On Tue, Oct 23, 2012 at 03:21:59PM -0600, Jeff Law wrote: > On 10/23/2012 02:50 PM, Jakub Jelinek wrote: > >>+static bool > >>+cond_arg_set_in_bb (edge e, basic_block bb, int n) > >>+{ > >>+ ssa_op_iter iter; > >>+ use_operand_p use_p; > >>+ gimple last = gsi_stmt (gsi_last_bb (e->dest)); > > >

Re: PR tree-optimization/54985

2012-10-23 Thread Jeff Law
On 10/23/2012 02:50 PM, Jakub Jelinek wrote: On Tue, Oct 23, 2012 at 02:35:24PM -0600, Jeff Law wrote: +/* Return TRUE if the statement at the end of e->dest depends on + the output of any statement in BB. Otherwise return FALSE. + + This is used when we are threading a backedge and need t

Follow-up to PR bootstrap/54820

2012-10-23 Thread Eric Botcazou
As reported by Ian and Peter in the audit trail, the check I added to detect whether -static-libstdc++ is supported by g++ doesn't work because the option is silently rejected by versions prior to 4.5. The attached patch forces an error for these versions so as to make the check always fail. T

Re: PR tree-optimization/54985

2012-10-23 Thread Jakub Jelinek
On Tue, Oct 23, 2012 at 02:35:24PM -0600, Jeff Law wrote: > +/* Return TRUE if the statement at the end of e->dest depends on > + the output of any statement in BB. Otherwise return FALSE. > + > + This is used when we are threading a backedge and need to ensure > + that temporary equivalenc

PR tree-optimization/54985

2012-10-23 Thread Jeff Law
When we try to thread across a back edge in the CFG we have a check to ensure that we don't use temporary equivalences which are invalidated by traversal of the back edge to simplify the final conditional. About a year ago I added code to pick up secondary threading opportunities after an in

Re: patch to fix constant math - 4th patch - the wide-int class.

2012-10-23 Thread Lawrence Crowl
On 10/23/12, Kenneth Zadeck wrote: > On 10/23/2012 02:38 PM, Lawrence Crowl wrote: >> On 10/23/12, Kenneth Zadeck wrote: >>> On 10/23/2012 10:12 AM, Richard Biener wrote: + inline bool minus_one_p () const; + inline bool zero_p () const; + inline bool one_p () const; + inl

[lra] patch fixing a testsuite failure

2012-10-23 Thread Vladimir Makarov
The patch is a patch posted for trunk on http://gcc.gnu.org/ml/gcc-patches/2012-10/msg02085.html Committed as rev. 192744. Index: lra-constraints.c === --- lra-constraints.c (revision 192719) +++ lra-constraints.c (working copy) @

Re: [lra] patch to fix testsuite regressions

2012-10-23 Thread Vladimir Makarov
On 10/21/2012 05:27 AM, Richard Sandiford wrote: Hi Vlad, Richard, sorry for long delay with the answer. I was really busy all these days trying to fix a lot of GCC testsuite failures (for some ones it was extremely difficult to find failure reasons) and push LRA into trunk. Vladimir Makaro

patch to fix a testsuite failure in LRA

2012-10-23 Thread Vladimir Makarov
Uros reported that GCC after LRA submitting has a new test failure on x86. The reason was in wrong update live info in EBB containing empty BB as the last EBB block. For such case live_out of the last EBB was undefined and actually a garbage. It resulted in triggering a code which checks t

[lra] patch to prevent ASHIFT exchange and dead early clobber operand

2012-10-23 Thread Vladimir Makarov
The following patch prevents mistaken ASHIFT exchange found by Richard Sandiford and removes check early_clobber matching operand. The patch was successfully bootstrap on x86-64. Committed as rev. 192742. 2012-10-23 Vladimir Makarov * lra-constraints.c (extract_loc_address_regs):

Re: [m68k] Fix option handling for -m68020-40 and -m68020-60

2012-10-23 Thread Andreas Schwab
Gunther Nikl writes: > While working with GCC 4.7, I noticed that the -m68020-40 and -m68020-60 > options are broken. Broken in which way? Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something complete

Re: Fourth ping: Re: Add a configure option to disable system header canonicalizations (issue6495088)

2012-10-23 Thread Ian Lance Taylor
On Tue, Oct 23, 2012 at 12:38 PM, Tom Tromey wrote: > Steven> Probably you mean the revised patch here: > Steven> http://gcc.gnu.org/ml/gcc-patches/2012-09/msg00459.html > > Steven> The patch look OK to me but I can't approve it. > > I'm sorry about the delay on this. > > The libcpp bits are ok. >

Re: Fourth ping: Re: Add a configure option to disable system header canonicalizations (issue6495088)

2012-10-23 Thread Tom Tromey
Steven> Probably you mean the revised patch here: Steven> http://gcc.gnu.org/ml/gcc-patches/2012-09/msg00459.html Steven> The patch look OK to me but I can't approve it. I'm sorry about the delay on this. The libcpp bits are ok. I can't approve the other parts. I think new configure options sho

Re: [PATCH] [7/10] AArch64 Port

2012-10-23 Thread Tom Tromey
> "Marcus" == Marcus Shawcroft writes: Marcus> This patch adjusts the libcpp configury for AArch64. Marcus> Proposed ChangeLog: Marcus> * configure.ac: Enable AArch64. Marcus> * configure: Regenerate. This is ok. Thanks. Tom

Re: PR c/53063 Handle Wformat with LangEnabledBy

2012-10-23 Thread Manuel López-Ibáñez
On 19 October 2012 18:17, Joseph S. Myers wrote: > On Wed, 17 Oct 2012, Manuel López-Ibáñez wrote: > >> documentation but I can also implement -Wformat=0 being an alias for >> -Wno-format and -Wformat=1 an alias for -Wformat and simply reject >> -Wno-format=. > > I think that's what's wanted; -Wno

Re: patch to fix constant math - 4th patch - the wide-int class.

2012-10-23 Thread Kenneth Zadeck
On 10/23/2012 02:38 PM, Lawrence Crowl wrote: On 10/23/12, Kenneth Zadeck wrote: On 10/23/2012 10:12 AM, Richard Biener wrote: + inline bool minus_one_p () const; + inline bool zero_p () const; + inline bool one_p () const; + inline bool neg_p () const; what's wrong with w == -1, w == 0,

Re: PING^2: [patch] pr/54508: fix incomplete debug information for class

2012-10-23 Thread Paul_Koning
On Oct 23, 2012, at 2:02 PM, Jason Merrill wrote: > OK. > > Jason Thanks. Committed. paul

Re: patch to fix constant math - 4th patch - the wide-int class.

2012-10-23 Thread Lawrence Crowl
On 10/23/12, Kenneth Zadeck wrote: > On 10/23/2012 10:12 AM, Richard Biener wrote: > > + inline bool minus_one_p () const; > > + inline bool zero_p () const; > > + inline bool one_p () const; > > + inline bool neg_p () const; > > > > what's wrong with w == -1, w == 0, w == 1, etc.? > > I would

Re: LRA has been merged into trunk.

2012-10-23 Thread Vladimir Makarov
On 10/23/2012 01:57 PM, Uros Bizjak wrote: Hello! Hi, I was going to merge LRA into trunk last Sunday. It did not happen. LRA was actively changed last 4 weeks by implementing reviewer's proposals which resulted in a lot of new LRA regressions on GCC testsuite in comparison with reload. Fina

Re: [Patch] Fix the tests gcc.dg/vect/vect-8[23]_64.c

2012-10-23 Thread Sharad Singhai
+cc richard.guent...@gmail.com If it is approved, I will be happy to commit it for you. Thanks, Sharad Sharad On Tue, Oct 23, 2012 at 6:52 AM, Dominique Dhumieres wrote: > Following the changes in [PATCH] Add option for dumping to stderr > (issue6190057) > the tests gcc.dg/vect/vect-8[23]_64.

Re: gcc 4.7 libgo patch committed: Set libgo version number

2012-10-23 Thread Ian Lance Taylor
On Tue, Oct 23, 2012 at 9:59 AM, Jakub Jelinek wrote: > On Tue, Oct 23, 2012 at 09:57:21AM -0700, Ian Lance Taylor wrote: >> The problem is that I forgot to do that when the 4.7 branch was >> created. So the 4.7 branch and the 4.6 branch were using the same >> SONAME although they had completely

Re: Tidy store_bit_field_1 & co.

2012-10-23 Thread Richard Sandiford
Eric Botcazou writes: >> + enum machine_mode op_mode = mode_for_extraction (EP_insv, 3); >>if (HAVE_insv >>&& GET_MODE (value) != BLKmode >>&& bitsize > 0 >> @@ -690,25 +670,34 @@ store_bit_field_1 (rtx str_rtx, unsigned >> -fstrict-volatile-bitfields is in effect.

Re: libgo patch committed: Update to current Go library

2012-10-23 Thread Ian Lance Taylor
On Tue, Oct 23, 2012 at 10:47 AM, Uros Bizjak wrote: > > Additional test fails on alphaev68-linux-gnu: > > --- FAIL: TestPassFD (0.15 seconds) > passfd_test.go:62: FileConn: dup: Bad file descriptor > FAIL > FAIL: syscall As far as I can see this error message occurs when calling dup on the

[m68k] Fix option handling for -m68020-40 and -m68020-60

2012-10-23 Thread Gunther Nikl
Hello, While working with GCC 4.7, I noticed that the -m68020-40 and -m68020-60 options are broken. This bug was introduced in May 2011 with the patch at . The code in m68k_option_override to set cpu and tune doesn't trigger when using -m680

Re: patch to fix constant math - 4th patch - the wide-int class.

2012-10-23 Thread Lawrence Crowl
On 10/23/12, Richard Biener wrote: > I wonder if for the various ways to specify precision/len there > is a nice C++ way of moving this detail out of wide-int. I can > think only of one: > > struct WIntSpec { > WIntSpec (unsigned int len, unsigned int precision); > WIntSpec (const_tree); >

Re: libgo patch committed: Update to current Go library

2012-10-23 Thread Ian Lance Taylor
On Tue, Oct 23, 2012 at 10:47 AM, Uros Bizjak wrote: > > On my x86_64-linux-gnu (Fedora 18) libgo testsuite fails following test: > > --- FAIL: TestCgoCrashHandler (0.01 seconds) > testing.go:377: program exited with error: exec: "go": > executable file not found in $PATH > --- FAIL: TestC

Re: PING^2: [patch] pr/54508: fix incomplete debug information for class

2012-10-23 Thread Jason Merrill
OK. Jason

Re: [PATCH] [0/10] AArch64 Port

2012-10-23 Thread Marcus Shawcroft
On 23/10/12 10:42, Marcus Shawcroft wrote: Folks, We would like to request the merge of aarch64-branch into trunk. All of the patches approved by Jeff and Jakub are now committed, with the documentation correction requested by Joseph. /Marcus

Re: Fix bugs introduced by switch-case profile propagation

2012-10-23 Thread Easwaran Raman
On Tue, Oct 23, 2012 at 3:03 AM, Jan Hubicka wrote: >> Ping. >> >> >> On Wed, Oct 17, 2012 at 1:48 PM, Easwaran Raman wrote: >> > Hi, >> > This patch fixes bugs introduced by my previous patch to propagate >> > profiles during switch expansion. Bootstrap and profiledbootstrap >> > successful on

Re: [PATCH] [2/10] AArch64 Port

2012-10-23 Thread Marcus Shawcroft
On 23/10/12 15:39, Joseph S. Myers wrote: On Tue, 23 Oct 2012, Marcus Shawcroft wrote: +@item -mcmodel=tiny +@opindex mcmodel=tiny +Generate code for the tiny code model. The program and its statically defined +symbols must be within 1GB of each other. Pointers are 64 bits. Programs can +be

Re: LRA has been merged into trunk.

2012-10-23 Thread Uros Bizjak
Hello! > Hi, I was going to merge LRA into trunk last Sunday. It did not happen. LRA > was actively changed last 4 weeks by implementing reviewer's proposals which > resulted in a lot of new LRA regressions on GCC testsuite in comparison with > reload. Finally, they were fixed and everything lo

Re: [PATCH] Fix sizeof related pt.c ICE (PR c++/54844)

2012-10-23 Thread Jason Merrill
OK. Jason

Re: [C++ Patch] PR 54922

2012-10-23 Thread Jason Merrill
OK. Jason

Re: gcc 4.7 libgo patch committed: Set libgo version number

2012-10-23 Thread Jakub Jelinek
On Tue, Oct 23, 2012 at 09:57:21AM -0700, Ian Lance Taylor wrote: > The problem is that I forgot to do that when the 4.7 branch was > created. So the 4.7 branch and the 4.6 branch were using the same > SONAME although they had completely different ABIs. > > That is, there is no ABI change on the

Re: PING^2: [patch] pr/54508: fix incomplete debug information for class

2012-10-23 Thread Paul_Koning
On Oct 5, 2012, at 6:05 PM, Cary Coutant wrote: >> There certainly is a fair amount of code in dwarf2read.c in gdb to handle >> DW_AT_declaration and do things differently for declarations. >> >> Should I rework this patch to use that mechanism instead? If so, how? If >> the class is marked

Re: libgo patch committed: Update to current Go library

2012-10-23 Thread Uros Bizjak
Hello! > I have committed a patch to update the mainline version of libgo to the > current master Go library sources. At this point I will only be > updating the gcc 4.7 branch for bug fixes. > > This is a substantial patch that brings in several months of work. As > usual I am not posting the c

Re: [PATCH] [6/10] AArch64 Port

2012-10-23 Thread Marcus Shawcroft
On 23/10/12 16:14, Jeff Law wrote: On 10/23/2012 03:42 AM, Marcus Shawcroft wrote: This patch adjusts the libatomic configury for AArch64. Proposed ChangeLog: * configure.tgt: Mark libatomic unsupported. This is good. Please install. Presumably at some point in the not too distan

Re: gcc 4.7 libgo patch committed: Set libgo version number

2012-10-23 Thread Ian Lance Taylor
On Tue, Oct 23, 2012 at 9:27 AM, Jakub Jelinek wrote: > On Tue, Oct 23, 2012 at 06:16:25PM +0200, Matthias Klose wrote: >> On 23.10.2012 06:55, Ian Lance Taylor wrote: >> > PR 54918 points out that libgo is not using version numbers as it >> > should. At present none of libgo in 4.6, 4.7 and main

[PATCH] variably_modified_type_p tweak for cdtor cloning (PR debug/54828)

2012-10-23 Thread Jakub Jelinek
Hi! The following testcase ICEs, because the VLA ARRAY_TYPE in the ctor isn't considered variably_modified_type_p during cloning of the ctor when the types weren't gimplified yet. The size is a non-constant expression that has SAVE_EXPR of a global VAR_DECL in it, so when variably_modified_type_p

Re: [PATCH] [3/10] AArch64 Port

2012-10-23 Thread Marcus Shawcroft
On 23/10/12 16:38, Jeff Law wrote: Given that you and Richard Earnshaw are the approved maintainers for the AAarch64 port, I'm going to give this an OK without diving into it. I'm going to assume you and Richard will iterate with anyone who does dive deeply into the port and has comments/sugges

Re: [C++ PATCH] Fix cplus_decl_attributes (PR c++/54988)

2012-10-23 Thread Jason Merrill
OK. Jason

Re: gcc 4.7 libgo patch committed: Set libgo version number

2012-10-23 Thread Jakub Jelinek
On Tue, Oct 23, 2012 at 06:16:25PM +0200, Matthias Klose wrote: > On 23.10.2012 06:55, Ian Lance Taylor wrote: > > PR 54918 points out that libgo is not using version numbers as it > > should. At present none of libgo in 4.6, 4.7 and mainline are > > compatible with each other. This patch to the

[Patch] Potential fix for PR55033

2012-10-23 Thread Sebastian Huber
-- Sebastian Huber, embedded brains GmbH Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany Phone : +49 89 18 90 80 79-6 Fax : +49 89 18 90 80 79-9 E-Mail : sebastian.hu...@embedded-brains.de PGP : Public key available on request. Diese Nachricht ist keine geschäftliche Mitteilu

Re: gcc 4.7 libgo patch committed: Set libgo version number

2012-10-23 Thread Matthias Klose
On 23.10.2012 06:55, Ian Lance Taylor wrote: > PR 54918 points out that libgo is not using version numbers as it > should. At present none of libgo in 4.6, 4.7 and mainline are > compatible with each other. This patch to the 4.7 branch sets the > version number for libgo there. Bootstrapped and

Re: patch to fix constant math - 4th patch - the wide-int class.

2012-10-23 Thread Kenneth Zadeck
On 10/23/2012 10:12 AM, Richard Biener wrote: On Tue, Oct 9, 2012 at 5:09 PM, Kenneth Zadeck wrote: This patch implements the wide-int class.this is a more general version of the double-int class and is meant to be the eventual replacement for that class.The use of this class removes a

Re: [PATCH, ARM] Fix offset_ok_for_ldrd_strd in Thumb1

2012-10-23 Thread Richard Earnshaw
On 23/10/12 16:54, Greta Yorsh wrote: The function offset_ok_for_ldrd_strd should return false for Thumb1, because TARGET_LDRD and Thumb1 can be both enabled (for example, the default for cortex-m0). This patch fixes ICE that is caused by gcc r192678 and occurs when building gcc with newlib for

Re: [PATCH 3/3] [asan] Instrument built-in memory access function calls

2012-10-23 Thread Xinliang David Li
On Tue, Oct 23, 2012 at 8:58 AM, Jakub Jelinek wrote: > On Tue, Oct 23, 2012 at 08:47:48AM -0700, Xinliang David Li wrote: >> > + /* The builtin below cannot be safely instrumented as their >> > + length parameter is just a mere limit. >> > + >> >> Why can't the following be instrumen

Re: [PATCH 3/3] [asan] Instrument built-in memory access function calls

2012-10-23 Thread Jakub Jelinek
On Tue, Oct 23, 2012 at 03:11:29PM +0200, Dodji Seketeli wrote: > + /* (src, n) style memops. */ > +case BUILT_IN_STRNDUP: > + source0 = gimple_call_arg (call, 0); > + len = gimple_call_arg (call, 1); > + break; I think you can't instrument strndup either, the length is ju

Re: [PATCH, ARM] Fix offset_ok_for_ldrd_strd in Thumb1

2012-10-23 Thread Ramana Radhakrishnan
Ok for trunk? Ok. ramana

Re: [PATCH] [3/10] AArch64 Port

2012-10-23 Thread Ramana Radhakrishnan
On 10/23/12 16:38, Jeff Law wrote: > On 10/23/2012 03:42 AM, Marcus Shawcroft wrote: > > The one question in the back of my mind is whether or not this uses the > new iterator support we discussed a few months ago? I can't recall if > that was integrated into the trunk or not. Generic support fo

Re: [PATCH 3/3] [asan] Instrument built-in memory access function calls

2012-10-23 Thread Jakub Jelinek
On Tue, Oct 23, 2012 at 08:47:48AM -0700, Xinliang David Li wrote: > > + /* The builtin below cannot be safely instrumented as their > > + length parameter is just a mere limit. > > + > > Why can't the following be instrumented? The length is min (n, strlen (str)). Because that would

[PATCH, ARM] Fix offset_ok_for_ldrd_strd in Thumb1

2012-10-23 Thread Greta Yorsh
The function offset_ok_for_ldrd_strd should return false for Thumb1, because TARGET_LDRD and Thumb1 can be both enabled (for example, the default for cortex-m0). This patch fixes ICE that is caused by gcc r192678 and occurs when building gcc with newlib for arm-none-eabi cortex-m0. Ok for trunk?

[patch] libitm: Clarify ABI requirements for data-logging functions.

2012-10-23 Thread Torvald Riegel
This patch clarifies the ABI requirements for data-logging functions in libitm's documentation. Thanks to Luke Dalessandro for pointing this out. OK for trunk? Torvald commit b9cbb260f958f53afbea69675458f3f15a04b812 Author: Torvald Riegel Date: Tue Oct 23 14:56:32 2012 +0200 Clarify ABI

[Patch] libitm: Ask dispatch whether it requires serial mode.

2012-10-23 Thread Torvald Riegel
This patch cleans up an implicit assumption about which TM methods actually need to be run in serial mode. Instead, the transaction begin code now asks a TM method's dispatch what it needs. OK for trunk? Torvald commit 12170ba5013e855bf4ea784823961f63e3e2de4c Author: Torvald Riegel Date: Tue

Re: [PATCH 3/3] [asan] Instrument built-in memory access function calls

2012-10-23 Thread Xinliang David Li
On Tue, Oct 23, 2012 at 6:11 AM, Dodji Seketeli wrote: > This patch instruments many memory access patterns through builtins. > > Basically, for a call like: > > __builtin_memset (from, 0, n_bytes); > > the patch would only instrument the accesses at the beginning and at > the end of the memo

Re: [PATCH] [10/10] AArch64 Port

2012-10-23 Thread Jeff Law
On 10/23/2012 03:43 AM, Marcus Shawcroft wrote: This patch provides the AArch64 libstdc++-v3 port, it contains both the required configury adjustment to config.host and the new file introduced by the AArch64 port. Proposed ChangeLog: * config/cpu/aarch64/cxxabi_tweaks.h: New file.

Re: [PATCH] [1/10] AArch64 Port

2012-10-23 Thread Jeff Law
On 10/23/2012 03:42 AM, Marcus Shawcroft wrote: This patch contains the adjustments to top level gcc configury required to enable the AArch64 port. Proposed ChangeLog: * config.gcc: Add AArch64. * configure.ac: Add AArch64 TLS support detection. * configure: Regen

[PATCH] Invalidate in cselib sp after processing frame_pointer_needed fp setter (PR rtl-optimization/54921)

2012-10-23 Thread Jakub Jelinek
Hi! This is an attempt to hopefully end the endless stream of aliasing miscompilations where alias.c assumes that hfp based accesses can't alias sp based accesses, but both sp and fp can appear in VALUEs pretty much randomly. As detailed in the PR, we have the r variable at rbp - 48 and rsp is al

Re: [PATCH] [3/10] AArch64 Port

2012-10-23 Thread Jeff Law
On 10/23/2012 03:42 AM, Marcus Shawcroft wrote: This patch contains all of the new files for the target port itself, the patch does not modify any existing file. Proposed ChangeLog: * common/config/aarch64/aarch64-common.c: New file. * config/aarch64/aarch64-arches.def: New

Re: [PATCH] Fix PR55011

2012-10-23 Thread Michael Matz
Hi, On Tue, 23 Oct 2012, Richard Biener wrote: > > So, one question, are you claiming that a VRP worker like this: > > > >VR derive_new_range_from_operation (VR a, VR b) > > > > is _ever_ allowed to return UNDEFINED when a or b is something else than > > UNDEFINED? You seem to claim so AF

Re: [PATCH] [2/10] AArch64 Port

2012-10-23 Thread Jeff Law
On 10/23/2012 03:42 AM, Marcus Shawcroft wrote: This patch contains the additions to the gcc/doc files to document the AArch64 port. Proposed ChangeLog: * doc/invoke.texi (AArch64 Options): New. * doc/md.texi (Machine Constraints): Add AArch64. This is fine. Please install

Re: [PATCH] [8/10] AArch64 Port

2012-10-23 Thread Jeff Law
On 10/23/2012 03:42 AM, Marcus Shawcroft wrote: This patch provides the AArch64 libgcc port, it contains both the required configury adjustment to config.host and the new files introduced by the AArch64 port. Proposed ChangeLog: * config.host (aarch64*-*-elf, aarch64*-*-linux*): New.

Re: [PATCH] [5/10] AArch64 Port

2012-10-23 Thread Jeff Law
On 10/23/2012 03:42 AM, Marcus Shawcroft wrote: This patch contains all of the new files added to the test suite for AArch64, the patch does not modify any existing file. Proposed ChangeLog: * gcc.target/aarch64/aapcs/aapcs64.exp: New file. * gcc.target/aarch64/aapcs/abitest-2

  1   2   >