[PATCH, rtl-optimization] Fix PR63475, Postreload CSE propagates aliased memory operand

2014-10-14 Thread Uros Bizjak
Hello! Attached patch fixes PR63475, where postreload CSE propagates aliased memory operand. The core of the problem was with the call to base_alias_check when VALUE RTXes are involved. Before the call, find_base_term is used to extract the base of x_addr and mem_addr. Please note that find_base_

Re: [PATCH i386 AVX512] [56/n] Add plus/minus/abs/neg/andnot insn patterns.

2014-10-14 Thread Kirill Yukhin
Hello Uroš, It seems like I missed to post uppdated patch. On 25 Sep 20:11, Uros Bizjak wrote: > I'd rather go with the second approach, it is less confusing from the > maintainer POV. All other patterns with masking use some consistent > template, so I'd suggest using the same approach for everyth

Move loop peeling from RTL to gimple

2014-10-14 Thread Jan Hubicka
Hi, this is update of my 2013 update to 2012 patch to move rtl loop peeling to tree level. This is to expose optimization oppurtunities earlier. Incrementally I think I can also improve profiling to provide a histogram on loop iterations and get more sensible peeling decisions. profiled-bootstrapp

Re: Towards GNU11

2014-10-14 Thread Marek Polacek
On Tue, Oct 07, 2014 at 11:07:56PM +0200, Marek Polacek wrote: > I'd like to kick off a discussion about moving the default standard > for C from gnu89 to gnu11. The consensus seems to be to go forward with this change. I will commit the patch in 24 hours unless I hear objections. Thanks,

Re: [PATCH 6/n] OpenMP 4.0 offloading infrastructure: option handling

2014-10-14 Thread Richard Biener
On Mon, 13 Oct 2014, Bernd Schmidt wrote: > On 10/13/2014 12:33 PM, Ilya Verbin wrote: > > On 13 Oct 12:19, Jakub Jelinek wrote: > > > But I'd like to understand why is this one needed. > > > Why should the compilers care? Aggregates layout and alignment of > > > integral/floating types must matc

[PATCH] Fix PR63512

2014-10-14 Thread Richard Biener
I forgot to mark stmts as modified. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2014-10-14 Richard Biener PR tree-optimization/63512 * tree-ssa-pre.c (create_expression_by_pieces): Mark stmts modified. * g++.dg/torture/pr63512.C: N

Re: Move loop peeling from RTL to gimple

2014-10-14 Thread Richard Biener
On Tue, 14 Oct 2014, Jan Hubicka wrote: > Hi, > this is update of my 2013 update to 2012 patch to move rtl loop peeling > to tree level. This is to expose optimization oppurtunities earlier. > Incrementally I think I can also improve profiling to provide a histogram > on loop iterations and get mo

[PATCH][match-and-simplify] Change back default behavior of fold_stmt

2014-10-14 Thread Richard Biener
This changes default behavior of fold_stmt back to _not_ following SSA use-def chains when trying to simplify things. I had to force that already for one caller and for the merge to trunk I'd rather not track down issues in every other existing caller. This means that fold_stmt will not become m

[v3] Rename a few testcases

2014-10-14 Thread Paolo Carlini
Hi, I'm renaming a few testcases which actually are about alias declarations not typedefs. Thanks, Paolo. 2014-10-14 Paolo Carlini * testsuite/20_util/add_lvalue_reference/requirements/typedefs.cc: Rename to alias_decl.cc. * testsuite/20_util/add_r

Re: [PATCH 3/5] timevar.h: Add an auto_timevar class

2014-10-14 Thread Richard Biener
On Mon, Oct 13, 2014 at 7:45 PM, David Malcolm wrote: > This is used in a couple of places in jit/jit-playback.c to ensure > that we pop the timevar on every exit path from a function. > > I could rewrite them if need be, but it does simplify things. Sorry to be bikeshedding but auto_timevar soun

Re: [PATCH 1/2] Revert PR49721's patch

2014-10-14 Thread Richard Biener
On Tue, Oct 14, 2014 at 12:35 AM, Andrew Pinski wrote: > On Fri, Aug 8, 2014 at 8:51 PM, Andrew Pinski wrote: >> OK? When the second patch is approved? > > Ping? Ok if the second patch was approved. Richard. >> >> Thanks, >> Andrew Pinski >> >> ChangeLog: >> Revert: >> 2011-08-

Re: [PATCH 4/n] OpenMP 4.0 offloading infrastructure: lto-wrapper

2014-10-14 Thread Jakub Jelinek
On Tue, Oct 14, 2014 at 02:42:47AM +0400, Ilya Verbin wrote: > > For that I guess > > lhd_begin_section > > would need to replace: > > section = get_section (name, SECTION_DEBUG, NULL); > > with: > > section = get_section (name, SECTION_DEBUG | SECTION_EXCLUDE, NULL); > > either just for the .g

[PING] [PATCH] Fix PR ipa/61190, 2nd edition‏

2014-10-14 Thread Bernd Edlinger
Ping... see: https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00536.html > Hi Honza, > > > as you know, we have a wrong code bug, when a pure or const method is called > via a virtual thunk. > I had some more Ideas, how to fix that, but all of them had some serious > draw-backs, so I leave the det

[PATCH][match-and-simplify] More TLC to genmatch

2014-10-14 Thread Richard Biener
This applies more comment / whitespace TLC to genmatch and does minor refactoring on-the-fly. Bootstrap running on x86_64-unknown-linux-gnu. Richard. 2014-10-14 Richard Biener * genmatch.c: Whitespace and comment fixes, some minor refactoring. Index: gcc/genmatch.c

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [4/n] Memory accesses instrumentation

2014-10-14 Thread Ilya Enkovich
On 13 Oct 14:52, Jeff Law wrote: > On 10/08/14 13:01, Ilya Enkovich wrote: > >Hi, > > > >This is the main chunk of instrumentation codes. This patch introduces > >instrumentation pass which instruments memory accesses. > > > >Thanks, > >Ilya > >-- > >2014-10-08 Ilya Enkovich > > > > * tree-c

[PATCH][match-and-simplify] Update texi documentation

2014-10-14 Thread Richard Biener
This updates it with changed/added features. pfd-build checked and inspected, applied. Richard. 2014-10-14 Richard Biener * doc/match-and-simplify.texi: Update. Index: gcc/doc/match-and-simplify.texi === --- gcc/doc/ma

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [6/n] Instrument calls and returns

2014-10-14 Thread Ilya Enkovich
On 13 Oct 14:49, Ilya Enkovich wrote: > On 10 Oct 12:50, Jeff Law wrote: > > On 10/08/14 13:04, Ilya Enkovich wrote: > > >Hi, > > > > > >This patch adds intrumentation of calls and returns into instrumentation > > >pass. > > > > > >Thanks, > > >Ilya > > >-- > > >2014-10-08 Ilya Enkovich > > > >

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [3/n] Helper functions

2014-10-14 Thread Ilya Enkovich
On 14 Oct 01:13, Ilya Enkovich wrote: > 2014-10-14 1:05 GMT+04:00 Jeff Law : > > > > Where does chkp_can_be_shared get used?Normally the thing to do would > > just be to call unshare_expr. It'll create copies as needed. If it's > > something that is supposed to be shared then it'll leave it

Re: [PATCH 6/n] OpenMP 4.0 offloading infrastructure: option handling

2014-10-14 Thread Bernd Schmidt
On 10/14/2014 09:25 AM, Richard Biener wrote: On Mon, 13 Oct 2014, Bernd Schmidt wrote: On 10/13/2014 12:33 PM, Ilya Verbin wrote: On 13 Oct 12:19, Jakub Jelinek wrote: But I'd like to understand why is this one needed. Why should the compilers care? Aggregates layout and alignment of integr

[PATCH][match-and-simplify] Fix ICE

2014-10-14 Thread Richard Biener
This fixes an ICE that occurs when valueziation returns NULL and we are looking at single-rhs REALPART_EXPR. We should check for is_gimple_min_invariant before valueization. Bootstrapped on x86_64-unknown-linux-gnu, applied. Richard. 2014-10-14 Richard Biener * genmatch.c (dt_opera

Re: [PATCH 1/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-14 Thread Jakub Jelinek
On Fri, Oct 10, 2014 at 10:03:38AM -0600, Jeff Law wrote: > Can you add a PR markers to your changelog > > PR target/8340 > PR middle-end/47602 > PR rtl-optimization/55458 > > Actually I think there is an additional test in 47602. Can you please add > it to the suite? You'll a

Re: [PATCH][match-and-simplify] Change back default behavior of fold_stmt

2014-10-14 Thread Richard Biener
On Tue, 14 Oct 2014, Richard Biener wrote: > > This changes default behavior of fold_stmt back to _not_ following > SSA use-def chains when trying to simplify things. I had to force > that already for one caller and for the merge to trunk I'd rather > not track down issues in every other existin

[PATCH, DWARF] re-init dw_frame_pointer_regnum between functions

2014-10-14 Thread Christian Bruel
Hello, ARM and Thumb modes use different hard_frame_pointer_regnum ABIs. The problem is that dwarf2cfi.c:dw_frame_pointer_regnum cache is initialized only once per file, when creating the CIE. While testing the ARM attribute target to switch modes between functions, I got a few assertion with -g,

[patch libstdc++]: Fix PR/59807

2014-10-14 Thread Kai Tietz
Hi, this patch fixes issue PR/59807 "mutex misses destructor if non-function call initialization is used". This issue just got reported for mingw-w64, as this is the only venture providing posix-threading enabled toolchains (C++11). Nevertheless this issue could happen for other native Windows t

Re: [PATCH][match-and-simplify] Change back default behavior of fold_stmt

2014-10-14 Thread Richard Biener
On Tue, 14 Oct 2014, Richard Biener wrote: > On Tue, 14 Oct 2014, Richard Biener wrote: > > > > > This changes default behavior of fold_stmt back to _not_ following > > SSA use-def chains when trying to simplify things. I had to force > > that already for one caller and for the merge to trunk I

Re: [C++] Handle && || ! for simd vectors

2014-10-14 Thread Jason Merrill
On 10/13/2014 03:45 PM, Marc Glisse wrote: Ping https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00361.html (sorry that my message looked like I had committed as obvious) Indeed. OK. :) On Sat, 4 Oct 2014, Marc Glisse wrote: On Thu, 2 Oct 2014, Jason Merrill wrote: OK. Thanks. While commi

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-14 Thread Jason Merrill
On 10/13/2014 04:54 PM, DJ Delorie wrote: This is what I ended up with for the test case. It was a bit tricky since it only works with msp430x (not msp430) and requires the gnu extensions. Is this OK? If so, is there anything else, or can I check the whole mess in yet? Go ahead. Jason

Re: [PATCH, i386, Pointer Bounds Checker 31/x] Pointer Bounds Checker builtins for i386 target

2014-10-14 Thread Ilya Enkovich
On 10 Oct 21:20, Ilya Enkovich wrote: > 2014-10-10 20:45 GMT+04:00 Jeff Law : > > On 10/09/14 10:54, Uros Bizjak wrote: > >> > >> On Thu, Oct 9, 2014 at 4:07 PM, Ilya Enkovich > >> wrote: > >>> > >>> It appeared I changed a semantics of BNDMK expand when replaced tree > >>> operations with rtl one

Re: [PATCH] support ggc hash_map and hash_set

2014-10-14 Thread Richard Biener
On Tue, Sep 2, 2014 at 3:56 AM, wrote: > From: Trevor Saunders > > Hi, > > There are still some issues to make this work really nicely, but this part is > probably good enough its worth reviewing. > > For one thing you can't use ggc hash_map or set in front ends with some types > or gengtype wil

Re: [PATCH] Add D demangling support to libiberty

2014-10-14 Thread Joel Brobecker
Hello Ian, > libiberty/ChangeLog > 2014-08-05 Iain Buclaw > > * Makefile.in (CFILES): Add d-demangle.c. > (REQUIRED_OFILES): Add d-demangle.o. > * cplus-dem.c (libiberty_demanglers): Add dlang_demangling case. > (cplus_demangle): Likewise. > * d-demangle.c: New file. >

Re: New rematerialization sub-pass in LRA

2014-10-14 Thread Vladimir Makarov
On 10/13/2014 12:24 PM, Wilco Dijkstra wrote: >> Here is a new rematerialization sub-pass of LRA. >> >> I've tested and benchmarked the sub-pass on x86-64 and ARM. The >> sub-pass permits to generate a smaller code in average on both >> architecture (although improvement no-significant), adds

[PATCH][match-and-simplify] Remove/revert unneeded changes

2014-10-14 Thread Richard Biener
This removes duplicate/not needed code from generic-match-head.c and removes integral_op_p (if needed these new predicates should go to tree.h). It also revers one unnecessary Makefile.in change. Applied. Richard. 2014-10-14 Richard Biener * Makefile.in (BUILD_RTL): Revert not need

Re: [PATCH] Add D demangling support to libiberty

2014-10-14 Thread Ian Lance Taylor
On Tue, Oct 14, 2014 at 7:12 AM, Joel Brobecker wrote: > >> libiberty/ChangeLog >> 2014-08-05 Iain Buclaw >> >> * Makefile.in (CFILES): Add d-demangle.c. >> (REQUIRED_OFILES): Add d-demangle.o. >> * cplus-dem.c (libiberty_demanglers): Add dlang_demangling case. >> (cplus_demangl

Re: [PATCH 1/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-14 Thread H.J. Lu
On Mon, Oct 13, 2014 at 11:49 AM, H.J. Lu wrote: > On Mon, Oct 13, 2014 at 9:32 AM, Evgeny Stupachenko > wrote: >> Reattached. >> >> On Mon, Oct 13, 2014 at 8:22 PM, Uros Bizjak wrote: >>> On Mon, Oct 13, 2014 at 4:53 PM, Evgeny Stupachenko >>> wrote: >>> ChangeLog for testsuite: >>

Re: [PATCH] AutoFDO patch for trunk

2014-10-14 Thread Jan Hubicka
> Index: gcc/cgraphclones.c > === > --- gcc/cgraphclones.c(revision 215826) > +++ gcc/cgraphclones.c(working copy) > @@ -453,6 +453,11 @@ > } >else > count_scale = 0; > + /* In AutoFDO, if edge count is l

[PATCH x86, pr63534] Fix go bootstrap

2014-10-14 Thread Evgeny Stupachenko
Hi, Bootstaped with --enable-languages=c,c++,fortran,lto,go passed. Make check in progress. Is it ok? ChangeLog 2014-10-14 Evgeny Stupachenko * config/i386/i386.c (ix86_expand_split_stack_prologue): Make __morestack calls local. diff --git a/gcc/config/i386/i386.c b/gcc/con

Patches 5-10 of jit merger (was: Re: [PATCH 0/5] Merger of jit branch (v2))

2014-10-14 Thread David Malcolm
On Mon, 2014-10-13 at 13:45 -0400, David Malcolm wrote: > I'd like to merge the JIT branch into trunk: > https://gcc.gnu.org/wiki/JIT > > This is "v2" since it incorporates fixes for the various issues > identified by Joseph in an earlier submission: > https://gcc.gnu.org/ml/gcc-patches/2014-0

[PATCH 05/10] JIT-related changes outside of jit subdir

2014-10-14 Thread David Malcolm
ChangeLog: * MAINTAINERS (Various Maintainers): Add myself as jit maintainer. contrib/ChangeLog: * jit-coverage-report.py: New file: a script to print crude code-coverage information for the libgccjit API. gcc/ChangeLog: * doc/install.texi (--enable-host-shared): S

[PATCH 06/10] Heart of the JIT implementation (was: Re: [PATCH 0/5] Merger of jit branch (v2))

2014-10-14 Thread David Malcolm
On Tue, 2014-10-14 at 11:09 -0400, David Malcolm wrote: > On Mon, 2014-10-13 at 13:45 -0400, David Malcolm wrote: > > I'd like to merge the JIT branch into trunk: > > https://gcc.gnu.org/wiki/JIT > > > > This is "v2" since it incorporates fixes for the various issues > > identified by Joseph in

[PATCH 07/10] Testsuite for the JIT (Re: Patches 5-10 of jit merger (was: Re: [PATCH 0/5] Merger of jit branch (v2)))

2014-10-14 Thread David Malcolm
On Tue, 2014-10-14 at 11:09 -0400, David Malcolm wrote: > On Mon, 2014-10-13 at 13:45 -0400, David Malcolm wrote: > > I'd like to merge the JIT branch into trunk: > > https://gcc.gnu.org/wiki/JIT > > > > This is "v2" since it incorporates fixes for the various issues > > identified by Joseph in

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [16/n] Reduce bounds lifetime

2014-10-14 Thread Ilya Enkovich
On 09 Oct 11:32, Jeff Law wrote: > On 10/08/14 13:24, Ilya Enkovich wrote: > >Hi, > > > >This patch adds a bounds lifetime reduction into checker optimization. > > > >Thanks, > >Ilya > >-- > >2014-10-08 Ilya Enkovich > > > > * tree-chkp.c (chkp_reduce_bounds_lifetime): New. > > (chkp_opt

[PATCH 10/10] ChangeLog files (Re: Patches 5-10 of jit merger (was: Re: [PATCH 0/5] Merger of jit branch (v2)))

2014-10-14 Thread David Malcolm
On Tue, 2014-10-14 at 11:09 -0400, David Malcolm wrote: > On Mon, 2014-10-13 at 13:45 -0400, David Malcolm wrote: > > I'd like to merge the JIT branch into trunk: > > https://gcc.gnu.org/wiki/JIT > > > > This is "v2" since it incorporates fixes for the various issues > > identified by Joseph in

Re: [PATCH] Add zero-overhead looping for xtensa backend

2014-10-14 Thread Felix Yang
PING? Cheers, Felix On Tue, Oct 14, 2014 at 12:30 AM, Felix Yang wrote: > Thanks for the comments. > > The patch checked the usage of teh trip count register, making sure > that it is not used in the loop body other than the doloop_end or > lives past the doloop_end instruction, as the following

Re: [PATCH x86, pr63534] Fix go bootstrap

2014-10-14 Thread Richard Henderson
On 10/14/2014 08:08 AM, Evgeny Stupachenko wrote: > Hi, > > Bootstaped with --enable-languages=c,c++,fortran,lto,go passed. > Make check in progress. > > Is it ok? > > ChangeLog > > 2014-10-14 Evgeny Stupachenko > > * config/i386/i386.c (ix86_expand_split_stack_prologue): Make >

Re: [PATCH x86, pr63534] Fix go bootstrap

2014-10-14 Thread Jakub Jelinek
On Tue, Oct 14, 2014 at 08:43:39AM -0700, Richard Henderson wrote: > On 10/14/2014 08:08 AM, Evgeny Stupachenko wrote: > > Hi, > > > > Bootstaped with --enable-languages=c,c++,fortran,lto,go passed. > > Make check in progress. > > > > Is it ok? > > > > ChangeLog > > > > 2014-10-14 Evgeny Stupa

Re: [PATCH 3/5] timevar.h: Add an auto_timevar class

2014-10-14 Thread David Malcolm
On Tue, 2014-10-14 at 11:03 +0200, Richard Biener wrote: > On Mon, Oct 13, 2014 at 7:45 PM, David Malcolm wrote: > > This is used in a couple of places in jit/jit-playback.c to ensure > > that we pop the timevar on every exit path from a function. > > > > I could rewrite them if need be, but it do

Re: [PATCH] Implement -fsanitize=object-size

2014-10-14 Thread Jakub Jelinek
On Fri, Oct 10, 2014 at 12:26:44PM +0200, Jakub Jelinek wrote: > 2014-10-10 Jakub Jelinek > > * ubsan/Makefile.am (DEFS): Add -DPIC. > * ubsan/Makefile.in: Regenerated. I've now bootstrapped/regtested this on x86_64-linux and i686-linux and committed as obvious. 2014-10-14 Jakub

RE: New rematerialization sub-pass in LRA

2014-10-14 Thread Wilco Dijkstra
> Vladimir Makarov wrote: > > On SPECINT2k performance is ~0.5% worse (5.5% regression on perlbmk), and > > SPECFP is ~0.2% faster. > Thanks for reporting this. It is important for me as I have no aarch64 > machine for benchmarking. > > Perlbmk performance degradation is too big and I'll definite

[PATCH] Fix optimize_range_tests_diff

2014-10-14 Thread Jakub Jelinek
Hi! When hacking on range reassoc opt, I've noticed we can emit code with undefined behavior even when there wasn't one originally, in particular for: (X - 43U) <= 3U || (X - 75U) <= 3U and this loop can transform that into ((X - 43U) & ~(75U - 43U)) <= 3U. */ we actually don't transform

Re: [PATCH 3/5] IPA ICF pass

2014-10-14 Thread Jan Hubicka
> diff --git a/gcc/cgraph.h b/gcc/cgraph.h > index fb41b01..2de98b4 100644 > --- a/gcc/cgraph.h > +++ b/gcc/cgraph.h > @@ -172,6 +172,12 @@ public: >/* Dump referring in list to FILE. */ >void dump_referring (FILE *); > > + /* Get number of references for this node. */ > + inline unsi

[gomp] [3/3] OpenACC 2.0 support for libgomp - documentation

2014-10-14 Thread Julian Brown
This is a version of the patch: https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02024.html against gomp4 branch instead of mainline. OK to apply? Thanks, Julian -xx-xx Thomas Schwinge James Norris libgomp/ * libgomp.texi: Outline documentation for OpenACC. From c58

Re: [PATCH] Add D demangling support to libiberty

2014-10-14 Thread Iain Buclaw
On 14 October 2014 15:28, Ian Lance Taylor wrote: > On Tue, Oct 14, 2014 at 7:12 AM, Joel Brobecker wrote: >> >>> libiberty/ChangeLog >>> 2014-08-05 Iain Buclaw >>> >>> * Makefile.in (CFILES): Add d-demangle.c. >>> (REQUIRED_OFILES): Add d-demangle.o. >>> * cplus-dem.c (libiberty_d

RE: New rematerialization sub-pass in LRA

2014-10-14 Thread Wilco Dijkstra
> Wilco Dijkstra wrote: > > Vladimir Makarov wrote: > > > On SPECINT2k performance is ~0.5% worse (5.5% regression on perlbmk), and > > > SPECFP is ~0.2% faster. > > Thanks for reporting this. It is important for me as I have no aarch64 > > machine for benchmarking. > > > > Perlbmk performance deg

[PATCH] PR lto/61048 Define missed builtins on demand

2014-10-14 Thread Ilya Palachev
Hi all, Attached patch fixes PR lto/61048 - https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61048 The reason of failure was that the builtin information structure was not initialized properly at the link stage. The failed assertion was caused by missing builtin declaration ( BUILT_IN_ASAN_AFTER

Re: [PATCH 1/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-14 Thread Jeff Law
On 10/14/14 07:00, Jakub Jelinek wrote: For the first two, I think (and said it before already) that the current model of emitting set_got from a target hook during RA can't work, as there can be calls in the prologue, and the prologue is inserted before the set_got in that case. I really think

Re: [PATCH 1/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-14 Thread H.J. Lu
On Tue, Oct 14, 2014 at 9:43 AM, Jeff Law wrote: > > RA improvements are the way to go -- however, my understanding is that > overall the code is better now than it was before Intel's changes, so I > don't consider the performance side as a blocker for this code. > The new approach improves PIC c

Re: [PATCH 2/3] libstdc++: Add put_time support.

2014-10-14 Thread Jonathan Wakely
On 13/10/14 16:28 +0100, Jonathan Wakely wrote: On 13/10/14 13:08 +0100, Jonathan Wakely wrote: On 15/04/14 23:20 +0200, Rüdiger Sonderfeld wrote: Described in [ext.manip]. * libstdc++-v3/include/std/iomanip (_Put_time): New struct. (put_time): New manipulator. (operator<<): New overloaded fun

Re: [PATCH] Add D demangling support to libiberty

2014-10-14 Thread Joel Brobecker
> I've just seen this, so I'll repeat what I've said in gdb patches too. > > The call to strtold is only needed to decode templates which have a > floating point value encoded inside. This value may or may not have a > greater than double precision. > > Replacing long double with double will be f

[patch] Make std::align tests depend on stdint.h

2014-10-14 Thread Jonathan Wakely
Tested x86_64-linux, committed to trunk.

Re: [PATCH] Fix optimize_range_tests_diff

2014-10-14 Thread Richard Biener
On October 14, 2014 6:02:19 PM CEST, Jakub Jelinek wrote: >Hi! > >When hacking on range reassoc opt, I've noticed we can emit >code with undefined behavior even when there wasn't one originally, >in particular for: > (X - 43U) <= 3U || (X - 75U) <= 3U > and this loop can transform that into >

Re: [PATCH] Fix optimize_range_tests_diff

2014-10-14 Thread Jeff Law
On 10/14/14 10:02, Jakub Jelinek wrote: Hi! When hacking on range reassoc opt, I've noticed we can emit code with undefined behavior even when there wasn't one originally, in particular for: (X - 43U) <= 3U || (X - 75U) <= 3U and this loop can transform that into ((X - 43U) & ~(75U -

[patch] Update libstdc++ status docs

2014-10-14 Thread Jonathan Wakely
Committed to trunk. commit a94516a841a0588c6c7bf95248c2eaefd5e406f1 Author: Jonathan Wakely Date: Tue Oct 14 18:21:03 2014 +0100 * doc/xml/manual/intro.xml: Update. * doc/xml/manual/status_cxx2011.xml: Update. * doc/html/manual/*: Regenerate. diff --git a/libstdc++-v3/doc/xml/m

Re: [PATCH] Add D demangling support to libiberty

2014-10-14 Thread Ian Lance Taylor
On Tue, Oct 14, 2014 at 10:07 AM, Joel Brobecker wrote: > > libiberty/ChangeLog: > > * d-demangle.c: Replace strtold with strtod in global comment. > (strtold): Remove declaration. > (strtod): New declaration. > (dlang_parse_real): Declare value as double instead of

Re: [PATCH] Add D demangling support to libiberty

2014-10-14 Thread Iain Buclaw
On 14 October 2014 18:07, Joel Brobecker wrote: >> I've just seen this, so I'll repeat what I've said in gdb patches too. >> >> The call to strtold is only needed to decode templates which have a >> floating point value encoded inside. This value may or may not have a >> greater than double precis

Re: [PATCH] Fix optimize_range_tests_diff

2014-10-14 Thread Jakub Jelinek
On Tue, Oct 14, 2014 at 11:23:22AM -0600, Jeff Law wrote: > On 10/14/14 10:02, Jakub Jelinek wrote: > >When hacking on range reassoc opt, I've noticed we can emit > >code with undefined behavior even when there wasn't one originally, > >in particular for: > >(X - 43U) <= 3U || (X - 75U) <= 3U >

Re: [PATCH] Fix optimize_range_tests_diff

2014-10-14 Thread Jeff Law
On 10/14/14 11:40, Jakub Jelinek wrote: On Tue, Oct 14, 2014 at 11:23:22AM -0600, Jeff Law wrote: On 10/14/14 10:02, Jakub Jelinek wrote: When hacking on range reassoc opt, I've noticed we can emit code with undefined behavior even when there wasn't one originally, in particular for: (X - 4

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [4/n] Memory accesses instrumentation

2014-10-14 Thread Jeff Law
On 10/14/14 04:08, Ilya Enkovich wrote: Are you just looking for the parameter in which we pass the static chain? Look at get_chain_decl for how we set it up. You may actually have to peek at more fields. I don't think there's a single magic bit that says "this is the static chain". Though i

Re: [PATCH] Add D demangling support to libiberty

2014-10-14 Thread Joel Brobecker
> > libiberty/ChangeLog: > > > > * d-demangle.c: Replace strtold with strtod in global comment. > > (strtold): Remove declaration. > > (strtod): New declaration. > > (dlang_parse_real): Declare value as double instead of long > > double. Replace call to strt

Re: [PATCH 2/3] libstdc++: Add put_time support.

2014-10-14 Thread Rüdiger Sonderfeld
On Tuesday 14 October 2014 18:01:59 Jonathan Wakely wrote: > So let's just test the full name and not worry about how it's > abbreviated. > > Tested x86_64-linux, committed to trunk. Sorry for causing the trouble. I had it tested on my local machine. Maybe the de_DE.utf8 locale is different.

Re: [PATCH, DWARF] re-init dw_frame_pointer_regnum between functions

2014-10-14 Thread Richard Henderson
On 10/14/2014 06:02 AM, Christian Bruel wrote: > 2014-09-23 Christian Bruel > > * execute_dwarf2_frame (dw_frame_pointer_regnum): Reinitialize for each > function. It's tempting to make this a local variable within dwarf2out_frame_debug_expr and not try to cache it at all. But this is

Re: [PATCH, DWARF] re-init dw_frame_pointer_regnum between functions

2014-10-14 Thread Richard Henderson
On 10/14/2014 11:25 AM, Richard Henderson wrote: > On 10/14/2014 06:02 AM, Christian Bruel wrote: >> 2014-09-23 Christian Bruel >> >> * execute_dwarf2_frame (dw_frame_pointer_regnum): Reinitialize for each >> function. > > It's tempting to make this a local variable within dwarf2out_frame

Re: [PATCH, rtl-optimization] Fix PR63475, Postreload CSE propagates aliased memory operand

2014-10-14 Thread Jeff Law
On 10/14/14 01:11, Uros Bizjak wrote: 2014-10-14 Uros Bizjak PR rtl-optimization/63475 * alias.c (true_dependence_1): Always use get_addr to extract true address operands from x_addr and mem_addr. Use extracted address operands to check for references with alignment ANDs

Re: [PATCH i386 AVX512] [56/n] Add plus/minus/abs/neg/andnot insn patterns.

2014-10-14 Thread Uros Bizjak
On Tue, Oct 14, 2014 at 9:18 AM, Kirill Yukhin wrote: > Hello Uroš, > It seems like I missed to post uppdated patch. > On 25 Sep 20:11, Uros Bizjak wrote: >> I'd rather go with the second approach, it is less confusing from the >> maintainer POV. All other patterns with masking use some consistent

Re: [PATCH 2/2] Fix ILP32 ld.so.

2014-10-14 Thread Richard Henderson
On 08/08/2014 08:51 PM, Andrew Pinski wrote: > ChangeLog: > * explow.c (convert_memory_address_addr_space): Rename to ... > (convert_memory_address_addr_space_1): This. Add in_const argument. > Inside a CONST RTL, permute the conversion and addition of constant > for zero a

[PATCH v2 03/13] Allow the static chain to be set from C

2014-10-14 Thread Richard Henderson
Replacing the hacky v1 with the proposed syntax relayed by PCC, and changing the name to __builtin_call_with_static_chain. Which is kinda long, but at least it's more properly descriptive. Adds documentation and an errors test case. r~ >From 7e31234f2e112bad576b748b2ff6cc615194c0f7 Mon Sep 17 0

Re: [PATCH] Fix typo in comment for IRA

2014-10-14 Thread Jeff Law
On 10/13/14 20:49, Kito Cheng wrote: Hi Marc: - -1 if it is not a cost classe. */ + -1 if it is not a cost classes. */ "a cost class", no plural here. Thank you for correcting me :) Hi Jeff: Thanks, and updated patch in attachment, However I don't have commit right yet, can you

Re: RFA: fix mode confusion in caller-save.c:replace_reg_with_saved_mem

2014-10-14 Thread Jeff Law
On 10/13/14 18:16, Joern Rennecke wrote: On 13 October 2014 20:43, Jeff Law wrote: ... I think you want "smode" in the mode_for_size call rather than "mode", right (both instances)? No, nregs is the number of hard registers of regno in "mode". Hence we must use the size of "mode". OK. My b

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-14 Thread DJ Delorie
> > extensions. Is this OK? If so, is there anything else, or can I > > check the whole mess in yet? > > Go ahead. Thanks! Committed.

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-14 Thread Markus Trippelsdorf
On 2014.08.25 at 23:03 -0400, DJ Delorie wrote: > > > I'd like to see the updated version of the whole of patch 3 (tested > > to be actually independent of the other patches) for review, though > > I won't be reviewing the C++ parts. > > Here it is. Tested on x86_64. I include the msp430-modes.

Re: New rematerialization sub-pass in LRA

2014-10-14 Thread Peter Bergner
On Fri, 2014-10-10 at 11:02 -0400, Vladimir Makarov wrote: > Here is a new rematerialization sub-pass of LRA. When Mike and I build with this patch along with the patch that enables LRA by default on powerpc64*-linux (attached below), we're seeing the following error message. I'm not sure how you

Fix PR ada/62019

2014-10-14 Thread Eric Botcazou
Someone broke again weak external symbols in Ada in exactly the same way as: https://gcc.gnu.org/ml/gcc-patches/2014-01/msg00431.html probably during the ongoing C++ reshuffling: FAIL: gnat.dg/weak2.adb (test for excess errors) Tested on x86_64-suse-linux, applied on the mainline as obvious.

Re: [PATCH] AutoFDO patch for trunk

2014-10-14 Thread Dehao Chen
On Tue, Oct 14, 2014 at 8:02 AM, Jan Hubicka wrote: >> Index: gcc/cgraphclones.c >> === >> --- gcc/cgraphclones.c(revision 215826) >> +++ gcc/cgraphclones.c(working copy) >> @@ -453,6 +453,11 @@ >> } >>else >>

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-14 Thread DJ Delorie
> ../../gcc/gcc/config/rs6000/rs6000-c.c:237:24: error: ‘RID_INT128’ was not > declared in this scope Two options: 1. If you know the RS6000 will never have any __intN other than __int128, just use RID_INT_N_0, although this is a hack it will work as long as there *is* an __int128 for RS6

Re: __intN patch 5/5: msp430-specific changes

2014-10-14 Thread DJ Delorie
> This is the MSP430-specific use of the new intN framework to enable > true 20-bit pointers. Since I'm one of the MSP430 maintainers, this > patch is being posted for reference, not for approval. Now that the other parts are committed, I'm checking this one in too. > gcc/config/msp430 >

Re: [PATCH] AutoFDO patch for trunk

2014-10-14 Thread Dehao Chen
The new patch is attached. I used clang-format for format auto-profile.{c|h} Thanks, Dehao On Tue, Oct 14, 2014 at 2:05 PM, Dehao Chen wrote: > On Tue, Oct 14, 2014 at 8:02 AM, Jan Hubicka wrote: >>> Index: gcc/cgraphclones.c >>> =

[Bug libstdc++/63500] [4.9/5 Regression] bug in debug version of std::make_move_iterator?

2014-10-14 Thread François Dumont
Hi Here is a proposal to fix the issue with iterators which do not expose lvalue references when dereferenced. I simply chose to detect such an issue in c++11 mode thanks to the is_lvalue_reference meta function. 2014-10-15 François Dumont PR libstdc++/63500 * include/bits/cpp

Re: New rematerialization sub-pass in LRA

2014-10-14 Thread Vladimir Makarov
On 2014-10-14 4:17 PM, Peter Bergner wrote: On Fri, 2014-10-10 at 11:02 -0400, Vladimir Makarov wrote: Here is a new rematerialization sub-pass of LRA. When Mike and I build with this patch along with the patch that enables LRA by default on powerpc64*-linux (attached below), we're seeing the

Re: [PATCH] AutoFDO patch for trunk

2014-10-14 Thread Andi Kleen
Dehao Chen writes: > + > +@item -fauto-profile > +@itemx -fauto-profile=@var{path} > +@opindex fauto-profile > +Enable sampling based feedback directed optimizations, and optimizations > +generally profitable only with profile feedback available. > + > +The following options are enabled: @code{-fb

Re: [PATCH] support ggc hash_map and hash_set

2014-10-14 Thread Trevor Saunders
On Tue, Oct 14, 2014 at 04:05:25PM +0200, Richard Biener wrote: > On Tue, Sep 2, 2014 at 3:56 AM, wrote: > > From: Trevor Saunders > > > > Hi, > > > > There are still some issues to make this work really nicely, but this part > > is > > probably good enough its worth reviewing. > > > > For one

Re: [PATCH 1/2] xtensa: drop unimplemented floating point operations

2014-10-14 Thread Max Filippov
On Mon, Oct 13, 2014 at 8:05 PM, augustine.sterl...@gmail.com wrote: > On Sun, Oct 12, 2014 at 3:46 PM, Max Filippov wrote: >> xtensa ISA never implemented FP division, reciprocal, square root and >> inverse square root as single opcode. Remove patterns that can emit >> them. >> >> 2014-10-09 Ma

Re: [PATCH 2/2] xtensa: use pre- and postincrement FP load/store when available

2014-10-14 Thread Max Filippov
On Mon, Oct 13, 2014 at 8:04 PM, augustine.sterl...@gmail.com wrote: > On Sun, Oct 12, 2014 at 3:46 PM, Max Filippov wrote: >> 2014-10-10 Max Filippov >> >> gcc/ >> * config/xtensa/xtensa.h (TARGET_HARD_FLOAT_POSTINC): new macro. >> * config/xtensa/xtensa.md (*lsiu, *ssiu): add depende

[committed] MAINTAINERS: add myself to write-after-approval list.

2014-10-14 Thread Max Filippov
2014-10-15 Max Filippov * MAINTAINERS (write-after-approval): Add myself. Index: MAINTAINERS === --- MAINTAINERS (revision 216231) +++ MAINTAINERS (revision 216232) @@ -380,6 +380,7 @@ Chris Fairles

[PATCH] Better tolerance of incoming profile insanities in jump threading

2014-10-14 Thread Teresa Johnson
The below patch fixes the overflow detection when recomputing probabilities after jump threading, in case of incoming profile insanities. It detects more cases where the computation will overflow not only the max probability but the max int and possibly wrap around. LTO profilebootstrapped and tes

Re: libffi patch RFA: Pass -Qunused-arguments for asm files

2014-10-14 Thread Paolo Bonzini
Il 30/09/2014 02:12, Ian Lance Taylor ha scritto: > Similar to a recent patch to libgo, this patch to the libffi configure > script checks whether the compiler support -Qunused-arguments. If it > does, it passes -Qunused-arguments when invoking the compiler on .s > files. This is because the clan

Re: [patch] Add -static-libquadmath option

2014-10-14 Thread FX
ping Patch needs: - C/driver options maintainer, or global reviewer, to OK the C changes (outside darwin). They are really simple - Fortran maintainer to OK the Fortran part > Version 2 of the patch, now handling the darwin case (thanks Iain) and > expressely noting in the documentation th