[arm-embedded] Patch to define multilibs for arm embedded-4_8-branch

2013-07-24 Thread Terry Guo
Hi Joey, This patch is to define multilibs for recently created embedded-4_8-branch. Is it OK to commit? BR, Terry 2013-07-24 Terry Guo terry@arm.com * configure.ac (with_multilib_list): Export its value. * Makefile.in (with_multilib_list): Import it from configure files.

Re: [PATCH, ARM/Thumb1] Adjust rtx cost to prevent expanding MULT into shift/add instructions

2013-07-24 Thread Richard Earnshaw
On 24/07/13 03:36, Terry Guo wrote: Hi there, This patch intends to update thumb1_size_rtx_costs function to correctly handle those RTXs defined by RTL expansion pass. Thus the GIMPLE multiplication will be expanded to single mul instruction instead of a bunch of shift/add/sub instructions

[PATCH] S/390: libitm: Use .machine to enable the HTM instructions in the assembler

2013-07-24 Thread Andreas Krebbel
Hi, the hardware transactional execution instructions in libitm are only enabled if actually supported by the assembler. So the option which enables the instructions in the assembler should only be set if we are already in a code path where we know the assembler can do it. With the attached

Re: [v3] libstdc++/57920

2013-07-24 Thread Paolo Carlini
Hi, On 07/23/2013 12:45 PM, Chung-Ju Wu wrote: 2013/7/23 Paolo Carlini paolo.carl...@oracle.com: ... in the future, when we manage to actually avoid including cstdio from string I think we can avoid including it from random if we play a bit with void*. Should be safe aliasing-wise. Something

Re: [Patch] Reimplment regex matcher using DFS

2013-07-24 Thread Paolo Carlini
On 07/23/2013 05:31 PM, Tim Shen wrote: Revised version, with _M_dfs_* moved to the class definition, PR testcases, and some comments on _M_dfs(). I nobody has further comments, I think this is Ok to go in now. Let's make sure we don't forget the various comments about optimization,

Re: Go patch committed: Update libgo to 1.1.1

2013-07-24 Thread Uros Bizjak
On Wed, Jul 24, 2013 at 9:09 AM, Uros Bizjak ubiz...@gmail.com wrote: The test was re-run with GOTESTFLAGS=--keep. When running the resulting a.out with strace -f -o strace-x86_64 ./a.out from the saved test directory, the test behaved in the same way as on alpha - it hever finished. I have

RE: [PING] Two patches pending

2013-07-24 Thread Bin Cheng
Ping this patch: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01057.html Thanks. bin -Original Message- From: Jeff Law [mailto:l...@redhat.com] Sent: Saturday, July 20, 2013 1:34 AM To: Bin Cheng Cc:

RE: [PATCH GCC/ARM]Disable -fira-hoist-pressure on Thumb2

2013-07-24 Thread Bin Cheng
ping. Thanks. bin -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of Bin Cheng Sent: Tuesday, July 09, 2013 4:08 PM To: gcc-patches@gcc.gnu.org Subject: [PATCH GCC/ARM]Disable -fira-hoist-pressure on Thumb2 Hi, For now the

Re: Go patch committed: Update libgo to 1.1.1

2013-07-24 Thread Rainer Orth
Ian Lance Taylor i...@google.com writes: On Thu, Jul 18, 2013 at 6:59 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: Ian Lance Taylor i...@google.com writes: I have committed a large patch to update libgo to the library that was part of the Go 1.1.1 release. As usual, I'm not

RFA: Fix rtl-optimization/57968

2013-07-24 Thread Joern Rennecke
The problem is fixed by simply moving the code that skips insns that don't set a part of the return value below the code that checks if the insn needs a non-MODE_EXIT mode setting. bootstrapped/regtested on i686-pc-linux-gnu 2013-07-24 Joern Rennecke joern.renne...@embecosm.com PR

Re: [PATCH, PowerPC] Fix unaligned Altivec load/stores in LE mode

2013-07-24 Thread David Edelsohn
On Tue, Jul 23, 2013 at 11:19 PM, Bill Schmidt wschm...@linux.vnet.ibm.com wrote: In order to use vperm for aligning loads and stores in little endian mode, we need to reverse the order of the input operands and use lvsl instead of lvsr. This corrects 32 regressions in the test suite when run

Re: Go patch committed: Update libgo to 1.1.1

2013-07-24 Thread Ian Lance Taylor
On Wed, Jul 24, 2013 at 2:24 AM, Uros Bizjak ubiz...@gmail.com wrote: On Wed, Jul 24, 2013 at 9:09 AM, Uros Bizjak ubiz...@gmail.com wrote: The test was re-run with GOTESTFLAGS=--keep. When running the resulting a.out with strace -f -o strace-x86_64 ./a.out from the saved test directory, the

Re: Go patch committed: Update libgo to 1.1.1

2013-07-24 Thread Ian Lance Taylor
On Wed, Jul 24, 2013 at 4:49 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: Ian Lance Taylor i...@google.com writes: On Thu, Jul 18, 2013 at 6:59 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: Ian Lance Taylor i...@google.com writes: I have committed a large patch to update

[www] Mention avx512 and mpx branches

2013-07-24 Thread Kirill Yukhin
Hi, Patch adds mention of avx512 and mpx branches in svn.html. Ok to install? Index: htdocs/svn.html === RCS file: /cvs/gcc/wwwdocs/htdocs/svn.html,v retrieving revision 1.185 diff -p -r1.185 svn.html *** htdocs/svn.html 27 May

Re: [Patch] Reimplment regex matcher using DFS

2013-07-24 Thread Tim Shen
On Wed, Jul 24, 2013 at 5:18 PM, Paolo Carlini paolo.carl...@oracle.com wrote: Then we should also add something (by hand, unfortunately, as I explained) to the audit trails of those bugs, because we want the original submitters to double check the fixes with their own code, get as much

[PATCH] ppc-vector-memcpy.c

2013-07-24 Thread David Edelsohn
gcc.target/ppc-vector-memcpy.c testcase currently fails and has been failing since early in the GCC 4.6 development cycle. A change by Eric Botcazou in April 2008 to the handling of constants reduced the extra alignment of initializers matching the alignment of the variable. The testcases

[PATCH 1/2] Introduce context class

2013-07-24 Thread David Malcolm
gcc/ Introduce context class. * Makefile.in (CONTEXT_H): New. (OBJS): Add context.o. (toplev.o): Add CONTEXT_H to dependencies. (context.o): New. * toplev.c (general_init): Create the singleton gcc::context instance. * context.c:

[PATCH 0/2] Introduce context and pipeline classes

2013-07-24 Thread David Malcolm
The following pair of patches is aimed at eliminating global variables from gcc's internals, as per my Cauldron talk on state-removal [1] Patch 1 introduces a gcc::context class, along with a singleton instance. The name of the instance is chosen so as to be easy to type in the debugger:

[PATCH 2/2] Introduce beginnings of a pipeline class.

2013-07-24 Thread David Malcolm
gcc/ * Makefile.in (PIPELINE_H): New. (lto-cgraph.o): Depend on CONTEXT_H and PIPELINE_H. (passes.o): Likewise. (statistics.o): Likewise. (cgraphunit.o): Likewise. (context.o): Depend on PIPELINE_H. * pipeline.h: New. * cgraphunit.c

Re: [Patch] Reimplment regex matcher using DFS

2013-07-24 Thread Paolo Carlini
On 07/24/2013 04:50 PM, Tim Shen wrote: On Wed, Jul 24, 2013 at 5:18 PM, Paolo Carlini paolo.carl...@oracle.com wrote: Then we should also add something (by hand, unfortunately, as I explained) to the audit trails of those bugs, because we want the original submitters to double check the fixes

Re: [Patch] Reimplment regex matcher using DFS

2013-07-24 Thread Jonathan Wakely
On 24 July 2013 15:50, Tim Shen wrote: Committed. Thanks! Should I register a new bugzilla account to comment on these two bugs(53622 and 57173), or any other one will do this? I've added a comment to the bugs. With svn commit rights you should be able to receive email at your

[PATCH, i386, MPX] Support of Intel MPX ISA

2013-07-24 Thread Ilya Enkovich
Hi, Here is a patch which adds support for new instructions from Intel Memory Protection Extensions (MPX) ISA [1] This patch introduces bound type, modes, registers and all MPX instructions. Control transfer instructions are modified to support 'bnd' prefix used by Intel MPX. Bootstrap and

Re: [PATCH 3/4] Introduce NEXT_PASS_NUM macro

2013-07-24 Thread Martin Jambor
Hi, On Tue, Jul 23, 2013 at 11:43:04AM -0400, David Malcolm wrote: On Tue, 2013-07-23 at 16:46 +0200, Martin Jambor wrote: Hi, On Mon, Jul 22, 2013 at 03:22:33PM -0400, David Malcolm wrote: On Mon, 2013-07-22 at 20:25 +0200, Martin Jambor wrote: On Wed, Jul 17, 2013 at 09:18:22PM

Re: [PATCH 1/6] Andes nds32: configure settings for nds32 target.

2013-07-24 Thread Chung-Ju Wu
Hi, Joseph, Sorry for the late revised patch. We have completed all of it based on your review comments. On 7/10/13 6:56 AM, Joseph S. Myers wrote: On Mon, 8 Jul 2013, Chung-Ju Wu wrote: + nds32*-*-*) + supported_defaults=arch nds32_lib + + # process

Re: [PATCH, i386, MPX] Support of Intel MPX ISA

2013-07-24 Thread Joseph S. Myers
On Wed, 24 Jul 2013, Ilya Enkovich wrote: Here is a patch which adds support for new instructions from Intel Memory Protection Extensions (MPX) ISA [1] This patch introduces bound type, modes, registers and all MPX instructions. Control transfer instructions are modified to support 'bnd'

Re: [www] Mention avx512 and mpx branches

2013-07-24 Thread Gerald Pfeifer
On Wed, 24 Jul 2013, Kirill Yukhin wrote: Patch adds mention of avx512 and mpx branches in svn.html. Ok to install? Yes, thanks! Gerald

Re: [PATCH 2/6] Andes nds32: machine description of nds32 porting (1).

2013-07-24 Thread Joseph S. Myers
On Wed, 24 Jul 2013, Chung-Ju Wu wrote: We removed the fancy comment formatting as you suggested. However, for the comments like: /* = */ /* Title of section */ /* = */ /* Title of subsection 1 */ /* Title of subsection 2 */ I wish I can

Re: [Patch, PR 57800] Waste work in gfc_match_call()

2013-07-24 Thread Bernhard Reutner-Fischer
On 23 July 2013 23:32:27 pcha...@cs.wisc.edu wrote: Hi, The problem appears in revision 201034 in version 4.9. I attached a one-line patch that fixes it. I also reported this problem at http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57800 Bootstrap and regression-tested on x86_64-linux. In

Re: [Patch] Reimplment regex matcher using DFS

2013-07-24 Thread Paolo Carlini
On 07/24/2013 05:18 PM, Jonathan Wakely wrote: Should I register a new bugzilla account to comment on these two bugs(53622 and 57173), or any other one will do this? I've added a comment to the bugs. With svn commit rights you should be able to receive email at your username@gcc.gnu.org and if

Re: [PATCH 4/6] Andes nds32: testsuite modifications for nds32 target.

2013-07-24 Thread Chung-Ju Wu
Hi, Joseph, Sorry for the late revised patch. We have completed all of it based on your review comments. On 7/10/13 8:08 AM, Joseph S. Myers wrote: On Mon, 8 Jul 2013, Chung-Ju Wu wrote: * gcc.dg/ucnid-11.c: Skip for nds32*-*-*. * gcc.dg/ucnid-2.c: Skip for nds32*-*-*.

Re: [PATCH 5/6] Andes nds32: documentation for nds32 target.

2013-07-24 Thread Chung-Ju Wu
On 7/12/13 1:35 AM, Chung-Ju Wu wrote: On 7/10/13 8:11 AM, Joseph S. Myers wrote: Don't you have built-in functions to document as well? You are right. We did forget to describe built-in functions in the documentation. Thanks for pointing out our carelessness. :) Use @: (or a comma) after

Re: [www] Mention avx512 and mpx branches

2013-07-24 Thread Kirill Yukhin
Thanks, checked in! On Wed, Jul 24, 2013 at 8:06 PM, Gerald Pfeifer ger...@pfeifer.com wrote: On Wed, 24 Jul 2013, Kirill Yukhin wrote: Patch adds mention of avx512 and mpx branches in svn.html. Ok to install? Yes, thanks! Gerald

Re: [Patch, PR 57800] Waste work in gfc_match_call()

2013-07-24 Thread Bernhard Reutner-Fischer
On 24 July 2013 18:12:00 Bernhard Reutner-Fischer rep.dot@gmail.com wrote: On 23 July 2013 23:32:27 pcha...@cs.wisc.edu wrote: Hi, The problem appears in revision 201034 in version 4.9. I attached a one-line patch that fixes it. I also reported this problem at

Re: [PATCH 3/6] Andes nds32: libgcc of nds32 porting.

2013-07-24 Thread Chung-Ju Wu
Hi, Joseph, Sorry for that we have to zip this patch. It seems that the plaintext revised patch is too large to be sent to mailing list due to lots of copyright information. And I am really sorry for that you probably kept receiving the same mail. Somehow the this [PATCH 3/6] mail just didn't

[PATCH] doc: mention that -ftls-model is subject to optimization

2013-07-24 Thread Alexander Monakov
Hello, As discussed here, the current behavior of -ftls-model is intended: http://gcc.gnu.org/ml/gcc/2013-07/msg00248.html (in short: unlike the attribute, -ftls-model sets a most general model to be used, but the compiler can use a less general model than specified, making it impossible to set

Re: Go patch committed: Update libgo to 1.1.1

2013-07-24 Thread Ian Lance Taylor
On Wed, Jul 24, 2013 at 4:49 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: Ian Lance Taylor i...@google.com writes: On Thu, Jul 18, 2013 at 6:59 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: Ian Lance Taylor i...@google.com writes: I have committed a large patch to update

Re: Go patch committed: Update libgo to 1.1.1

2013-07-24 Thread Ian Lance Taylor
On Wed, Jul 24, 2013 at 4:49 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: Ian Lance Taylor i...@google.com writes: On Thu, Jul 18, 2013 at 6:59 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: Ian Lance Taylor i...@google.com writes: I have committed a large patch to update

Re: [PATCH, i386, MPX 1/X] Support of Intel MPX ISA

2013-07-24 Thread Ilya Enkovich
2013/7/24 Joseph S. Myers jos...@codesourcery.com: On Wed, 24 Jul 2013, Ilya Enkovich wrote: Here is a patch which adds support for new instructions from Intel Memory Protection Extensions (MPX) ISA [1] This patch introduces bound type, modes, registers and all MPX instructions. Control

Go patch committed: Fix handling of zero sized struct

2013-07-24 Thread Ian Lance Taylor
When a Go global variable has zero size, we fudge it to make it larger to avoid problems with the linker. This turned out to cause a bug with a struct with a zero-sized field, since code still expected to be able to access the field. This patch changes the zero size handling to recreate such

Merge from gcc 4.8 branch to gccgo branch

2013-07-24 Thread Ian Lance Taylor
I merged gcc 4.8 branch revision 201223 to the gccgo branch. Ian

Re: RFA: Fix rtl-optimization/57968

2013-07-24 Thread Jeff Law
On 07/24/2013 06:07 AM, Joern Rennecke wrote: The problem is fixed by simply moving the code that skips insns that don't set a part of the return value below the code that checks if the insn needs a non-MODE_EXIT mode setting. bootstrapped/regtested on i686-pc-linux-gnu 2013-07-24 Joern

[PATCH, PowerPC] Fix vector pack insns for little endian

2013-07-24 Thread Bill Schmidt
In little endian mode, we need to reverse the order of operands for vector pack operations. This corrects 20 failures in the test suite in LE mode. Patch based on work by Anton Blanchard; I simplified it and generalized it to all the vector pack insns. Bootstrapped and tested on

Re: msp430 port

2013-07-24 Thread Jeff Law
On 07/19/2013 05:32 PM, DJ Delorie wrote: Every pattern that is using (subreg:SI (thing:PSI)) needs to be explained on this list and given an explicit clearance. It really looks like you're just papering over problems elsewhere. Most of them are just optimizations, but the problem with reload

Re: [PATCH, PowerPC] Fix vector pack insns for little endian

2013-07-24 Thread David Edelsohn
On Wed, Jul 24, 2013 at 5:05 PM, Bill Schmidt wschm...@linux.vnet.ibm.com wrote: In little endian mode, we need to reverse the order of operands for vector pack operations. This corrects 20 failures in the test suite in LE mode. Patch based on work by Anton Blanchard; I simplified it and

[Patch, Fortran] PR 57639: [OOP] ICE with polymorphism (and illegal code)

2013-07-24 Thread Janus Weil
Hi all, here is a straightforward patch for an ICE-on-invalid problem, which basically adds some checks for 'class_ok'. Regtested on x86_64-unknown-linux-gnu. Ok for trunk? Cheers, Janus 2013-07-24 Janus Weil ja...@gcc.gnu.org PR fortran/57639 * interface.c (compare_parameter):

Re: [Patch, Fortran] PR 57639: [OOP] ICE with polymorphism (and illegal code)

2013-07-24 Thread Steve Kargl
On Wed, Jul 24, 2013 at 11:53:09PM +0200, Janus Weil wrote: Hi all, here is a straightforward patch for an ICE-on-invalid problem, which basically adds some checks for 'class_ok'. Regtested on x86_64-unknown-linux-gnu. Ok for trunk? OK. -- Steve

[google][4.8] Expose all target specific builtins unconditionally for LIPO builds

2013-07-24 Thread Sriraman Tallam
The following test-case fails in LIPO mode during profile-use build: main.cc __attribute__((target(sse4.2))) unsigned int problem_aux (unsigned int A, unsigned int B); int main (int argc, char *argv[]) { return problem_aux (0, 0); } aux.cc -

Re: [google][4.8] Expose all target specific builtins unconditionally for LIPO builds

2013-07-24 Thread Xinliang David Li
Can you collect some number on ggc_memory increase with this change in profile_gen build -- the value is recorded gcov_module_info object in coverage.c. thanks, David On Wed, Jul 24, 2013 at 3:14 PM, Sriraman Tallam tmsri...@google.com wrote: The following test-case fails in LIPO mode during

[C++ Patch] PR 57880

2013-07-24 Thread Paolo Carlini
Hi, submitter ran a static analysis tool on parser.c and found in the main switch of cp_parser_operator some broken cases having to do with user defined literals. When I looked into it I found a rather convoluted logic which tried to possibly emit *both* the error message for and invalid

libgo patch committed: Handle allocating memory in cgo/SWIG return

2013-07-24 Thread Ian Lance Taylor
When a function created by cgo or SWIG returns an interface type and at runtime returns a value that requires allocation when converting to interface, it will call the memory allocation routine while appearing to be in a system call. In particular there will be no memory cache allocated. This

Merge from gcc 4.8 branch to gccgo branch

2013-07-24 Thread Ian Lance Taylor
I merged gcc 4.8 branch revision 201227 to the gccgo branch. Ian

Re: [PATCH 1/2] Introduce context class

2013-07-24 Thread Diego Novillo
On Wed, Jul 24, 2013 at 11:09 AM, David Malcolm dmalc...@redhat.com wrote: gcc/ Introduce context class. * Makefile.in (CONTEXT_H): New. (OBJS): Add context.o. (toplev.o): Add CONTEXT_H to dependencies. (context.o): New. * toplev.c

Re: [PATCH 2/2] Introduce beginnings of a pipeline class.

2013-07-24 Thread Diego Novillo
Could you please add a description of what this does? On Wed, Jul 24, 2013 at 11:09 AM, David Malcolm dmalc...@redhat.com wrote: gcc/ * Makefile.in (PIPELINE_H): New. (lto-cgraph.o): Depend on CONTEXT_H and PIPELINE_H. (passes.o): Likewise. (statistics.o):

Re: [PATCH, i386, MPX 1/X] Support of Intel MPX ISA

2013-07-24 Thread Joseph S. Myers
On Wed, 24 Jul 2013, Ilya Enkovich wrote: Well, this patch does not introduce any changes on user-visible level. It just adds MPX instructions support to i386 target. Usually each new x86 instruction has corresponding builtin function and therefore is provided with a testcase. But MPX

Re: [PATCH 2/2] Introduce beginnings of a pipeline class.

2013-07-24 Thread Diego Novillo
On Wed, Jul 24, 2013 at 6:56 PM, Diego Novillo dnovi...@google.com wrote: Could you please add a description of what this does? Sorry. You did, but in a previous message that I had managed to miss. Maybe include a reference to it in future postings? Diego.

Re: [PATCH 1/2] Introduce context class

2013-07-24 Thread David Malcolm
On Wed, 2013-07-24 at 18:55 -0400, Diego Novillo wrote: On Wed, Jul 24, 2013 at 11:09 AM, David Malcolm dmalc...@redhat.com wrote: gcc/ Introduce context class. * Makefile.in (CONTEXT_H): New. (OBJS): Add context.o. (toplev.o): Add CONTEXT_H to

Re: [PATCH 2/2] Introduce beginnings of a pipeline class.

2013-07-24 Thread David Malcolm
On Wed, 2013-07-24 at 19:10 -0400, Diego Novillo wrote: On Wed, Jul 24, 2013 at 6:56 PM, Diego Novillo dnovi...@google.com wrote: Could you please add a description of what this does? Sorry. You did, but in a previous message that I had managed to miss. Maybe include a reference to it in

Re: [Patch, PR 57800] Waste work in gfc_match_call()

2013-07-24 Thread pchang9
On 23 July 2013 23:32:27 pcha...@cs.wisc.edu wrote: Hi, The problem appears in revision 201034 in version 4.9. I attached a one-line patch that fixes it. I also reported this problem at http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57800 Bootstrap and regression-tested on x86_64-linux. In

Re: [C++ Patch] PR 57880

2013-07-24 Thread Jason Merrill
You need to have tests for the other string prefixes, since you're changing the code for them. Jason

Re: [C++ Patch] PR 57880

2013-07-24 Thread Paolo Carlini
On 07/25/2013 03:35 AM, Jason Merrill wrote: You need to have tests for the other string prefixes, since you're changing the code for them. Oh yes, something like this? (believe me, I had it, but then the last minute I simplified the testcase because the last lines didn't seem to exercise any