[PATCH] Make autofdo tests safer for parallel builds

2016-08-06 Thread Andi Kleen
From: Andi Kleen There were some reports that the autofdo tests are non deterministic with parallel builds. I wasn't able to reproduce this, but here are two changes that may help: - Always use unique file names for temporary files. - Don't print file names in the test log because the directorie

[PATCH] Fix warning breaking profiled bootstrap

2016-08-06 Thread Andi Kleen
From: Andi Kleen This patch fixes an bootstrap error with autoprofiledbootstrap due to uninitiliazed variables, because the compiler cannot figure out they don't need to be initialized in an error path. Just always initialize them. gcc/: 2016-08-06 Andi Kleen * tree-vrp.c (get_singl

Re: [RS6000] Force source of fix_truncsi2 to reg

2016-08-06 Thread Alan Modra
On Fri, Aug 05, 2016 at 04:25:26PM -0400, Michael Meissner wrote: > Ummm, this patch looks wrong. Because the insn uses the SFDF iterator, the > mode of operands[1] could be either SFmode or DFmode. I think it should be: > > - rtx tmp, stack; > + rtx src = force_reg (mode, operands[1])

Re: [Patch] Implement std::experimental::variant

2016-08-06 Thread Ville Voutilainen
On 6 August 2016 at 08:45, Tim Shen wrote: >> using namespace std; >> constexpr variant v{}; >> constexpr variant v1{in_place_type}; >> constexpr variant v2{in_place_index<0>}; > > Good news! This compiles now! I learned the technique from Anthony > Williams's implementation, whose code also c

Re: [PATCH GCC]Resolve compilation time known alias checks in vectorizer

2016-08-06 Thread Andreas Schwab
On Mi, Jul 13 2016, "Bin.Cheng" wrote: > Patch re-tested/applied on trunk as r238301. This breaks gcc.dg/vect/vect-117.c on powerpc. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something completely dif

[PATCH] Fix regno_in_use_p for RTL checking bootstrap (PR rtl-optimization/72821)

2016-08-06 Thread Jakub Jelinek
On Wed, Aug 03, 2016 at 02:59:30PM -0400, Vladimir N Makarov wrote: > --- lra-spills.c (revision 239000) > +++ lra-spills.c (working copy) > @@ -686,16 +686,40 @@ return_regno_p (unsigned int regno) >return false; > } > > -/* Return true if REGNO is one of subsequent USE after INSN

[PATCH] Allow relayout_decl on FIELD_DECLs (PR c/72816)

2016-08-06 Thread Jakub Jelinek
Hi! As the testcase shows, the C FE can call relayout_decl even on FIELD_DECLs in certain cases. Trying to call only layout_decl on FIELD_DECL and relayout_decl on other decls would be insufficient, we'd need to repeat there most of the relayout_decl code (except for SET_DECL_RTL, which FIELD_DEC

Re: [tree-ssa-ccp] modify extend_mask to extend bits based on signop

2016-08-06 Thread Prathamesh Kulkarni
On 5 August 2016 at 13:52, Richard Biener wrote: > On Fri, 5 Aug 2016, Prathamesh Kulkarni wrote: > >> Ah, the mail failed to be delivered to gcc-patches, sorry for the >> double-post. >> On 5 August 2016 at 01:26, Prathamesh Kulkarni >> wrote: >> > Hi, >> > Is the attached patch OK ? >> > Since

[committed] Fix crash in selftest::test_lexer_string_locations_ucn4 (PR bootstrap/72823)

2016-08-06 Thread David Malcolm
On Sat, 2016-08-06 at 11:29 +0530, Prathamesh Kulkarni wrote: > On 6 August 2016 at 11:16, Markus Trippelsdorf < > mar...@trippelsdorf.de> wrote: > > On 2016.08.05 at 14:16 -0400, David Malcolm wrote: > > > Successfully bootstrapped®rtested the updated patch on x86_64 > > > -pc > > > -linux-gnu, an

Re: [PATCH, www] Mention download_prerequisites in installation docs

2016-08-06 Thread Manish Goregaokar
I'm a GDB committer and have filed bugs on GCC's bugzilla before; one of these could be the reason behind me having that account (I don't know, I don't remember making it :) ). I'll need someone to push it for me. Your proposed improvement sounds good to me. Thanks, -Manish On Sat, Aug 6, 2016 a

Re: [PATCH, www] Mention download_prerequisites in installation docs

2016-08-06 Thread Gerald Pfeifer
Hi Manish, On Thu, 30 Jun 2016, Manish Goregaokar wrote: > * doc/install.texi: Mention download_prerequisites this is okay (and sorry for the delay, I have unexpectedly "disappeared" for the month of July). One suggestion: > +install the libraries. Alternatively, running > +@code{contrib/d

libgo patch committed: fix build for 32-bit PPC

2016-08-06 Thread Ian Lance Taylor
This patch fixes the libgo build for 32-bit PPC, by adding a "ppc" build constraint to the internal/syscall/unix library. Tested by verifying that match.sh returned the right file, and by verifying that the build on x86_64-pc-linux-gnu did not change. Ian Index: gcc/go/gofrontend/MERGE ==

Re: protected alloca class for malloc fallback

2016-08-06 Thread Richard Biener
On August 6, 2016 12:15:26 PM GMT+02:00, Aldy Hernandez wrote: >On 08/05/2016 04:07 PM, Richard Biener wrote: >> On August 5, 2016 8:15:54 PM GMT+02:00, Oleg Endo > wrote: >>> On Fri, 2016-08-05 at 19:55 +0200, Richard Biener wrote: >>> Please don't use std::string. For string building

Re: [PATCH] Update C++17 library implementation status table

2016-08-06 Thread Jonathan Wakely
On 04/08/16 14:32 +0100, Jonathan Wakely wrote: + Minimal incomplete type support for standard containers + + http://www.w3.org/1999/xlink"; xlink:href="http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4510.html";> + N4510 + + + 7 (3.0) +

Re: [PATCH] Use __invoke in std::function internals

2016-08-06 Thread Jonathan Wakely
On 05/08/16 10:00 +0100, Jonathan Wakely wrote: This fixes a bug in the _Callable SFINAE constraint of std::function, where it was using an rvalue _Func in the result_of expression, when the target is actually invoked as an lvalue. I'm backporting this part to gcc-5 and gcc-6. Tested x86_64-li

Re: [PATCH] Remove unnecessary jump threading restriction

2016-08-06 Thread Richard Biener
On Fri, Aug 5, 2016 at 10:43 PM, Jeff Law wrote > On 08/05/2016 01:45 AM, Richard Biener wrote: >> >> >> There is no need to avoid threading to a loop header, the threading >> code can cope with this just fine. Noticed when working on PR72772. >> >> Bootstrapped and tested on x86_64-unknown-linux

Re: [PATCH] Improve comments on check_v3_target_filesystem_ts

2016-08-06 Thread Jonathan Wakely
On 02/08/16 20:35 +0100, Jonathan Wakely wrote: * testsuite/lib/libstdc++.exp (check_v3_target_filesystem_ts): Improve comments. Oops, I've fixed the typo in the new comment. commit 5b137ebccbc667dbd868c575d79903ca58cef5e6 Author: Jonathan Wakely Date: Sat Aug 6 11:44:02

Re: [PATCH] Update C++17 library implementation status table

2016-08-06 Thread Jonathan Wakely
On 04/08/16 14:32 +0100, Jonathan Wakely wrote: This adds all the features from the latest draft of SD-6, and reformats the table to be closer to the tables at https://gcc.gnu.org/projects/cxx-status.html (including adding the feature-test macros). I'd like to do the same for the C++14 status t

Re: [PATCH] gcov tool: Implement Hawick's algorithm for cycle detection, (PR gcov-profile/67992)

2016-08-06 Thread Jakub Jelinek
On Fri, Aug 05, 2016 at 08:27:39AM -0700, Andrew Pinski wrote: > On Fri, Aug 5, 2016 at 12:32 AM, Martin Liška wrote: > > On 08/05/2016 09:30 AM, Martin Liška wrote: > >> Hi. > >> > >> Sorry for the mistake with the enum, that was silly ;) > >> New patch version also handles the unnecessary braces

[PATCH] Use ::new to avoid finding overloaded operator new

2016-08-06 Thread Jonathan Wakely
As I mentioned in the variant patch thread yesterday, we're not qualifying the new operator correctly in std::function, so the test case in this patch was failing. PR libstdc++/72820 * include/std/functional (_Function_base::_Base_manager::_M_clone): Qualify new operator.

Re: protected alloca class for malloc fallback

2016-08-06 Thread Aldy Hernandez
On 08/05/2016 04:07 PM, Richard Biener wrote: On August 5, 2016 8:15:54 PM GMT+02:00, Oleg Endo wrote: On Fri, 2016-08-05 at 19:55 +0200, Richard Biener wrote: Please don't use std::string. For string building you can use obstacks. Just out of curiosity ... why? I remember there was som

Re: protected alloca class for malloc fallback

2016-08-06 Thread Aldy Hernandez
On 08/05/2016 04:07 PM, Richard Biener wrote: On August 5, 2016 8:15:54 PM GMT+02:00, Oleg Endo wrote: On Fri, 2016-08-05 at 19:55 +0200, Richard Biener wrote: Please don't use std::string. For string building you can use obstacks. Just out of curiosity ... why? I remember there was som

Ping [Fortran, Patch, pr70524, v1] [5/6/7 Regression] ICE when using -frepack-arrays -Warray-temporaries

2016-08-06 Thread Andre Vehreschild
Ping. On Mon, 25 Jul 2016 12:55:14 +0200 Andre Vehreschild wrote: > Hi all, > > the attached patch fixes the ICE when the options in the title are > used. Two issues caused this ICE: > > 1. the error-printing routines relied on the locus.nextc which was not > set by the gfc_set_backend_locus()

[testsuite] add require-effective-target c99_runtime for pr71078-*.c tests

2016-08-06 Thread Prathamesh Kulkarni
Hi Richard, The patch for PR71078 broke the test-cases for arm and aarch64 bare metal targets :/ In the attached patch, restricting the tests to c99_runtime. Sorry for the breakage. Ok for trunk ? Thanks, Prathamesh 2016-08-06 Prathamesh Kulkarni testsuite/ * gcc.dg/tree-ssa/pr71078-1.