Re: New parameters to control stringop expansion libcall strategy

2013-08-07 Thread Xinliang David Li
Updated patch attached (fixed header, buffer overflow, and warning --> error problems). Ok for trunk? thanks, David On Wed, Aug 7, 2013 at 6:04 PM, Joseph S. Myers wrote: > On Wed, 7 Aug 2013, Xinliang David Li wrote: > >> > Why the exception? This should only be used on the host, not the tar

-mtune-ctrl=.... support, round-2

2013-08-07 Thread Xinliang David Li
Hi, the attached is a follow up patch. The patch includes the following changes: 1. new option -mno-default -- suggested by H.J. It can be combined with -mtune-ctrl= to clear the default settings. 2. new option -mdump-tune-features -- this option can be used to dump all feature names and their def

Re: New parameters to control stringop expansion libcall strategy

2013-08-07 Thread Joseph S. Myers
On Wed, 7 Aug 2013, Xinliang David Li wrote: > > Why the exception? This should only be used on the host, not the target. > > Sorry, I copied the boiler-plate header from i386.h -- is it wrong there too? tm.h gets included in target code because we haven't finished separating target macros use

Re: [PATCH/Merge Request] Vtable Verification feature.

2013-08-07 Thread Joseph S. Myers
Looking at the patch as committed, there seems to be some confusion about the nature of the --enable-vtable-verify configure option. It's documented only for libstdc++. Now, I still consider the existence of separate documentation for libstdc++ configure options to be unfortunate - I think all

Re: New parameters to control stringop expansion libcall strategy

2013-08-07 Thread Xinliang David Li
On Wed, Aug 7, 2013 at 5:23 PM, Joseph S. Myers wrote: > On Wed, 7 Aug 2013, Xinliang David Li wrote: > >> Index: config/i386/stringop.def >> === >> --- config/i386/stringop.def (revision 0) >> +++ config/i386/stringop.def (revision

Re: New parameters to control stringop expansion libcall strategy

2013-08-07 Thread Joseph S. Myers
On Wed, 7 Aug 2013, Xinliang David Li wrote: > Index: config/i386/stringop.def > === > --- config/i386/stringop.def (revision 0) > +++ config/i386/stringop.def (revision 0) > @@ -0,0 +1,42 @@ > +/* Definitions for option handling fo

Re: New option to do fine grain control [on|off] of micro-arch specific features : -mtune-ctrl=....

2013-08-07 Thread Xinliang David Li
On Wed, Aug 7, 2013 at 4:54 PM, Joseph S. Myers wrote: > On Sun, 4 Aug 2013, Andi Kleen wrote: > >> Richard Biener writes: >> > >> > The patch fails to add documentation. >> >> That seems like a feature, it's likely not useful for the general >> public. More for specialized tools that automatical

Re: [Patch] Whole regex refactoring and current status

2013-08-07 Thread Tim Shen
On Wed, Aug 7, 2013 at 12:04 AM, Paolo Carlini wrote: > By the way, your last patch is fine with me, I like the new *.tcc a lot. > Just allow a day or so for further comments.. Committed. -- Tim Shen

Re: New option to do fine grain control [on|off] of micro-arch specific features : -mtune-ctrl=....

2013-08-07 Thread Joseph S. Myers
On Sun, 4 Aug 2013, Andi Kleen wrote: > Richard Biener writes: > > > > The patch fails to add documentation. > > That seems like a feature, it's likely not useful for the general > public. More for specialized tools that automatically search > for the best tuning. If such a tool is not part of

[vtv] fix default configure

2013-08-07 Thread Benjamin De Kosnik
Here's a patch for the build failure on darwin. tested x86/linux -benjamindiff --git a/libvtv/Makefile.am b/libvtv/Makefile.am index ab3233f..73acfb4 100644 --- a/libvtv/Makefile.am +++ b/libvtv/Makefile.am @@ -19,7 +19,11 @@ ## along with this library; see the file COPYING3. If not see ##

Re: [PATCH] - C11 expressions and stdatomic.h - Just for current state

2013-08-07 Thread Joseph S. Myers
On Fri, 2 Aug 2013, Andrew MacLeod wrote: > The atomic expression patch works on many cases, but falls short with any kind > of complexity on the RHS of an expression which needs to be mapped to an > atomic_load. Whilst parsing, we cant tell, and the rhs tends to become an > expression before we

Re: [PATCH/Merge Request] Vtable Verification feature.

2013-08-07 Thread Michael Meissner
On Wed, Aug 07, 2013 at 02:29:35PM +0200, Dominique Dhumieres wrote: > Revision 201555 breaks boostrap on x86_64-apple-darwin10: > > ... > Checking multilib configuration for libvtv... > make all-recursive > Making all in testsuite > /bin/sh: line 0: cd: testsuite: No such file or directory > mak

[patch, mips] Add nan2008 multilibs to mips-mti-* targets.

2013-08-07 Thread Steve Ellcey
I would like to add new variations to the mips-mti-elf and mips-mti-linux targets to support the new NaN format on MIPS (mnan=2008 for IEEE 754-2008). While I was doing this I noticed that I was handling mips16 and micromips in a dumb manner. I had them in MULTILIB_OPTIONS as non-exclusive flags a

Re: [PATCH/Merge Request] Vtable Verification feature.

2013-08-07 Thread Caroline Tice
Thank you! -- Caroline On Wed, Aug 7, 2013 at 3:09 PM, Paolo Carlini wrote: > .. I removed it. > > Thanks, > Paolo.

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

2013-08-07 Thread Joseph S. Myers
On Fri, 2 Aug 2013, Ilya Enkovich wrote: > Hi All, > > I've updated MPX Wiki page > (http://gcc.gnu.org/wiki/Intel%20MPX%20support%20in%20the%20GCC%20compiler). > I added instrumentation description, programming model description, > differences with other checkers, implementation details. Thanks

Re: [PATCH/Merge Request] Vtable Verification feature.

2013-08-07 Thread Paolo Carlini
.. I removed it. Thanks, Paolo.

Re: [PATCH/Merge Request] Vtable Verification feature.

2013-08-07 Thread Caroline Tice
+ gcc_patches list On Wed, Aug 7, 2013 at 2:56 PM, Caroline Tice wrote: > No, it was not intended. How do I undo that? > > -- Caroline > cmt...@google.com > > On Wed, Aug 7, 2013 at 2:54 PM, David Edelsohn wrote: >> Caroline, >> >> When libvtv was checked in, libvtv/autom4te.cache subdirectory

Re: [PATCH/Merge Request] Vtable Verification feature.

2013-08-07 Thread David Edelsohn
Caroline, When libvtv was checked in, libvtv/autom4te.cache subdirectory was committed as well. I don't think that this was intended. - David

Re: [PATCH, libstdc++]: Avoid -Wcast-qual warnings in src/c++98/compatibility.cc

2013-08-07 Thread Paolo Carlini
Hi, On 08/07/2013 10:48 PM, Uros Bizjak wrote: 2013-08-07 Uros Bizjak * src/c++98/compatibility.cc (_ZTIe): Use const_cast to avoid warning. (_ZTIPe): Ditto. (ZTIPKe): Ditto. The patch was bootstrapped on alpha-linux-gnu, regression test is still running. OK for mainline if

Aw: RFC/A: PR 58079: CONST_INT mode assert in combine.c:do_SUBST

2013-08-07 Thread Jürgen Urban
> PR 58079 is about the do_SUBST assert: > > /* Sanity check that we're replacing oldval with a CONST_INT >that is a valid sign-extension for the original mode. */ > gcc_assert (INTVAL (newval) > == trunc_int_for_mode (INTVAL (newval), GET_MODE (oldval))); > >

[PATCH, libstdc++]: Avoid -Wcast-qual warnings in src/c++98/compatibility.cc

2013-08-07 Thread Uros Bizjak
Hello! Attached patch avoids a bunch of -Wcast-qual warnings when building libstdc++ on alpha-linux-gnu, a _GLIBCXX_LONG_DOUBLE_COMPAT target: ../../../../gcc-svn/trunk/libstdc++-v3/src/c++98/compatibility.cc:521:16: warning: cast from type ‘const char*’ to type ‘void*’ casts away qualifiers [-Wc

[ping] 3 pending patches

2013-08-07 Thread Eric Botcazou
Fix error recovery issue with alias: http://gcc.gnu.org/ml/gcc-patches/2013-06/msg01417.html Improve debug info for small structures passed by reference: http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00895.html Implement static stack checking for the ARM: http://gcc.gnu.org/ml/gcc-patches/20

Re: [buildbot] r201508: Build failures after pass C++ conversion

2013-08-07 Thread David Malcolm
/xgcc -B../../ -c -DIN_GCC -DUSE_LIBUNWIND_EXCEPTIONS -O2 -g -W -Wall \ > -iquote /usr/local/gcc/gcc-20130807/gcc \ > -iquote . -iquote .. -iquote ../.. -iquote > /usr/local/gcc/gcc-20130807/gcc/ada -iquote /usr/local/gcc/gcc-20130807/gcc > -I/usr/local/

Re: [C++ Patch / RFC] PR 54864

2013-08-07 Thread Paolo Carlini
Hi, On 08/07/2013 09:52 PM, Jason Merrill wrote: The rule that allows use of *this is pretty specific: Unlike the object expression in other contexts, *this is not required to be of complete type for purposes of class member access (5.2.5) outside the member function body. That doesn't seem

Re: MIPS tests (nan-legacy.c and nans-legacy.c) failing

2013-08-07 Thread Richard Sandiford
"Steve Ellcey " writes: > Maciej, > > When I run two of your new tests in gcc.target/mips (nan-legacy.c and > nans-legacy.c), they are failing because my GCC is putting out > > .word 4294967295 > > instead of > > .word -1 > > like the test is expecting. > > I believe they are equi

Re: [PATCH] MIPS/libgcc: Avoid the PLT in MIPS16 stub calls

2013-08-07 Thread Richard Sandiford
"Maciej W. Rozycki" writes: > BTW, what's the "Check for MicroMIPS support." note seen in the > config.host piece of the patch referring to? No idea, please remove it. > /* Define a function NAME that moves a return value of mode MODE from > FPRs to GPRs. */ > > -#define RET_FUNCTION(N

Re: [PATCH i386 2/8] [AVX512] Add mask registers.

2013-08-07 Thread Kirill Yukhin
Hello! > You can't read from memory into a mask register in QImode. > This will fail if the memory was the last byte of a page, > and the following page is not mapped. > > I expected you to need the following patch, to help spill > and fill QImode values, but I havn't found a test case that > ac

Re: [FIXED] Generic lambda symbol table bug

2013-08-07 Thread Adam Butcher
On 07.08.2013 16:59, Jason Merrill wrote: On 08/07/2013 03:52 AM, Adam Butcher wrote: But a cleaner way might be to extend the "processing template declaration" state from lambda declarator all the way to the end of the lambda body. This would match with the scenario that occurs with a standa

Re: [C++ Patch / RFC] PR 54864

2013-08-07 Thread Jason Merrill
On 08/07/2013 03:17 PM, Paolo Carlini wrote: the issue here is that... I'm not sure the bug report is valid ;) Seriously, *if* we think it is, must be fixable with a moderate effort. Here Jason fixed the closely related c++/53721: http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01445.html and

Re: PPC64 HTM support (was [buildbot] r201513: Invalid cast)

2013-08-07 Thread Oleg Endo
On Tue, 2013-08-06 at 22:01 -0500, Peter Bergner wrote: > Oleg Endo wrote: > > Speaking of GEN_FCN usage in rs6000.c. The recently added HTM builtin > > code has one interesting piece: > > > > static rtx > > htm_expand_builtin (tree exp, rtx target, bool * expandedp) > > { > > ... > > switch (nop

Re: [PATCH] Fix illegal cast to rtx (*insn_gen_fn) (rtx, ...)

2013-08-07 Thread Oleg Endo
On Wed, 2013-08-07 at 15:08 -0400, Michael Meissner wrote: > On Tue, Aug 06, 2013 at 11:45:40PM +0200, Oleg Endo wrote: > > On Mon, 2013-08-05 at 13:25 -1000, Richard Henderson wrote: > > > On 08/05/2013 12:32 PM, Oleg Endo wrote: > > > > Thanks, committed as rev 201513. > > > > 4.8 also has the sa

[C++ Patch / RFC] PR 54864

2013-08-07 Thread Paolo Carlini
Hi, the issue here is that... I'm not sure the bug report is valid ;) Seriously, *if* we think it is, must be fixable with a moderate effort. Here Jason fixed the closely related c++/53721: http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01445.html and tweaking a bit more the check would al

Re: [PATCH] Fix illegal cast to rtx (*insn_gen_fn) (rtx, ...)

2013-08-07 Thread Michael Meissner
On Tue, Aug 06, 2013 at 11:45:40PM +0200, Oleg Endo wrote: > On Mon, 2013-08-05 at 13:25 -1000, Richard Henderson wrote: > > On 08/05/2013 12:32 PM, Oleg Endo wrote: > > > Thanks, committed as rev 201513. > > > 4.8 also has the same problem. The patch applies on 4.8 branch without > > > problems a

Go patch committed: Fix missing return error for type switches

2013-08-07 Thread Ian Lance Taylor
The Go frontend gave an incorrect "missing return" error message for a type switch with a case with multiple types, as in "case T1, T2:". This patch fixes that problem. This patch also changes the "missing return" error message to use the same text as the gc compiler, and reports it at the end of

Re: [patch] Make cxxfilt demangle internal-linkage templates

2013-08-07 Thread Paul Pluzhnikov
On Wed, Aug 7, 2013 at 11:34 AM, Andrew Pinski wrote: > I think this should also be send to the GCC List as libiberty is > officially maintained by GCC. http://gcc.gnu.org/ml/gcc-patches/2013-08/msg00394.html Thanks, -- Paul Pluzhnikov

[patch] Make cxxfilt demangle internal-linkage templates

2013-08-07 Thread Paul Pluzhnikov
Greetings, I've been redirected here from binutils: http://sourceware.org/ml/binutils/2013-08/msg00052.html http://sourceware.org/ml/binutils/2013-08/msg00056.html The following source: template static void f(); void g() { f(); } results in "_Z1fIiEvv" under g++, but in "_ZL1fIiEvv" under c

Re: i686 pic & DCmode

2013-08-07 Thread Ian Lance Taylor
On Wed, Aug 7, 2013 at 10:34 AM, Nathan Sidwell wrote: > Having poked further, I find this in the testsuite: > > pr11001-strlen-2.c > --begin > register int regvar asm("%eax"); > > char * > do_copy (char *str) > { > return malloc (strlen (str) + 1); > } > --end > > Is that even meaningful? The

Re: [patch] Make cxxfilt demangle internal-linkage templates

2013-08-07 Thread Andrew Pinski
On Wed, Aug 7, 2013 at 11:31 AM, Paul Pluzhnikov wrote: > On Wed, Aug 7, 2013 at 11:09 AM, Jan Kratochvil > wrote: > >> there should be a new case in: >> libiberty/testsuite/demangle-expected > > Thanks. I updated the patch. > -- > Paul Pluzhnikov I think this should also be send to the

[patch][google] Update libtool.m4 in google/gcc-4_8 branch.

2013-08-07 Thread Brooks Moses
A not-yet-reviewed libtool patch is needed for powerpc64le shared-library support: http://lists.gnu.org/archive/html/libtool-patches/2013-06/msg1.html (See also http://lists.gnu.org/archive/html/libtool-patches/2013-07/msg1.html) This patch merges that change into the google/gcc-4_8 branc

Re: PATCH: PR debug/54568: --eh-frame-hdr should also be enabled for static executable

2013-08-07 Thread H.J. Lu
On Thu, Sep 13, 2012 at 7:46 PM, H.J. Lu wrote: > Hi, > > There is no reason why --eh-frame-hdr can't be used with static > executable on Linux. This patch enables --eh-frame-hdr for static > executable on Linux and adds an exception test for static executable. > Other platforms may also work cor

Re: [GOOGLE] Refactor AutoFDO

2013-08-07 Thread Xinliang David Li
ok for google branch. David On Tue, Aug 6, 2013 at 3:41 PM, Dehao Chen wrote: > Patch updated. > > http://codereview.appspot.com/12079043 > > Thanks, > Dehao > > > On Fri, Aug 2, 2013 at 11:21 AM, Xinliang David Li wrote: >> More to follow. >> >> David >> static void read_profile (void

Re: [PATCH/Merge Request] Vtable Verification feature.

2013-08-07 Thread Caroline Tice
Hi Iain, Thanks for the pointer (I had noticed this but I appreciate the help!). I do not know of anyone working on a Darwin port for this at this time. -- Caroline Tice cmt...@google.com On Wed, Aug 7, 2013 at 10:31 AM, Iain Sandoe wrote: > hi Caroline, > > A (very) quick look suggests that

Re: i686 pic & DCmode

2013-08-07 Thread Nathan Sidwell
Having poked further, I find this in the testsuite: pr11001-strlen-2.c --begin register int regvar asm("%eax"); char * do_copy (char *str) { return malloc (strlen (str) + 1); } --end Is that even meaningful? The doc's for a global reg var say: 'Choose a register that is normally saved and re

small patch to accept = inside plugin arguments

2013-08-07 Thread Basile Starynkevitch
Hello All, It should be perfectly possible for a plugin argument to contain the equal sign. For plugin with scripting or DSL behavior (like MELT, see http://gcc-melt.org/ for more) it is useful to pass some commands or expressions to the plugin. For MELT we might want to run gcc -fplugin=

Re: small patch to accept = inside plugin arguments

2013-08-07 Thread David Malcolm
On Wed, 2013-08-07 at 19:15 +0200, Basile Starynkevitch wrote: > Hello All, > > It should be perfectly possible for a plugin argument to contain the > equal sign. > > For plugin with scripting or DSL behavior (like MELT, see > http://gcc-melt.org/ for more) it is useful to pass some commands > o

Re: [PATCH/Merge Request] Vtable Verification feature.

2013-08-07 Thread Iain Sandoe
hi Caroline, A (very) quick look suggests that only *) results in a return of UNSUPPORTED from libvtv/configure.tgt Do you know if anyone is working on a Darwin port for this lib? thanks, Iain On 7 Aug 2013, at 17:57, Caroline Tice wrote: > libvtv was supposed to be automatically disabled fo

[PATCH] Add vtable verification feature announcement to news on main page...

2013-08-07 Thread Caroline Tice
As requested, here is the patch to announce the vtable verification feature on the main gcc.gnu.org web page. Is this ok to commit? -- Caroline Tice cmt...@google.com wwwdocs.patch Description: Binary data

Re: i686 pic & DCmode

2013-08-07 Thread Eric Botcazou
> The usual x86 ports avoid this, because their must_return_in_mem hooks DTRT > already and we never fall into the default case. So why not do the same for i686-elf? > I am very surprised this hasn't bitten someone before -- I presume this > never worked with i686-elf. It is possible this'll cha

Re: New parameters to control stringop expansion libcall strategy

2013-08-07 Thread Xinliang David Li
Fixed the do while formatting. Ok for trunk with this version? thanks, David On Tue, Aug 6, 2013 at 2:42 AM, Jan Hubicka wrote: >> >>> 2013-08-02 Xinliang David Li >> >>> >> >>> * config/i386/stringop.def: New file. >> >>> * config/i386/stringop.opt: New file. >> >>>

Re: [PATCH/Merge Request] Vtable Verification feature.

2013-08-07 Thread Caroline Tice
libvtv was supposed to be automatically disabled for darwin; we are in the process of trying to figure out why this did not work as it was supposed to. In the meantime, if you add "--disable-libvtv" explicitly to your configure command, that should turn off the attempts to configure/build it. I'l

Re: Fix PR middle-end/57370

2013-08-07 Thread Easwaran Raman
Ping. On Wed, Jul 31, 2013 at 4:34 PM, Easwaran Raman wrote: > I have a new patch that supersedes this. The new patch also fixes PR > tree-optimization/57393 and PR tree-optimization/58011. Bootstraps and > no test regression on x86_64/linux. Ok for trunk? > > 2013-07-31 Easwaran Raman > >

Re: New option to do fine grain control [on|off] of micro-arch specific features : -mtune-ctrl=....

2013-08-07 Thread Xinliang David Li
On Wed, Aug 7, 2013 at 8:37 AM, Jan Hubicka wrote: >> Ping? >> >> David >> >> On Sat, Aug 3, 2013 at 12:36 PM, Xinliang David Li >> wrote: >> > Hi, GCC/i386 currently has about 73 boolean parameters/knobs (defined >> > in ix86_tune_features[], indexed by ix86_tune_indices) to perform >> > micro-

Re: Request to merge Undefined Behavior Sanitizer in (take 2)

2013-08-07 Thread Marek Polacek
On Wed, Aug 07, 2013 at 10:24:59AM -0400, Jason Merrill wrote: > On 08/07/2013 06:06 AM, Marek Polacek wrote: > >I might've misunderstood what you mean. If we drop the hunk above, > >then we'll call > > error ("%q+E is not a constant expression", t); > >so, we'll print "is not a constant expre

Re: [FIXED] Generic lambda symbol table bug

2013-08-07 Thread Jason Merrill
On 08/07/2013 03:52 AM, Adam Butcher wrote: But a cleaner way might be to extend the "processing template declaration" state from lambda declarator all the way to the end of the lambda body. This would match with the scenario that occurs with a standard in-class member function template definiti

Re: avx512 mask register spilling

2013-08-07 Thread Vladimir Makarov
On 13-08-06 3:11 PM, Richard Henderson wrote: On 08/06/2013 03:57 AM, Kirill Yukhin wrote: On 05 Aug 09:55, Richard Henderson wrote: On 08/05/2013 08:07 AM, Kirill Yukhin wrote: Hello Richard, Vlad, On 31 Jul 06:26, Richard Henderson wrote: On 07/31/2013 05:02 AM, Kirill Yukhin wrote: There

i686 pic & DCmode

2013-08-07 Thread Nathan Sidwell
In poking at another set of fails encountered with an i686-elf toolchain (http://gcc.gnu.org/ml/gcc/2013-08/msg00081.html), I find that complex double returns and PIC do not play together. Libgcc's __muldc3 was attempting to return the value in registers, whereas every caller expected it to be

Re: [PATCH] Fix broken build on sparc (was Re: [buildbot] r201508: Build failures after pass C++ conversion)

2013-08-07 Thread David Malcolm
On Wed, 2013-08-07 at 15:25 +0200, Eric Botcazou wrote: > Looks good, please install if not already done. Thanks; I've now committed this to trunk as r201569.

Re: New option to do fine grain control [on|off] of micro-arch specific features : -mtune-ctrl=....

2013-08-07 Thread Jan Hubicka
> Ping? > > David > > On Sat, Aug 3, 2013 at 12:36 PM, Xinliang David Li wrote: > > Hi, GCC/i386 currently has about 73 boolean parameters/knobs (defined > > in ix86_tune_features[], indexed by ix86_tune_indices) to perform > > micro-arch specific performance tuning. However such settings are ha

Re: New option to do fine grain control [on|off] of micro-arch specific features : -mtune-ctrl=....

2013-08-07 Thread Xinliang David Li
Ping? David On Sat, Aug 3, 2013 at 12:36 PM, Xinliang David Li wrote: > Hi, GCC/i386 currently has about 73 boolean parameters/knobs (defined > in ix86_tune_features[], indexed by ix86_tune_indices) to perform > micro-arch specific performance tuning. However such settings are hard > coded (fixe

Re: Request to merge Undefined Behavior Sanitizer in (take 2)

2013-08-07 Thread Marek Polacek
On Tue, Aug 06, 2013 at 05:24:08PM -0400, Jason Merrill wrote: > On 08/05/2013 07:24 AM, Marek Polacek wrote: > >On Sat, Aug 03, 2013 at 12:24:32PM -0400, Jason Merrill wrote: > >>Where are the SAVE_EXPRs coming from? It doesn't seem to me that x > >>needs to be wrapped in a SAVE_EXPR at all in th

RE: [PATCH][tree-optimization] Fix PR58088

2013-08-07 Thread Kyrylo Tkachov
> This issue is present in 4.8.1 as well as trunk. However, I think 4.8 > uses a different API for double_int, so this patch will need to be reworked for > 4.8. Actually, I was confused. This patch applies to 4.8.1 as well and fixes the issue. Passes bootstrap on x86_64-linux-gnu and testing arm-n

Re: Request to merge Undefined Behavior Sanitizer in (take 2)

2013-08-07 Thread Jason Merrill
On 08/07/2013 06:06 AM, Marek Polacek wrote: I might've misunderstood what you mean. If we drop the hunk above, then we'll call error ("%q+E is not a constant expression", t); so, we'll print "is not a constant expression" no matter what Yes, that's fine; 1/0 is not a constant expression,

[Patch, Fortran] PR 58099: [4.8/4.9 Regression] [F03] over-zealous procedure-pointer error checking

2013-08-07 Thread Janus Weil
Hi all, here is a small regression-fix patch for a problem with procedure pointers and the PURE attribute, for details see the PR. In essence: gfc_compare_interfaces is asymmetric in the two interfaces it compares (e.g. regarding the PURE attribute), and we must not symmetrize it by calling it twi

Re: [patch, ARM] require 64-bit hw-int for all arm targets

2013-08-07 Thread Richard Earnshaw
On 07/08/13 01:25, Joseph S. Myers wrote: > On Tue, 30 Jul 2013, Richard Earnshaw wrote: > >> Most arm target configs now require a 64-bit HW-int. Unfortunately a few of >> the older, less commonly used config targets do not. The code in arm.c now >> pretty much requires that a 64-bit HW-int is

lm32 build error

2013-08-07 Thread Joel Sherrill
Hi This has been around for a while as PR52466 which has a patch. I have attached the patch. Is it OK to apply to the open branches? 2013-08-07 Joel Sherrill * configure.ac (lm32*-*-*): Use SJLJ exceptions. -- Joel Sherrill, Ph.D. Director of Research & Development joel.sher

Re: [PATCH] Fix broken build on sparc (was Re: [buildbot] r201508: Build failures after pass C++ conversion)

2013-08-07 Thread Eric Botcazou
> I think it's r201508, but in any case, I'm attaching a patch which fixes > this build error. Only very lightly tested so far, with configure > --target=sparc-linux with build&host x86_64. Was able to build a cc1 > and step through the changed code in the debugger, though am getting > "cc1: erro

Re: PPC64 HTM support (was [buildbot] r201513: Invalid cast)

2013-08-07 Thread Peter Bergner
On Tue, 2013-08-06 at 22:01 -0500, Peter Bergner wrote: > I'll bootstrap and regtest the change > below and commit it as an obvious fix when that is done. Everything looked as as expected. Committed. Peter

[PATCH][tree-optimization] Fix PR58088

2013-08-07 Thread Kyrylo Tkachov
Hi all, In PR58088 the constant folder goes into an infinite recursion and runs out of stack space because of two conflicting optimisations: (X * C1) & C2 plays dirty when nested inside an IOR expression like so: ((X * C1) & C2) | C4. One can undo the other leading to an infinite recursion. Thank

Re: Passes are now C++ classes (was Re: [PATCH 03/11] Handwritten part of conversion of passes to C++ classes)

2013-08-07 Thread Eric Botcazou
> The mips*-*-* targets are not building. It looks like the mips reorg > pass (pass_mips_machine_reorg2) in config/mips/mips.c was not converted > and/or was not converted correctly. Likewise for the SPARC targets, because of the same issue. David, could you take a quick look? Thanks in advanc

Re: [PATCH/Merge Request] Vtable Verification feature.

2013-08-07 Thread Dominique Dhumieres
Revision 201555 breaks boostrap on x86_64-apple-darwin10: ... Checking multilib configuration for libvtv... make all-recursive Making all in testsuite /bin/sh: line 0: cd: testsuite: No such file or directory make[4]: *** [all-recursive] Error 1 make[3]: *** [all] Error 2 make[2]: *** [all-stage1

[v3] Fix libstdc++/58098

2013-08-07 Thread Paolo Carlini
Hi, I'm applying this mainline and 4_8-branch. Tested x86_64-linux. Thanks, Paolo. // 2013-08-07 Paolo Carlini PR libstdc++/58098 * include/bits/random.h (cauchy_distribution<>::min, extreme_value_distribution<>::min, normal_distribution<>::min,

Re: RFC/A: PR 58079: CONST_INT mode assert in combine.c:do_SUBST

2013-08-07 Thread Richard Biener
Richard Sandiford wrote: >PR 58079 is about the do_SUBST assert: > > /* Sanity check that we're replacing oldval with a CONST_INT >that is a valid sign-extension for the original mode. */ > gcc_assert (INTVAL (newval) > == trunc_int_for_mode (INTVAL (newval), GET

[c++-concepts] Merge from trunk

2013-08-07 Thread Gabriel Dos Reis
trunk was merged into c++-concepts branch at revision 201560. I resolved some conflicts in cp caused by recent merges and ongoing work on trunk. -- Gaby

Re: Go patch committed: Always put immutable structs in unique section

2013-08-07 Thread Uros Bizjak
On Tue, Aug 6, 2013 at 6:42 PM, Ian Lance Taylor wrote: > I committed this patch that I think should fix this problem. > Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. > Committed to mainline and 4.8 branch. > > 2013-08-06 Ian Lance Taylor > > * go-gcc.cc (Gcc_backend::

Re: Request to merge Undefined Behavior Sanitizer in (take 2)

2013-08-07 Thread Marek Polacek
On Tue, Aug 06, 2013 at 07:07:27PM -0400, Jason Merrill wrote: > >I think, what we could do, is to tweak verify_constant like this: > > > >+ /* This is to handle e.g. the goofy 'case 0 * (1 / 0)' case. */ > >+ if (flag_sanitize & SANITIZE_UNDEFINED > >+ && TREE_CODE (t) == CALL_EXPR > >+

Re: [C++ Patch / RFC] PR 46206

2013-08-07 Thread Paolo Carlini
Hi, On 08/07/2013 01:08 AM, Jason Merrill wrote: On 08/06/2013 06:14 PM, Paolo Carlini wrote: Ah I see, thanks. We reject this before and after the patch. Shall I add this variant to the new testcase? Sure. Thanks. Thus after a further round of testing I'm going to apply the below. Thanks, P

Re: [FIXED] Generic lambda symbol table bug

2013-08-07 Thread Adam Butcher
Hi Jason, On Mon, 05 Aug 2013 17:26:12 -0400, Jason Merrill wrote: > On 08/04/2013 07:45 PM, Adam Butcher wrote: > > What should I do about the symtab nullptr issue? > > (http://gcc.gnu.org/ml/gcc-patches/2013-07/msg00043.html) Should I > > leave the workaround in my patch set as a standalone com

Re: New parameters to control stringop expansion libcall strategy

2013-08-07 Thread Michael Zolotukhin
> the option is designed for purpose like this. That's great, thanks! Michael > David On 6 August 2013 20:42, Xinliang David Li wrote: > Corrected two small problems reported by the style checker (The > warnings about the EnumValue for options in stringopt.opt are not > valid). > > On Tue, Aug

[PATCH] target/58065 ARM MALLOC_ABI_ALIGNMENT is wrong

2013-08-07 Thread Bernd Edlinger
Hello, in the discussion about the PR middle-end/57748 it became obvious that the ARM target architecture should define a value for MALLOC_ABI_ALIGNMENT, because otherwise the default is simply word aligned, which causes sub-optimal code, at least for the neon fpu. This simple patch fixes PR t

Re: [buildbot] r201508: Build failures after pass C++ conversion

2013-08-07 Thread Andreas Schwab
class_t; > > +class rtl_opt_pass; > + > +namespace gcc { > + class context; > +} > + This break Ada. ../../xgcc -B../../ -c -DIN_GCC -DUSE_LIBUNWIND_EXCEPTIONS -O2 -g -W -Wall \ -iquote /usr/local/gcc/gcc-20130807/gcc \ -iquote . -iquote ..