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

2013-08-07 Thread Andreas Schwab
; +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 .. -iquote ../.. -iquote /usr/local/gcc

[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

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 davi...@google.com wrote: Corrected two small problems reported by the style checker (The warnings about the EnumValue for options in stringopt.opt are not valid).

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 commit to

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,

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: 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 i...@google.com 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 i...@google.com *

[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: RFC/A: PR 58079: CONST_INT mode assert in combine.c:do_SUBST

2013-08-07 Thread Richard Biener
Richard Sandiford rdsandif...@googlemail.com 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

[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 paolo.carl...@oracle.com PR libstdc++/58098 * include/bits/random.h (cauchy_distribution::min, extreme_value_distribution::min,

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]: ***

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 advance.

[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. Thanks

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

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 buildhost x86_64. Was able to build a cc1 and step through the changed code in the debugger, though am getting cc1: error: no

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 joel.sherr...@oarcorp.com * configure.ac (lm32*-*-*): Use SJLJ exceptions. -- Joel Sherrill, Ph.D. Director of

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 used,

[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

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,

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

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 this case.

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 davi...@google.com 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

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 davi...@google.com 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

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.

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

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:

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

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 expression no

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 hubi...@ucw.cz wrote: Ping? David On Sat, Aug 3, 2013 at 12:36 PM, Xinliang David Li davi...@google.com wrote: Hi, GCC/i386 currently has about 73 boolean parameters/knobs (defined in ix86_tune_features[], indexed by ix86_tune_indices) to

Re: Fix PR middle-end/57370

2013-08-07 Thread Easwaran Raman
Ping. On Wed, Jul 31, 2013 at 4:34 PM, Easwaran Raman era...@google.com 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

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'll

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 hubi...@ucw.cz wrote: 2013-08-02 Xinliang David Li davi...@google.com * config/i386/stringop.def: New file. * config/i386/stringop.opt: New file.

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 change

[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: [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

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 or

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

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: [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 i...@codesourcery.com wrote: hi Caroline, A (very) quick

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 de...@google.com wrote: Patch updated. http://codereview.appspot.com/12079043 Thanks, Dehao On Fri, Aug 2, 2013 at 11:21 AM, Xinliang David Li davi...@google.com wrote: More to follow. David static void

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 hongjiu...@intel.com 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

[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

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 ppluzhni...@google.com wrote: On Wed, Aug 7, 2013 at 11:09 AM, Jan Kratochvil jan.kratoch...@redhat.com wrote: there should be a new case in: libiberty/testsuite/demangle-expected Thanks. I updated the patch. -- Paul Pluzhnikov I

Re: i686 pic DCmode

2013-08-07 Thread Ian Lance Taylor
On Wed, Aug 7, 2013 at 10:34 AM, Nathan Sidwell nat...@acm.org 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

[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: templatetypename T static void f(); void g() { fint(); } results in _Z1fIiEvv under g++, but in

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 pins...@gmail.com 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

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 the

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 and make

[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

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 same problem.

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 (nopnds) {

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

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

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 actually

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

2013-08-07 Thread Richard Sandiford
Maciej W. Rozycki ma...@codesourcery.com 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

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

2013-08-07 Thread Richard Sandiford
Steve Ellcey sell...@mips.com 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

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

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

2013-08-07 Thread David Malcolm
*const_basic_block; in target.h. */ typedef int reg_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

[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:

[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

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)));

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 ubiz...@gmail.com * 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.

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/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 cmt...@google.com 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 dje@gmail.com wrote: Caroline, When libvtv was checked in,

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

2013-08-07 Thread Paolo Carlini
.. 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. As

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 paolo.carl...@oracle.com wrote: .. I removed it. Thanks, Paolo.

[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

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 make[4]:

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

[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: 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 richard.guent...@gmail.com 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

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 paolo.carl...@oracle.com 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 Xinliang David Li
On Wed, Aug 7, 2013 at 4:54 PM, Joseph S. Myers jos...@codesourcery.com wrote: On Sun, 4 Aug 2013, Andi Kleen wrote: Richard Biener richard.guent...@gmail.com writes: The patch fails to add documentation. That seems like a feature, it's likely not useful for the general public. More for

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 for

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 jos...@codesourcery.com wrote: On Wed, 7 Aug 2013, Xinliang David Li wrote: Index: config/i386/stringop.def === --- config/i386/stringop.def (revision 0) +++

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

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 used