[Ping] Allow dg-skip-if to use compiler flags specified through set_board_info cflags

2012-09-18 Thread Senthil Kumar Selvaraj
Hello, Could one of the maintainers please check in the patch below? It was already approved at http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00689.html Regards Senthil On Tue, Aug 21, 2012 at 03:08:43PM -0700, Mike Stump wrote: > On Aug 11, 2012, at 10:39 AM, Senthil Kumar Selvaraj wrote: > > T

Re: [patch, mips] Patch for new mips triplet - mips-mti-elf

2012-09-18 Thread Richard Sandiford
Steve Ellcey writes: > On Mon, 2012-09-17 at 21:36 +0100, Richard Sandiford wrote: >> It's a hosted vs. embedded thing. Hosted targets like *-linux-gnu have >> dynamic ABI requirements and so are keyed off an ABI rather than an >> architecture. The only effect of -march= and --with-arch= should

Re: [Patch, fortran] PR46897 - [OOP] type-bound defined ASSIGNMENT(=) not used for derived type component in intrinsic assign

2012-09-18 Thread Mikael Morin
On 17/09/2012 20:45, Mikael Morin wrote: >> *** resolve_fl_derived0 (gfc_symbol *sym) >> *** 12282,12289 >> --- 12558,12574 >> || c->attr.proc_pointer >> || c->attr.allocatable)) == FAILURE) >>

Re: [Patch ARM] big-endian support for Neon vext tests

2012-09-18 Thread Richard Earnshaw
On 17/09/12 20:04, Christophe Lyon wrote: > On 17 September 2012 20:04, Richard Earnshaw wrote: >> On 17/09/12 16:50, Christophe Lyon wrote: >>> On 17 September 2012 17:21, Richard Earnshaw wrote: On 17/09/12 16:13, Christophe Lyon wrote: > On 17 September 2012 14:56, Richard Earnshaw w

[libbacktrace] Fix bootstrap with gcc 4.4

2012-09-18 Thread Rainer Orth
The libbacktrace integration broke Solaris 10 and 11 bootstrap when using gcc 4.4 (any version of gcc without __sync_* support actually): stage1 config.h has /* #undef HAVE_SYNC_FUNCTIONS */ and fileline.c and mmap.c fail to compile: /vol/gcc/src/hg/trunk/local/libbacktrace/fileline.c: In funct

Re: [PATCH] PR other/54411: libiberty: objalloc_alloc integer overflows (CVE-2012-3509)

2012-09-18 Thread Florian Weimer
On 09/17/2012 05:59 PM, Ian Lance Taylor wrote: Fair enough. I've added a wraparound check to the macro. Okay for trunk? { + unsigned long len = original_len; /* We avoid confusion from zero sized objects by always allocating at least 1 byte. */ Please add a blank line afte

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Richard Guenther
On Mon, Sep 17, 2012 at 7:17 PM, Ian Lance Taylor wrote: > This patch to the diagnostic code uses the new backtrace library to > print a backtrace on an ICE. For example, here is the output of a test > case I took from a C++ PR: > > /home/iant/foo2.cc:6:6: internal compiler error: in cp_lexer_new

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Richard Guenther
On Tue, Sep 18, 2012 at 10:49 AM, Richard Guenther wrote: > On Mon, Sep 17, 2012 at 7:17 PM, Ian Lance Taylor wrote: >> This patch to the diagnostic code uses the new backtrace library to >> print a backtrace on an ICE. For example, here is the output of a test >> case I took from a C++ PR: >> >

Re: [libbacktrace] Fix bootstrap with gcc 4.4

2012-09-18 Thread Richard Guenther
On Tue, Sep 18, 2012 at 10:32 AM, Rainer Orth wrote: > The libbacktrace integration broke Solaris 10 and 11 bootstrap when > using gcc 4.4 (any version of gcc without __sync_* support actually): Ouch, that's bad. > stage1 config.h has > > /* #undef HAVE_SYNC_FUNCTIONS */ > > and fileline.c and m

Re: [libbacktrace] Fix bootstrap with gcc 4.4

2012-09-18 Thread Richard Guenther
On Tue, Sep 18, 2012 at 10:54 AM, Richard Guenther wrote: > On Tue, Sep 18, 2012 at 10:32 AM, Rainer Orth > wrote: >> The libbacktrace integration broke Solaris 10 and 11 bootstrap when >> using gcc 4.4 (any version of gcc without __sync_* support actually): > > Ouch, that's bad. Btw, why do we

[PATCH, libbacktrace]: Fix compilation on CentOS 5.8

2012-09-18 Thread Uros Bizjak
Hello! CentOS 5.8 uses glibc version 2.5 that needs _GNU_SOURCE defined to use strnlen. 2012-09-18 Uros Bizjak * dwarf.c: Define _GNU_SOURCE. Tested on CentOS x86_64-pc-linux-gnu. OK for mainline? Uros. Index: dwarf.c

Re: [PATCH][RFC] Use overloads for gimple_build_assign_with_ops

2012-09-18 Thread Richard Guenther
On Mon, 17 Sep 2012, Richard Guenther wrote: > On Mon, 17 Sep 2012, Diego Novillo wrote: > > > On 2012-09-17 09:25 , Richard Guenther wrote: > > > > > > This makes use of the new builtin functions for FILE, LINE and FUNCTION > > > to turn gimple_build_assign_with_ops/gimple_build_assign_with_ops

Re: [PATCH, libbacktrace]: Fix compilation on CentOS 5.8

2012-09-18 Thread Richard Guenther
On Tue, Sep 18, 2012 at 11:09 AM, Uros Bizjak wrote: > Hello! > > CentOS 5.8 uses glibc version 2.5 that needs _GNU_SOURCE defined to use > strnlen. Hm, shouldn't libiberty contain a xstrnlen? I bet strnlen isn't available everywhere. Richard. > 2012-09-18 Uros Bizjak > > * dwarf.c

[PATCH, AArch64] Implement ctz and clrsb standard patterns

2012-09-18 Thread Ian Bolton
I've implemented the following standard patterns: * clrsb * ctz Regression runs passed and I have added compilation tests for them, and clz as well. (Execution tests are covered by gcc/testsuite/gcc.c-torture/execute/builtin-bitops-1.c.) OK for aarch64-branch and aarch64-4.7-branch? Cheers, I

Re: [PATCH, libbacktrace]: Fix compilation on CentOS 5.8

2012-09-18 Thread Uros Bizjak
On Tue, Sep 18, 2012 at 11:16 AM, Richard Guenther wrote: >> CentOS 5.8 uses glibc version 2.5 that needs _GNU_SOURCE defined to use >> strnlen. > > Hm, shouldn't libiberty contain a xstrnlen? I bet strnlen isn't available > everywhere. I didn't find it in the sources. OTOH, mmapio.c already

Re: [PATCH, AArch64] Implement ctz and clrsb standard patterns

2012-09-18 Thread Andreas Schwab
"Ian Bolton" writes: > diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md > index 33815ff..5278957 100644 > --- a/gcc/config/aarch64/aarch64.md > +++ b/gcc/config/aarch64/aarch64.md > @@ -153,6 +153,8 @@ > (UNSPEC_CMTST 83) ; Used in aarch64-simd.md. > (UNSPE

Re: [PATCH, libbacktrace]: Fix compilation on CentOS 5.8

2012-09-18 Thread Dominique Dhumieres
> ... I bet strnlen isn't available everywhere. You won!-(it is not available on darwin10) I had to change the strnlen to strlen in order to bootstrap. Dominique

RE: [PATCH, AArch64] Implement ctz and clrsb standard patterns

2012-09-18 Thread Ian Bolton
> > diff --git a/gcc/config/aarch64/aarch64.md > b/gcc/config/aarch64/aarch64.md > > index 33815ff..5278957 100644 > > --- a/gcc/config/aarch64/aarch64.md > > +++ b/gcc/config/aarch64/aarch64.md > > @@ -153,6 +153,8 @@ > > (UNSPEC_CMTST 83) ; Used in aarch64-simd.md. > > (UNSPEC_FMAX

RE: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-18 Thread Bin Cheng
Ping. > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On > Behalf Of Bin.Cheng > Sent: Tuesday, September 04, 2012 11:20 PM > To: Richard Guenther; gcc-patches@gcc.gnu.org > Cc: Richard Earnshaw > Subject: Re: Ping^2: [PATCH]Remove duplicat

Re: [PATCH, libbacktrace]: Fix compilation on CentOS 5.8

2012-09-18 Thread Richard Guenther
On Tue, Sep 18, 2012 at 11:25 AM, Uros Bizjak wrote: > On Tue, Sep 18, 2012 at 11:16 AM, Richard Guenther > wrote: > >>> CentOS 5.8 uses glibc version 2.5 that needs _GNU_SOURCE defined to use >>> strnlen. >> >> Hm, shouldn't libiberty contain a xstrnlen? I bet strnlen isn't available >> everyw

Re: Ping^2: [PATCH]Remove duplicate check on BRANCH_COST in fold-const.c

2012-09-18 Thread Richard Guenther
On Tue, Sep 18, 2012 at 11:32 AM, Bin Cheng wrote: > Ping. I already approved your original patch upthread. Richard. >> -Original Message- >> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] > On >> Behalf Of Bin.Cheng >> Sent: Tuesday, September 04, 2012 11:20

Re: [PATCH] Changes in mode switching

2012-09-18 Thread Vladimir Yakovlev
Hi Ricard, You are right I no need the changes in mode-switchig.c at all. After I remove additional argument from EMIT_MODE_SET and run 'make check' I found no differences with make check result of previous run. So I no need in any changes in the middle end part. Regards, Vladimir P.S. I'll be

Fix instability of -fschedule-insn for x86

2012-09-18 Thread Yuri Rumyantsev
Hi All, This patch aims to fix all stability issues related to using the first scheduler in gcc for x86 target (there several reported issues related to this problem). Main idea of this activity is mostly to provide user a possibility to safely turn on first scheduler for his codes. In some cases

[PATCH] Fix instability of -fschedule-insn for x86

2012-09-18 Thread Yuri Rumyantsev
Hi All, Forgot to attch the patch. 2012/9/18 Yuri Rumyantsev : > Hi All, > > This patch aims to fix all stability issues related to using the first > scheduler in gcc > for x86 target (there several reported issues related to this problem). > > Main idea of this activity is mostly to provide user

[PATCH] Add -Og optimization level - optimize for compile-time/debugging experience

2012-09-18 Thread Richard Guenther
This adds -Og as optimization level targeted at the devel-compile-debug cycle (formerly mostly tied to -O0 due to debug issues with even -O1). Discussion on g...@gcc.gnu.org at least shows interest in this, so this is a formal patch submission with a request for comments on the implementation (no

Re: [PATCH] Changes in mode switching

2012-09-18 Thread Uros Bizjak
Hello! > You are right I no need the changes in mode-switchig.c at all. After I > remove additional argument from EMIT_MODE_SET and run 'make check' I > found no differences with make check result of previous run. So I no > need in any changes in the middle end part. Vladimir, can you please inv

Re: [PATCH] Fix instability of -fschedule-insn for x86

2012-09-18 Thread Uros Bizjak
Hello! > This patch aims to fix all stability issues related to using the first > scheduler in gcc > for x86 target (there several reported issues related to this problem). > > Main idea of this activity is mostly to provide user a possibility to > safely turn on first scheduler for his codes. In

Re: [PATCH] Add -Og optimization level - optimize for compile-time/debugging experience

2012-09-18 Thread H.J. Lu
On Tue, Sep 18, 2012 at 4:23 AM, Richard Guenther wrote: > > This adds -Og as optimization level targeted at the devel-compile-debug > cycle (formerly mostly tied to -O0 due to debug issues with even -O1). > > Discussion on g...@gcc.gnu.org at least shows interest in this, so this > is a formal pa

[PATCH] Fix i386 costs (was: i386: Fix logic error in r188785, PR target/54592)

2012-09-18 Thread Jakub Jelinek
On Wed, Jun 27, 2012 at 02:36:14PM -0700, Richard Henderson wrote: > As noticed by Igor Zamyatin. Committed. > PR target/53749 > * config/i386/i386.c (ix86_rtx_costs): Fix typo vs UNITS_PER_WORD > in 2012-06-23 change. Adjust two other DImode tests as well. This change broke cos

Re: [PATCH] Changes in mode switching

2012-09-18 Thread Vladimir Yakovlev
I tried to perform vzeroupper emitting after reload as additional pass of mode switching. I sow one problem that I don't know haw to overcome. After 'pro_and_epilogue', there can be no flow edge to exit block and pre_exit block is not created in this case (see rotine create_pre_exit). Without that

[PATCH] Fix vector permutation forwprop optimization (PR tree-optimization/54610)

2012-09-18 Thread Jakub Jelinek
Hi! vect_gen_perm_mask is not suitable for use outside of the vectorizer, it uses current vector size to determine the number of units of a vector, which isn't something that should be used outside of the vectorizer. The following patch just does construct the mask inline, it is not that long cod

Re: [PATCH] Add -Og optimization level - optimize for compile-time/debugging experience

2012-09-18 Thread Richard Guenther
On Tue, 18 Sep 2012, H.J. Lu wrote: > On Tue, Sep 18, 2012 at 4:23 AM, Richard Guenther wrote: > > > > This adds -Og as optimization level targeted at the devel-compile-debug > > cycle (formerly mostly tied to -O0 due to debug issues with even -O1). > > > > Discussion on g...@gcc.gnu.org at least

Re: [PATCH] Fix vector permutation forwprop optimization (PR tree-optimization/54610)

2012-09-18 Thread Richard Guenther
On Tue, Sep 18, 2012 at 2:20 PM, Jakub Jelinek wrote: > Hi! > > vect_gen_perm_mask is not suitable for use outside of the vectorizer, > it uses current vector size to determine the number of units of a vector, > which isn't something that should be used outside of the vectorizer. > > The following

Re: [PATCH] Add -Og optimization level - optimize for compile-time/debugging experience

2012-09-18 Thread Jakub Jelinek
On Tue, Sep 18, 2012 at 02:38:27PM +0200, Richard Guenther wrote: > > Glibc must be compiled with optimization. Will -Og build glibc? > > Apply the patch and check for yourself. I suppose the answer will > be yes (whatever "optimization" requirements glibc has). glibc headers check __OPTIMIZE__

Re: [PATCH] Changes in mode switching

2012-09-18 Thread Uros Bizjak
On Tue, Sep 18, 2012 at 2:18 PM, Vladimir Yakovlev wrote: > I tried to perform vzeroupper emitting after reload as additional pass > of mode switching. > I sow one problem that I don't know haw to overcome. After > 'pro_and_epilogue', there can be no > flow edge to exit block and pre_exit block is

RE: [PATCH, AArch64] Implement ctz and clrsb standard patterns

2012-09-18 Thread Ian Bolton
> > > diff --git a/gcc/config/aarch64/aarch64.md > > b/gcc/config/aarch64/aarch64.md > > > index 33815ff..5278957 100644 > > > --- a/gcc/config/aarch64/aarch64.md > > > +++ b/gcc/config/aarch64/aarch64.md > > > @@ -153,6 +153,8 @@ > > > (UNSPEC_CMTST 83) ; Used in aarch64-simd.md. > >

RE: [PATCH, AArch64] Implement ctz and clrsb standard patterns

2012-09-18 Thread Ian Bolton
New version attached with better formatted test cases. OK for aarch64-branch and aarch64-4.7-branch? Cheers, Ian - 2012-09-18 Ian Bolton gcc/ * config/aarch64/aarch64.h: Define CTZ_DEFINED_VALUE_AT_ZERO. * config/aarch

Rewrite lto-symtab to work on symbol table

2012-09-18 Thread Jan Hubicka
Hi, this patch reorganize lto-symtab to work across symtab's symbol table instead of building its own. This simplifies things a bit and with the previous changes it is rather straighforward - i.e. replace all uses of lto_symtab_entry_t by symtab_node. There are few differences in between the symt

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 1:49 AM, Richard Guenther wrote: > On Mon, Sep 17, 2012 at 7:17 PM, Ian Lance Taylor wrote: >> This patch to the diagnostic code uses the new backtrace library to >> print a backtrace on an ICE. For example, here is the output of a test >> case I took from a C++ PR: >> >>

Re: [PATCH, libbacktrace]: Fix compilation on CentOS 5.8

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 2:25 AM, Uros Bizjak wrote: > On Tue, Sep 18, 2012 at 11:16 AM, Richard Guenther > wrote: > >>> CentOS 5.8 uses glibc version 2.5 that needs _GNU_SOURCE defined to use >>> strnlen. >> >> Hm, shouldn't libiberty contain a xstrnlen? I bet strnlen isn't available >> everywh

Re: [PATCH 1-2/12 ] New configure option --enable-espf=(all|ssp|pie|no)

2012-09-18 Thread Magnus Granberg
tisdag 11 september 2012 01.33.42 skrev Magnus Granberg: > fredag 07 september 2012 18.52.11 skrev du: > > On Fri, 7 Sep 2012, Magnus Granberg wrote: > > > * Makefile.in Add > > > -fno-stack-protector when > > > > > > needed for espf. > >

[AARCH64][PATCH] Remove hardwired multiarch.

2012-09-18 Thread Marcus Shawcroft
I've just committed this patch to aarch64-branch to remove the multi-arch fudge. /Marcus 2012-09-18 Marcus Shawcroft * config/aarch64/aarch64-linux.h (MULTIARCH_TUPLE): Remove. (STANDARD_STARTFILE_PREFIX_1): Likewise. (STANDARD_STARTFILE_PREFIX_2): Likewise.diff --git

[v3] libstdc++/54612

2012-09-18 Thread Paolo Carlini
Hi, tested x86_64-linux (with and without #include "opt_random.h" at the end of ext/random), committed to mainline. Should be fixed now. Thanks, Paolo. PS: I just noticed that in ext/random, inside namespace __gnu_cxx, we are using, unqualified, size_t and other types. We shouldn't: it's onl

Re: [PATCH, libbacktrace]: Fix compilation on CentOS 5.8

2012-09-18 Thread Andreas Schwab
Ian Lance Taylor writes: > mmapio.c uses _GNU_SOURCE so that getpagesize is available on GNU/Linux > systems. That should be available by default (which includes BSD things) unless the namespace has been restricted in some other way. There's of course also the POSIX way of using sysconf(_SC_PA

[AARCH64-4.7][PATCH] Remove hardwired multiarch.

2012-09-18 Thread Marcus Shawcroft
I've just back ported this into ARM/AARCH64-4.7. /Marcus 2012-09-18 Marcus Shawcroft * config/aarch64/aarch64-linux.h (MULTIARCH_TUPLE): Remove. (STANDARD_STARTFILE_PREFIX_1): Likewise. (STANDARD_STARTFILE_PREFIX_2): Likewise.diff --git a/gcc/config/aarch64/aarch64-lin

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Markus Trippelsdorf
On 2012.09.18 at 06:58 -0700, Ian Lance Taylor wrote: > On Tue, Sep 18, 2012 at 1:49 AM, Richard Guenther > wrote: > > On Mon, Sep 17, 2012 at 7:17 PM, Ian Lance Taylor wrote: > >> OK for mainline? > > > > Hm. Can you please be that verbose only for ENABLE_CHECKING compilers? > > That would be

Re: Rewrite lto-symtab to work on symbol table

2012-09-18 Thread Richard Guenther
On Tue, 18 Sep 2012, Jan Hubicka wrote: > Hi, > this patch reorganize lto-symtab to work across symtab's symbol table instead > of building its own. This simplifies things a bit and with the previous > changes it is rather straighforward - i.e. replace all uses of > lto_symtab_entry_t by symtab_n

Re: [PATCHv3] rs6000: Add 2 built-ins to read the Time Base Register on PowerPC

2012-09-18 Thread Segher Boessenkool
Hi Tulio, Thanks for all the cleanups! Two quite minor things... +(define_insn "rs6000_get_timebase_ppc64" + [(set (match_operand:DI 0 "gpc_reg_operand" "=r") +(unspec_volatile:DI [(const_int 0)] UNSPECV_GETTB))] + "TARGET_POWERPC64" +{ + if (TARGET_MFCRF) +return "mfspr %0, 268

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Richard Guenther
On Tue, Sep 18, 2012 at 4:40 PM, Markus Trippelsdorf wrote: > On 2012.09.18 at 06:58 -0700, Ian Lance Taylor wrote: >> On Tue, Sep 18, 2012 at 1:49 AM, Richard Guenther >> wrote: >> > On Mon, Sep 17, 2012 at 7:17 PM, Ian Lance Taylor wrote: >> >> OK for mainline? >> > >> > Hm. Can you please be

Re: [PATCH] Fix i386 costs

2012-09-18 Thread Richard Henderson
On 09/18/2012 05:10 AM, Jakub Jelinek wrote: > 2012-09-18 Jakub Jelinek > > PR target/54592 > * config/i386/i386.c (ix86_rtx_costs): Limit > UNITS_PER_WORD > AND/IOR/XOR cost calculation to MODE_INT class modes. > > * gcc.target/i386/pr54592.c: New test. Ok. r~

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Markus Trippelsdorf
On 2012.09.18 at 16:55 +0200, Richard Guenther wrote: > On Tue, Sep 18, 2012 at 4:40 PM, Markus Trippelsdorf > wrote: > > On 2012.09.18 at 06:58 -0700, Ian Lance Taylor wrote: > >> On Tue, Sep 18, 2012 at 1:49 AM, Richard Guenther > >> wrote: > >> > On Mon, Sep 17, 2012 at 7:17 PM, Ian Lance Tayl

Re: [PATCH] Add option for dumping to stderr (issue6190057)

2012-09-18 Thread Xinliang David Li
On Tue, Sep 18, 2012 at 1:48 AM, Sharad Singhai wrote: > In response to the recent comments, I have updated the patch to do the > following: > > - Remove pass handling from -fopt-info > - Support additional flags in regular dumps > > I have massaged the options so that they have the following (hop

Re: [PATCH] PowerPC VLE port

2012-09-18 Thread Maciej W. Rozycki
On Tue, 11 Sep 2012, David Edelsohn wrote: > > 2012-09-10 Maciej W. Rozycki > > > > gcc/ > > * config/rs6000/rs6000.c (print_operand) <'c'>: Remove. > > * config/rs6000/spe.md: Remove a leftover comment. > > Okay. I have applied this change now, thanks for your review

libiberty patch committed: Add strnlen

2012-09-18 Thread Ian Lance Taylor
This patch to libiberty adds support for strnlen if it is not already present. I rebuilt the Makefile dependencies. This revealed that maint-tool wasn't recognizing that files that include dwarf2.h also depend on dwarf2.def, so I fixed maint-tool. I also rebuilt functions.texi, which had not bee

libbacktrace patch committed: Declare strnlen if not declared

2012-09-18 Thread Ian Lance Taylor
This patch to libbacktrace declares strnlen if it is not declared in a standard header file. If necessary, the actual definition will come from libiberty. Bootstrapped on x86_64-unknown-linux-gnu. Committed to mainline. Ian 2012-09-18 Ian Lance Taylor * configure.ac: Check whether

Re: [PATCH, libbacktrace]: Fix compilation on CentOS 5.8

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 7:08 AM, Ian Lance Taylor wrote: > > I'll fix the strnlen issue some other way. I committed a pair of patches, to libiberty and libbacktrace, that should fix the problem. Let me know if it is still present. Sorry for the difficulty. Ian

Re: [PATCH, libbacktrace]: Fix compilation on CentOS 5.8

2012-09-18 Thread Joseph S. Myers
On Tue, 18 Sep 2012, Uros Bizjak wrote: > Index: dwarf.c > === > --- dwarf.c (revision 191413) > +++ dwarf.c (working copy) > @@ -30,6 +30,8 @@ > IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE > POSSIBILI

Re: [libbacktrace] Fix bootstrap with gcc 4.4

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 1:32 AM, Rainer Orth wrote: > The libbacktrace integration broke Solaris 10 and 11 bootstrap when > using gcc 4.4 (any version of gcc without __sync_* support actually): The patch is fine and should fix the problem, but GCC 4.4 does have __sync_* support. Might be worth l

Re: [libbacktrace] Fix bootstrap with gcc 4.4

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 1:55 AM, Richard Guenther wrote: > On Tue, Sep 18, 2012 at 10:54 AM, Richard Guenther > wrote: >> On Tue, Sep 18, 2012 at 10:32 AM, Rainer Orth >> wrote: >>> The libbacktrace integration broke Solaris 10 and 11 bootstrap when >>> using gcc 4.4 (any version of gcc without

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 7:40 AM, Markus Trippelsdorf wrote: > > If you use "make install-strip" to install, then libbacktrace will have > been build in vain. At least for this case a way to disable libbacktrace > should be available. Why bother? The library will fail to find anything useful to p

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 7:55 AM, Richard Guenther wrote: > > Indeed - we ship binaries with stripped debug info, usually not installed. > libbacktrace will only produce useless garbage then. So I want a way > to disable it (at least by default) at configure time. The library won't print useless

Re: Implement Nakagami distribution as an extension.

2012-09-18 Thread Ed Smith-Rowland
On 09/18/2012 10:28 AM, Paolo Carlini wrote: On 09/18/2012 03:23 AM, Ed Smith-Rowland wrote: Here is another tweak for the Nakagami distribution. operator() is a one liner without a local variable. The template friend is fixed and the relevant test has been added. Changed dates. If this ve

Re: [PATCH] Add option for dumping to stderr (issue6190057)

2012-09-18 Thread Sharad Singhai
On Sep 18, 2012 8:43 AM, "Xinliang David Li" wrote: > > On Tue, Sep 18, 2012 at 1:48 AM, Sharad Singhai wrote: > > In response to the recent comments, I have updated the patch to do the > > following: > > > > - Remove pass handling from -fopt-info > > - Support additional flags in regular dumps >

Re: [PATCH, AArch64] Implement ctz and clrsb standard patterns

2012-09-18 Thread Richard Earnshaw
On 18/09/12 14:24, Ian Bolton wrote: > New version attached with better formatted test cases. > > OK for aarch64-branch and aarch64-4.7-branch? > > Cheers, > Ian > > > - > > 2012-09-18 Ian Bolton > > gcc/ > * config/aarch64/aarc

Re: User directed Function Multiversioning via Function Overloading (issue5752064)

2012-09-18 Thread Sriraman Tallam
Ping. On Fri, Aug 24, 2012 at 5:34 PM, Sriraman Tallam wrote: > Hi Jason, > >I have created a new patch to use target hooks for all the > functionality and make the front-end just call the target hooks at the > appropriate places. This is more like what you suggested in a previous > mail. In

Re: libiberty patch committed: Add strnlen

2012-09-18 Thread Jakub Jelinek
On Tue, Sep 18, 2012 at 09:03:03AM -0700, Ian Lance Taylor wrote: > --- strnlen.c (revision 0) > +++ strnlen.c (revision 0) > @@ -0,0 +1,28 @@ > +/* Portable version of strnlen. > + This function is in the public domain. */ > + > +/* > + > +@deftypefn Supplemental size_t strnlen (const char *@va

Re: [PATCH, libbacktrace]: Fix compilation on CentOS 5.8

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 9:08 AM, Joseph S. Myers wrote: > > I think AC_USE_SYSTEM_EXTENSIONS is preferred to defining _GNU_SOURCE in > individual source files. Thanks for the pointer. I have committed this patch after bootstrap and libbacktrace test on x86_64-unknown-linux-gnu. Ian 2012-09-18

Re: libiberty patch committed: Add strnlen

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 9:25 AM, Jakub Jelinek wrote: > On Tue, Sep 18, 2012 at 09:03:03AM -0700, Ian Lance Taylor wrote: >> +*/ >> + >> +#include "config.h" > > Shouldn't this #include for size_t, or is config.h providing > size_t? Or #include ? From what I can see, config.h doesn't > always

libbacktrace patch committed: Fix test of HAVE_DECL_STRNLEN

2012-09-18 Thread Ian Lance Taylor
I foolishly assumed that the autoconf macro AC_CHECK_DECLS worked like most autoconf macros, and did not define HAVE_DECL_xx when the declaration is not available. However, it turns out that it actually #defines it to 0. This patch fixes the test of HAVE_DECL_STRNLEN to match that behaviour. Boo

Re: [PATCH] PR 53528 c++/ C++11 Generalized Attribute support

2012-09-18 Thread Jason Merrill
On 09/18/2012 09:51 AM, Dodji Seketeli wrote: + VEC_safe_push (scoped_attributes, heap, attributes_table, sa); + result = &VEC_last (scoped_attributes, attributes_table); Here you can set result from the return value of VEC_safe_push. + if ((flags & ATTR_FLAG_CXX11) + &

[PATCH] rs6000: Remove integer abs/nabs/min/max patterns

2012-09-18 Thread Segher Boessenkool
Without these patterns, the exact same code is generated. This is a leftover from when we still had a "doz" instruction. Tested on powerpc64-linux --enable-languages=c,c++,fortran; no regressions. Also tested all these patterns manually, -m32 and -m64, -misel and -mno-isel. Okay to apply? 201

[PATCH, ARM] 64-bit shifts in NEON

2012-09-18 Thread Ulrich Weigand
Hello, a while ago Andrew Stubbs posted a patch to use NEON registers and instructions to perform 64-bit integer shifts: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01645.html As Andrew no longer works on ARM, I've now picked this up and reworked it a bit: - Updated for current mainline changes

libbacktrace patch committed: Mark test functions as unused

2012-09-18 Thread Ian Lance Taylor
This patch to libbacktrace marks the test functions with the unused attribute. This avoids producing a compilation warning when building the test on a system that does not support libbacktrace. Bootstrapped and ran libbacktrace testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian

PING Re: [PATCH, MIPS] add new peephole for 74k dspr2

2012-09-18 Thread Sandra Loosemore
On 08/27/2012 10:36 AM, Richard Sandiford wrote: Sandra Loosemore writes: On 08/19/2012 11:22 AM, Richard Sandiford wrote: Not sure whether a peephole is the right choice here. In practice, I'd imagine these opportunities would only come from a DImode move of $0 into a doubleword register, s

[Patch,wwwdocs,4.7,committed]: Document --with-avrlibc

2012-09-18 Thread Georg-Johann Lay
Added the new avr-gcc configure option --with-avrlibc to the 4.7 release notes. Johann Index: gcc-4.7/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.7/changes.html,v retrieving revision 1.125 diff -u -p -r1.125 changes.html

Re: [PATCH] Add -Og optimization level - optimize for compile-time/debugging experience

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 4:23 AM, Richard Guenther wrote: > > 2012-09-18 Richard Guenther > > PR other/53316 > * common.opt (optimize_debug): New variable. > (Og): New optimization level. > * doc/invoke.texi (Og): Document. > * opts.c (maybe_default_option

Re: [PATCH] Add -Og optimization level - optimize for compile-time/debugging experience

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 4:23 AM, Richard Guenther wrote: > > This adds -Og as optimization level targeted at the devel-compile-debug > cycle (formerly mostly tied to -O0 due to debug issues with even -O1). This needs an entry in gcc-4.8/changes.html, of course. Ian

Re: [C++ Patch] for c++/54537

2012-09-18 Thread Fabien Chêne
2012/9/11 Fabien Chêne : > Oops, not sure how I test that change initially, or I must be blind, > because it triggers an error in tr1/cmath about pow. I'll see what I > can do... Well, as summarized in the code below, the problem seems to be the redundant overload of std::tr1::pow(double,double).

Re: [C++ Patch] for c++/54537

2012-09-18 Thread Fabien Chêne
... And the patch. 2012/9/18 Fabien Chêne : > 2012/9/11 Fabien Chêne : >> Oops, not sure how I test that change initially, or I must be blind, >> because it triggers an error in tr1/cmath about pow. I'll see what I >> can do... > > Well, as summarized in the code below, the problem seems to be the

[PATCH, middle-end]: Fix g++.dg/other/vector-compare.C testsuite failure on alpha

2012-09-18 Thread Uros Bizjak
Hello! g++.dg/other/vector-compare.C recently started to fail on alphaev68-pc-linux-gnu with: [uros@localhost other]$ ~/gcc-build-alpha/gcc/cc1plus -std=gnu++11 -quiet vector-compare.C vector-compare.C: In function ‘int main()’: vector-compare.C:26:5: internal compiler error: in emit_cmp_and_jump

Re: [PATCH] Add extra location information - PR43486

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 3:58 AM, Arnaud Charlet wrote: > > Since this issue is more general, I have split my changes and introduced a new > tentative switch called -fextra-slocs, which is the subject of this email. Sorry for picking on simple stuff, but the switch name seems meaningless, and ther

Re: [PATCH] Add extra location information - PR43486

2012-09-18 Thread Arnaud Charlet
> Sorry for picking on simple stuff, but the switch name seems No problem, and thanks for your feedback. > meaningless, and there isn't any documentation. Ah. I'm open for suggestion on a better name, or I can come up with a new one. I'll indeed add documentation as soon as there's some kind of

Re: [C++ Patch] for c++/54537

2012-09-18 Thread Paolo Carlini
Hi, "Fabien Chêne" ha scritto: >2012/9/11 Fabien Chêne : >> Oops, not sure how I test that change initially, or I must be blind, >> because it triggers an error in tr1/cmath about pow. I'll see what I >> can do... > >Well, as summarized in the code below, the problem seems to be the >redundant o

libbacktrace patch committed: Add some mingw support

2012-09-18 Thread Ian Lance Taylor
This patch to libbacktrace adds some support for mingw. The executable is opened with O_BINARY. The fcntl function is not called. Bootstrapped and ran libbacktrace testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian 2012-09-18 Ian Lance Taylor * posix.c (O_BINARY): D

Re: Use conditional casting with symtab_node

2012-09-18 Thread Eric Botcazou
> When, the property test is embedded within a larger condition, a little > restructuring is required to pull out the secondary conditions. For > example, > > if (symtab_variable_p (node) > && varpool (node)->finalized) > varpool_analyze_node (varpool (node)); > > becomes > > if (

Re: [PATCH] Add extra location information - PR43486

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 10:58 AM, Arnaud Charlet wrote: > >> and at that point you have lost your extra location >> information. > > Actually no, see the c-family/c-common.c patch, copied here, which > ensures that folding does preserve such information: Thanks. I think I would like some clarity

Re: [C++ Patch] for c++/54537

2012-09-18 Thread Fabien Chêne
2012/9/18 Paolo Carlini : > I don't understand: what's wrong - exactly - with the std::tr1::pow (double, > double) overload above? Now I can't immediately check, but do the EDG and > CLANG front ends accept it or not? They don't. The problem is that it conflicts with ::pow(double,double). // i

Re: PING Re: [PATCH, MIPS] add new peephole for 74k dspr2

2012-09-18 Thread Richard Sandiford
Sandra Loosemore writes: > On 08/27/2012 10:36 AM, Richard Sandiford wrote: >> Sandra Loosemore writes: >>> On 08/19/2012 11:22 AM, Richard Sandiford wrote: Not sure whether a peephole is the right choice here. In practice, I'd imagine these opportunities would only come from a DI

Re: [C++ Patch] for c++/54537

2012-09-18 Thread Paolo Carlini
Hi, "Fabien Chêne" ha scritto: >2012/9/18 Paolo Carlini : > >> I don't understand: what's wrong - exactly - with the std::tr1::pow >(double, double) overload above? Now I can't immediately check, but do >the EDG and CLANG front ends accept it or not? > >They don't. The problem is that it conflic

Re: Use conditional casting with symtab_node

2012-09-18 Thread Lawrence Crowl
On 9/18/12, Eric Botcazou wrote: >> When, the property test is embedded within a larger condition, a little >> restructuring is required to pull out the secondary conditions. For >> example, >> >> if (symtab_variable_p (node) >> && varpool (node)->finalized) >> varpool_analyze_node (v

Re: [PATCH] OpenBSD/hppa support

2012-09-18 Thread John David Anglin
On Thu, 06 Sep 2012, Mark Kettenis wrote: > Most bits are stolen from Linux, but there are a few subtle > differences since our assembler is configured to be slightly more > HP-UX-ish. > > > libgcc/: > > 2012-09-06 Mark Kettenis > > * config.host (hppa-*-openbsd*): New target. >

[patch, fortran] Fix an issue found by Coverity scan

2012-09-18 Thread Thomas Koenig
Hello world, pretty self-explanatory. No test case because there is no change in behavior. OK for trunk? Thomas 2012-09-18 Thomas König PR fortran/54599 * dependency.c (gfc_dep_compare_expr): Clarify logic, remove dead code. Index: dependency.c ==

[SH] Use more braced strings in MD

2012-09-18 Thread Oleg Endo
Hello, Like the topic says. No functional change, just cosmetics. Tested on rev 191342 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" and no new failures. OK to install? Cheers, Oleg ChangeLog: * config/sh/sh.md (

[SH] PR 54236 - Add another addc case

2012-09-18 Thread Oleg Endo
Hello, There is another opportunity where SH's addc insn can be used. Tested on rev 191342 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" and no new failures. OK to install? Cheers, Oleg gcc/ChangeLog: PR target/54

[SH] PR 54089 - Add another rotcr case

2012-09-18 Thread Oleg Endo
Hello, There is another opportunity where SH's rotcr insn can be used. Tested on rev 191342 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" and no new failures. OK to install? Cheers, Oleg gcc/ChangeLog: PR target/5

Re: [PATCH] Add extra location information - PR43486

2012-09-18 Thread Arnaud Charlet
> >> and at that point you have lost your extra location > >> information. > > > > Actually no, see the c-family/c-common.c patch, copied here, which > > ensures that folding does preserve such information: > > Thanks. I think I would like some clarity on when the extra location > information is

testsuite] remove dg-do run from a vect test

2012-09-18 Thread Janis Johnson
The infrastructure for gcc.dg/vect tests determines whether the default is for tests to be compile-only or compile plus execute. Tests that should not be executed use { dg-do compile }, but no test should use { dg-do run }. This patch removes { dg-do run} from pr52298.c. Tested on arm-none-eabi

[testsuite] vect effective targets should use arm_neon_ok

2012-09-18 Thread Janis Johnson
In most cases a test that requires ARM NEON should use effective target arm_neon, which means that flags run for all tests include NEON support. The result is cached the first time it is checked for a multilib. Vectorization tests, when run for ARM, add flags to support NEON if it's OK to do so, b

[testsuite] for vect_multiple_sizes, skip instead of xfail for some checks

2012-09-18 Thread Janis Johnson
Seventeen tests in gcc.dg/vect that use vect_multiple_sizes have checks similar to: /* { dg-final { scan-tree-dump-times "can't determine dependence" 2 "vect" { xfail vect_multiple_sizes } } } */ /* { dg-final { scan-tree-dump-times "can't determine dependence" 4 "vect" { target vect_multiple_si

[testsuite] vect/fast-math-pr35982: skip check instead of xfail

2012-09-18 Thread Janis Johnson
Test gcc.dg/vect/fast-math-pr35982.c uses xfail in a dg-final check when it should instead skip the check for that effective target. Tested on arm-none-eabi for a variety of test multilibs, checked in on trunk as obvious. I'll backport to 4.6 when the branch is open. Janis 2012-09-18 Janis John

  1   2   >