Re: [PATCH] (v2) Tweaks to print_rtx_function

2016-10-14 Thread Bernd Schmidt
On 10/14/2016 10:12 PM, David Malcolm wrote: gcc/ChangeLog: * print-rtl-function.c (print_edge): Omit "(flags)" when none are set. (print_rtx_function): Update example in comment for... * print-rtl.c (print_rtx_operand_code_r): In compact mode, print

Re: fix -fmax-errors & notes

2016-10-14 Thread David Malcolm
On Fri, 2016-10-14 at 15:50 -0400, Nathan Sidwell wrote: > On 10/14/16 15:17, David Malcolm wrote: > > > "Limits the maximum number of error messages to @var{n}, at which > > point > > GCC bails out rather than attempting to continue processing the > > source > > code. If @var{n} is 0 (the

[wwwdocs, coding conventions] Mention OVERRIDE/FINAL

2016-10-14 Thread David Malcolm
On Fri, 2016-10-14 at 16:27 +0100, Pedro Alves wrote: > On 10/12/2016 03:13 PM, Bernd Schmidt wrote: > > On 10/12/2016 04:09 PM, Pedro Alves wrote: > > > > > > Thanks. Here's a follow up patch that I was just testing. > > > > > > Need this if building with "g++ -std=gnu++11", with gcc < 4.7. >

Re: [wwwdocs, coding conventions] Mention OVERRIDE/FINAL

2016-10-14 Thread Pedro Alves
On 10/14/2016 10:28 PM, David Malcolm wrote: > I propose that we update our coding conventions to mention the OVERRIDE > and FINAL macros in the paragraph that discusses virtual funcs. > > The attached patch (to the website) does so. > Good idea, I like it. GDB is following GCC's C++ coding

[PATCH, libfortran] PR 48587 Newunit allocator

2016-10-14 Thread Janne Blomqvist
Currently GFortran newer reuses unit numbers allocated with NEWUNIT=, instead having a simple counter that is decremented each time such a unit is opened. For a long running program which repeatedly opens files with NEWUNIT= and closes them, the counter can wrap around and cause an abort. This

Re: fix -fmax-errors & notes

2016-10-14 Thread Nathan Sidwell
On 10/14/16 15:17, David Malcolm wrote: "Limits the maximum number of error messages to @var{n}, at which point GCC bails out rather than attempting to continue processing the source code. If @var{n} is 0 (the default), there is no limit on the number of error messages produced. If

Re: [PATCH v2,rs6000] Add built-in function support for Power9 string operations

2016-10-14 Thread Segher Boessenkool
On Thu, Oct 13, 2016 at 09:45:22AM -0600, Kelvin Nilsen wrote: > 3. Replace magic number 74 with CR6_REGNO in vsx.md (2 occurrences) >and vector.md (3 occurrences). Some remain, see below. > +moves bytes 16 - @code{len} to 15 of the corresponding vector. For the > +the element to be

Re: [PATCH, libfortran] PR 48587 Newunit allocator

2016-10-14 Thread Janne Blomqvist
On Fri, Oct 14, 2016 at 8:01 PM, Bernhard Reutner-Fischer wrote: > On 13 October 2016 22:08:21 CEST, Jerry DeLisle wrote: >>On 10/13/2016 08:16 AM, Janne Blomqvist wrote: > >>> >>> Regtested on x86_64-pc-linux-gnu. Ok for trunk? >>> >> >>Yes, OK,

Re: [PATCH] Improve DWARF constant attribute langhooks

2016-10-14 Thread Jason Merrill
OK. On Fri, Oct 14, 2016 at 1:29 PM, Jakub Jelinek wrote: > Hi! > > Before early dwarf changes, if we wanted to note some decl property so that > some corresponding DWARF attribute can be emitted, we had to use some > generic IL bit for that. Now a langhook can be used instead

C++ PATCH for P0017, C++17 aggregates with bases

2016-10-14 Thread Jason Merrill
Implementing this proposal was a pretty straightforward matter of changing the definition of aggregate and treating artificial base fields as initializable in aggregate initialization. For this to work with empty bases, I needed to create base fields for them, which we haven't done in the past.

Re: [PATCH, rs6000] pr65479 Add option to fix failing asan test cases.

2016-10-14 Thread Segher Boessenkool
On Fri, Oct 14, 2016 at 02:37:42PM -0500, Bill Seurer wrote: > [PATCH, rs6000] pr65479 Add option to fix failing asan test cases. > > This patch adds the -fasynchronous-unwind-tables option to several of the asan > test cases. The option causes a full strack trace to be produced when the >

Re: [PATCH] Check \0-termination of string in c_getstr (simplified version)

2016-10-14 Thread Richard Biener
On Thu, Oct 13, 2016 at 5:23 PM, Martin Liška wrote: > Hello. > > After receiving feedback from Richi and Wilco Dijkstra, I decided to fully not > support not null-terminated strings. It brings more complications and the > code has started > to be overengineered. Thus c_getstr

Re: Questionable code in gcov-io.c

2016-10-14 Thread Nathan Sidwell
On 10/13/16 18:10, Andrew Pinski wrote: /home/jenkins/workspace/BuildToolchainAARCH64_thunder_elf_upstream/toolchain/scripts/../src/libgcc/libgcov-driver.c:53:0: /home/jenkins/workspace/BuildToolchainAARCH64_thunder_elf_upstream/toolchain/scripts/../src/libgcc/../gcc/gcov-io.c: In function

Re: [PATCH] Remove x86 pcommit instruction

2016-10-14 Thread Richard Biener
On Thu, Oct 13, 2016 at 11:39 PM, H.J. Lu wrote: > On Thu, Oct 13, 2016 at 5:09 AM, Andrew Senkevich > wrote: >> 2016-10-11 20:09 GMT+03:00 H.J. Lu : >>> On Tue, Oct 11, 2016 at 10:04 AM, Andrew Senkevich >>>

Re: [PATCH] Remove x86 pcommit instruction

2016-10-14 Thread Richard Biener
On Fri, Oct 14, 2016 at 11:23 AM, Richard Biener wrote: > On Thu, Oct 13, 2016 at 11:39 PM, H.J. Lu wrote: >> On Thu, Oct 13, 2016 at 5:09 AM, Andrew Senkevich >> wrote: >>> 2016-10-11 20:09 GMT+03:00 H.J. Lu

Re: [PATCH] Add "__RTL" to cc1 (v2)

2016-10-14 Thread Bernd Schmidt
On 10/14/2016 11:33 AM, Richard Biener wrote: Ok, so at least for the GIMPLE FE side I thought it's useful to allow a correctness verification with something simpler than pattern matching on the pass output. By means of doing runtime verification of an expected result (this necessarily

[Ada] Fix PR ada/77968

2016-10-14 Thread Eric Botcazou
As reported by Richard, the LTO-specific change I made on 07/11 can yield ICEs on simple testcases with -flto, so the attached patch simply backs it out and adds an appropriate LTO testcase, as well as a couple of other LTO testcases which currently fail with a bogus warning. Tested on

Re: [PATCH] Remove x86 pcommit instruction

2016-10-14 Thread Andrew Senkevich
> That said - why do we need to remove pcommit support on the branches at all? I think it is better to have old branches updated because pcommit will not be implemented in hardware. -- WBR, Andrew

[PATCH] S/390: Add static OSC breaker if necessary.

2016-10-14 Thread Andreas Krebbel
This patch adds a magic OSC (operand store compare) break instruction which is necessary if a store is followed closely by a load with same base+indx+displ while either base or index get modified in between. The patch improves several SpecCPU testcases running on IBM z13. I'll wait a few days

Re: [PATCH GCC]Simplify (convert)(X op const) -> (convert)X op (convert)const by match

2016-10-14 Thread Richard Biener
On Thu, Oct 13, 2016 at 5:22 PM, Jeff Law wrote: > On 10/12/2016 02:48 AM, Richard Biener wrote: >> >> On Tue, Oct 11, 2016 at 11:34 PM, Marc Glisse >> wrote: >>> >>> On Tue, 11 Oct 2016, Bin Cheng wrote: >>> We missed folding (convert)(X op const) ->

Re: [PATCH] Fix -Wimplicit-fallthrough ICE (PR c/77946)

2016-10-14 Thread Jakub Jelinek
On Fri, Oct 14, 2016 at 11:05:22AM +0200, Bernd Schmidt wrote: > On 10/13/2016 12:27 PM, Bernd Schmidt wrote: > >On 10/13/2016 12:20 PM, Jakub Jelinek wrote: > > > >>both relied on TREE_PUBLIC be actually false for LABEL_DECLs, because > >>otherwise they have code later on that can't handle

Re: [PATCH] Add "__RTL" to cc1 (v2)

2016-10-14 Thread Richard Biener
On Fri, Oct 14, 2016 at 11:48 AM, Bernd Schmidt wrote: > On 10/14/2016 11:33 AM, Richard Biener wrote: > >> Ok, so at least for the GIMPLE FE side I thought it's useful to allow >> a correctness verification with something simpler than pattern matching >> on the pass output.

Re: basic_block flags, BB_VISITED

2016-10-14 Thread Nathan Sidwell
On 10/14/16 05:28, Richard Biener wrote: The BB_VISITED flag has indetermined state at the beginning of a pass. You have to ensure it is cleared yourself. In that case the openacc (?) passes should be modified to clear the flags at their start, rather than at their end. nathan

Re: [Fortran, patch, caf] Add unimplemented message for polymorphic objects with allocatable/pointer components

2016-10-14 Thread Andre Vehreschild
Hi all, commited as r241150. Regards, Andre On Thu, 13 Oct 2016 17:45:08 +0200 Andre Vehreschild wrote: > Hi all, > > attached patch adds an unimplemented message, when a polymorphic coarray > object with allocatable/pointer components is declared for coarray mode >

Re: [Patch] include in

2016-10-14 Thread Jonathan Wakely
On 14/10/16 02:25 -0700, Tim Shen wrote: On some platforms the compiler complains about __try and __catch not defined. I hope is the right thing to include. Yes, that's right. OK for trunk, thanks. By the way, I'm intending to add __throw_bad_variant_access_fmt to bits/functexcept.h and make

[PATCH][ARM] Fix testism with pure-code tests for non Cortex-M targets

2016-10-14 Thread Andre Vieira (lists)
Hi there, This patch makes sure the pure-code tests only run for Cortex-M targets. I was hoping '![check_effective_target_arm_arm_ok]' would do the trick but it seems the options passed with RUNTESTFLAGS take precedence over the one in the check. Anyhow, I now use

Re: basic_block flags, BB_VISITED

2016-10-14 Thread Richard Biener
On Fri, Oct 14, 2016 at 10:01 AM, Thomas Schwinge wrote: > Hi! > > After the "Add Early VRP" GCC trunk commit r240291 (Kugan CC for your > information), I've been observing all kinds of OpenACC offloading > failures. I now figured out what's going on. > > The "evrp" pass

basic_block flags, BB_VISITED

2016-10-14 Thread Thomas Schwinge
Hi! After the "Add Early VRP" GCC trunk commit r240291 (Kugan CC for your information), I've been observing all kinds of OpenACC offloading failures. I now figured out what's going on. The "evrp" pass uses basic_block's BB_VISITED flag. It first clears these all,

Re: [PATCH] Fix PR77937

2016-10-14 Thread Richard Biener
On Thu, Oct 13, 2016 at 5:38 PM, Bill Schmidt wrote: > The previous patch for > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77937 is necessary, but not > sufficient in all cases. It allows -1 to be used with a pointer > increment, which we really do not want given

[Patch] include in

2016-10-14 Thread Tim Shen
On some platforms the compiler complains about __try and __catch not defined. I hope is the right thing to include. I'm bootstrapping the compiler under x86_64-linux-gnu, but it shouldn't fail. Thanks! -- Regards, Tim Shen a.diff Description: Binary data

Re: [PATCH] Add "__RTL" to cc1 (v2)

2016-10-14 Thread Richard Biener
On Thu, Oct 13, 2016 at 3:51 PM, Bernd Schmidt wrote: > On 10/13/2016 03:49 PM, Richard Biener wrote: >> >> Does it really run a single pass only? Thus you can't do a { dg-do run } >> test >> with __RTL? > > > I think that's really not the intended use-case. To my mind this

Re: [Patch] include in

2016-10-14 Thread Tim Shen
On Fri, Oct 14, 2016 at 2:31 AM, Jonathan Wakely wrote: > Yes, that's right. OK for trunk, thanks. Committed. > By the way, I'm intending to add __throw_bad_variant_access_fmt to > bits/functexcept.h and make it use __snprintf_lite() so that we can > do: > >

Re: [PATCH] Tree-level fix for PR 69526

2016-10-14 Thread Robin Dapp
Ping :)

Re: [PATCH][AArch64] Add spellchecking hints for -march,-mcpu,-mtune and their attributes

2016-10-14 Thread Kyrill Tkachov
On 11/10/16 11:40, James Greenhalgh wrote: On Fri, Oct 07, 2016 at 01:34:37PM -0700, Andrew Pinski wrote: On Fri, Oct 7, 2016 at 7:52 AM, Kyrill Tkachov wrote: Hi all, This patch uses the spellcheck API from David and Jakub [1] to implement hints for the march,

Re: [PATCH] Fix -Wimplicit-fallthrough ICE (PR c/77946)

2016-10-14 Thread Bernd Schmidt
On 10/13/2016 12:27 PM, Bernd Schmidt wrote: On 10/13/2016 12:20 PM, Jakub Jelinek wrote: both relied on TREE_PUBLIC be actually false for LABEL_DECLs, because otherwise they have code later on that can't handle LABE_DECLs (plus callers also not expecting LABEL_DECLs might not bind locally or

[PATCH] Simplify reference_wrapper nested typedefs

2016-10-14 Thread Jonathan Wakely
More refactoring. This uses the void_t "detection idiom" to test for the members, with fewer templates and fewer lines of code. There are now no more uses of _GLIBCXX_HAS_NESTED_TYPE, so we could consider removing it from . I find it more straightforward to use void_t directly. *

Re: basic_block flags, BB_VISITED

2016-10-14 Thread Bernd Schmidt
On 10/14/2016 10:01 AM, Thomas Schwinge wrote: After the "Add Early VRP" GCC trunk commit r240291 (Kugan CC for your information), I've been observing all kinds of OpenACC offloading failures. I now figured out what's going on. The "evrp" pass uses basic_block's BB_VISITED flag. It first

Re: basic_block flags, BB_VISITED

2016-10-14 Thread Richard Biener
On Fri, Oct 14, 2016 at 11:15 AM, Bernd Schmidt wrote: > On 10/14/2016 10:01 AM, Thomas Schwinge wrote: >> >> After the "Add Early VRP" GCC trunk commit r240291 (Kugan CC for your >> information), I've been observing all kinds of OpenACC offloading >> failures. I now figured

Re: [PATCH] Test cases for PR77937

2016-10-14 Thread Uros Bizjak
>> Here are torture test cases for >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77937. Markus Trippelsdorf >> kindly provided the source for the tests and verified the correct >> dejagnu options on x86_64-pc-linux-gnu. Committed. >> >> Thanks, >> Bill >> >> >> 2016-10-13 Bill Schmidt

Re: [PATCH] rs6000: Fix shrink-wrap-separate for AIX

2016-10-14 Thread Segher Boessenkool
On Sat, Oct 15, 2016 at 03:00:20AM +, Segher Boessenkool wrote: > 2016-10-15 Segher Boessenkool > > * config/rs6000/rs6000.c (rs6000_get_separate_components): Do not > make LR a separately shrink-wrapped component if savres_strategy > contains

Re: [PATCH] Don't peel extra copy of loop in unroller for loops with exit at end

2016-10-14 Thread Andrew Pinski
On Fri, Oct 14, 2016 at 8:28 PM, Andrew Pinski wrote: > On Thu, Sep 22, 2016 at 12:10 PM, Pat Haugen > wrote: >> I noticed the loop unroller peels an extra copy of the loop before it enters >> the switch block code to round the iteration count to

Re: [PATCH] Don't peel extra copy of loop in unroller for loops with exit at end

2016-10-14 Thread Andrew Pinski
On Thu, Sep 22, 2016 at 12:10 PM, Pat Haugen wrote: > I noticed the loop unroller peels an extra copy of the loop before it enters > the switch block code to round the iteration count to a multiple of the > unroll factor. This peeled copy is only needed for the case

Re: [PATCH AArch64]Penalize vector cost for large loops with too many vect insns.

2016-10-14 Thread kugan
Hi Bin, On 15/10/16 00:15, Bin Cheng wrote: +/* Test for likely overcommitment of vector hardware resources. If a + loop iteration is relatively large, and too large a percentage of + instructions in the loop are vectorized, the cost model may not + adequately reflect delays from

[PATCH] rs6000: Fix shrink-wrap-separate for AIX

2016-10-14 Thread Segher Boessenkool
All out-of-line register save routines need LR, so we cannot wrap the LR component if there are out-of-line saves. This didn't show up for testing on Linux because none of the tests there use out-of-line FPR saves without also using out-of-line GPR saves, which we did handle. This patch fixes

Re: [PATCH] Fold __builtin_str{n}{case}cmp functions (simplified version 4)

2016-10-14 Thread Richard Biener
On Thu, Oct 13, 2016 at 5:24 PM, Martin Liška wrote: > Simplified version that just supports only null-terminated strings. > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed? Ok. Richard. > Martin

[PATCH] PR65122 extended alignment support in allocators

2016-10-14 Thread Jonathan Wakely
Now that the warning in PR77742 is fixed we can make std::allocator use aligned new for types with extended alignment. This only changes the allocators that actually allocate memory themselves. The others (array, extptr, the testsuite ones) don't need any changes because they simply forward

Re: [PATCH, ARM 5/7, ping2] Adapt other atomic operations to ARMv8-M Baseline

2016-10-14 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 03/10/16 17:45, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:47, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch adds support

Re: [PATCH] Test cases for PR77937

2016-10-14 Thread Bernd Schmidt
On 10/14/2016 01:33 PM, Markus Trippelsdorf wrote: Thanks. ChangeLog entries for the testsuite are not mandatory. They are actually. Bernd

Re: [PATCH, ARM 2/7, ping2] Adapt atomic and exclusive load and store to ARMv8-M Baseline

2016-10-14 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 03/10/16 17:42, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:41, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch adapts

Re: [PATCH, ARM 3/7, ping] Refactor atomic compare_and_swap to make it fit for ARMv8-M Baseline

2016-10-14 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 03/10/16 17:44, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:44, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch refactors

Re: [PATCH, ARM 4/7, ping] Adapt atomic compare and swap to ARMv8-M Baseline

2016-10-14 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 03/10/16 17:45, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:46, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch makes the

Re: [PATCH] Test cases for PR77937

2016-10-14 Thread Markus Trippelsdorf
On 2016.10.14 at 13:28 +0200, Rainer Orth wrote: > Hi Markus, > > > On 2016.10.13 at 22:04 +0200, Rainer Orth wrote: > >> Hi Bill, > >> > >> > Here are torture test cases for > >> > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77937. Markus Trippelsdorf > >> > kindly provided the source for the

[PATCH] Do not move outer binds block for -finstrument-functions or TSAN

2016-10-14 Thread Richard Biener
I've encountered another case where the assumption that only a bind vars and its block vars may share a common chain -- for -finstrument-functions or TSAN the gimplifier violates this, making the chain shared between different binds vars and block. This can cause BLOCK_VARs to be dropped (I

Re: Set nonnull attribute to ptr_info_def based on VRP

2016-10-14 Thread Richard Biener
On Fri, Oct 14, 2016 at 1:12 AM, kugan wrote: > Hi Richard, > > > On 13/10/16 20:44, Richard Biener wrote: >> >> On Thu, Oct 13, 2016 at 6:49 AM, kugan >> wrote: >>> >>> Hi Richard, >>> what does this try to do?

Re: basic_block flags, BB_VISITED

2016-10-14 Thread Richard Biener
On Fri, Oct 14, 2016 at 12:57 PM, Bernd Schmidt wrote: > On 10/14/2016 11:26 AM, Richard Biener wrote: >> >> On Fri, Oct 14, 2016 at 11:15 AM, Bernd Schmidt >> wrote: >>> >>> So maybe it should just call clear_bb_flags instead of doing the loop >>>

Re: [PATCH] Do not merge BBs with a different EH landing pads (PR, tree-optimization/77943)

2016-10-14 Thread Martin Liška
On 10/13/2016 11:39 AM, Richard Biener wrote: > Yes, as said, the patch is ok as-is. > > Richard. Installed, I've just tested the patch on 5 and 6 branch. May I install it as well? M.

Re: [PATCH] Tree-level fix for PR 69526

2016-10-14 Thread Richard Biener
On Tue, Sep 20, 2016 at 2:31 PM, Robin Dapp wrote: > Hi, > >> I meant to do sth like >> >> Index: tree-ssa-propagate.c >> === >> --- tree-ssa-propagate.c(revision 240133) >> +++ tree-ssa-propagate.c

Re: Compile-time improvement for if conversion.

2016-10-14 Thread Yuri Rumyantsev
Richard, Here is updated patch with the changes proposed by you. Bootstrapping and regression testing did not show any new failures. Is it OK for trunk? ChangeLog: 2016-10-14 Yuri Rumyantsev * dominance.c (dom_info::dom_info): Add new constructor for region presented by

Re: [PATCH] Test cases for PR77937

2016-10-14 Thread Rainer Orth
Hi Markus, > On 2016.10.13 at 22:04 +0200, Rainer Orth wrote: >> Hi Bill, >> >> > Here are torture test cases for >> > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77937. Markus Trippelsdorf >> > kindly provided the source for the tests and verified the correct >> > dejagnu options on

[PATCH] Fix PR77979

2016-10-14 Thread Richard Biener
The following fixes PR77979, yet another case where the jump-threading code in VRP makes things difficult. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2016-10-14 Richard Biener PR tree-optimization/77979 * tree-vrp.c

Re: Move OVERRIDE/FINAL from gcc/coretypes.h to include/ansidecl.h

2016-10-14 Thread Pedro Alves
On 10/12/2016 03:13 PM, Bernd Schmidt wrote: > On 10/12/2016 04:09 PM, Pedro Alves wrote: >> >> Thanks. Here's a follow up patch that I was just testing. >> >> Need this if building with "g++ -std=gnu++11", with gcc < 4.7. > > Lovely. That's ok too if the other one goes in. FYI, I pushed these

Re: [PATCH] Test cases for PR77937

2016-10-14 Thread Markus Trippelsdorf
On 2016.10.13 at 22:04 +0200, Rainer Orth wrote: > Hi Bill, > > > Here are torture test cases for > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77937. Markus Trippelsdorf > > kindly provided the source for the tests and verified the correct > > dejagnu options on x86_64-pc-linux-gnu.

Re: [PATCH] Check \0-termination of string in c_getstr (simplified version)

2016-10-14 Thread Martin Liška
On 10/14/2016 11:38 AM, Richard Biener wrote: > On Thu, Oct 13, 2016 at 5:23 PM, Martin Liška wrote: >> Hello. >> >> After receiving feedback from Richi and Wilco Dijkstra, I decided to fully >> not >> support not null-terminated strings. It brings more complications and the >>

Re: [PATCH] Do not merge BBs with a different EH landing pads (PR, tree-optimization/77943)

2016-10-14 Thread Richard Biener
On Fri, Oct 14, 2016 at 1:12 PM, Martin Liška wrote: > On 10/13/2016 11:39 AM, Richard Biener wrote: >> Yes, as said, the patch is ok as-is. >> >> Richard. > > Installed, I've just tested the patch on 5 and 6 branch. > May I install it as well? Yes please. Ricahrd. > M.

Re: [PATCH] Test folding of str{n}{case}cmp and memchr (simplified version 4)

2016-10-14 Thread Richard Biener
On Thu, Oct 13, 2016 at 5:26 PM, Martin Liška wrote: > Simplified version of tests, where I added tests for side effects. > > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed? Ok. Richard. > Martin

Re: [patch,avr] Introduce absdata attribute to allow LDS /STS on Tiny

2016-10-14 Thread Denis Chertykov
2016-10-12 14:32 GMT+03:00 Georg-Johann Lay : > This patch introduces a new variable attribute "absdata". > > Reduced Tiny cores have only a limited range of addresses 0x40..0xbf which > can be handled by LDS / STS directly. The attribute allows the user to > assert that it is

Re: [PATCH] Fix PR77937

2016-10-14 Thread Bill Schmidt
Hi Richard, > On Oct 14, 2016, at 4:19 AM, Richard Biener > wrote: > > On Thu, Oct 13, 2016 at 5:38 PM, Bill Schmidt > wrote: >> The previous patch for >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77937 is necessary, but not >>

Re: [PATCH] Test cases for PR77937

2016-10-14 Thread Bill Schmidt
Markus, thanks for fixing this during my night. Appreciate it! Bill > On Oct 14, 2016, at 6:08 AM, Markus Trippelsdorf > wrote: > > On 2016.10.13 at 22:04 +0200, Rainer Orth wrote: >> Hi Bill, >> >>> Here are torture test cases for >>>

[v3] Remove more 'test' variables from the testcases

2016-10-14 Thread Paolo Carlini
Hi, the below, once again trivial, patch takes care of three additional directories. Tested x86_64-linux. Thanks, Paolo. /// 2016-10-14 Paolo Carlini * testsuite/26_numerics/accumulate/1.cc: Remove 'test' variables. *

Re: [PATCH v2] aarch64: Add split-stack initial support

2016-10-14 Thread Jiong Wang
Hi Adhemerval, On 06/10/16 22:54, Adhemerval Zanella wrote: + bool split_stack_arg_pointer_used = split_stack_arg_pointer_used_p (); if (flag_stack_usage_info) current_function_static_stack_size = frame_size; @@ -3220,6 +3264,10 @@ aarch64_expand_prologue (void)

Re: [PATCH, ARM 7/7, ping2] Enable ARMv8-M atomic and synchronization support for ARMv8-M Baseline

2016-10-14 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 03/10/16 17:46, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:50, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch enables

[Patch][MIPS} [Committed] Fix gcc.dg/pch.exp failures for microMIPS multilibs

2016-10-14 Thread Moore, Catherine
I've committed this patch to fix failures in g++.dg/pch.exp and gcc.dg/pch.exp tests when compiling with -mmicromips. Invalid data was being read from the precompiled-header causing a segfault while trying to switch compression context. This patch fixes that segfault by initializing

[PATCH] PR fortran/77978 -- STOP code fixes

2016-10-14 Thread Steve Kargl
The attach patch fixes a number of shortcomings with STOP codes in gfortran. The updated comment in the code nicely summarizes the problem. /* Match a number or character constant after an (ERROR) STOP or PAUSE - statement. */ + statement. The requirements for a stop-code differs in the

Go patch committed: copy runtime package time code from Go 1.7 runtime

2016-10-14 Thread Ian Lance Taylor
This patch to the Go frontend and libgo copies the runtime package time code from the Go 1.7 runtime. This tweaks the frontend to fix the handling of function values for -fgo-c-header to generate FuncVal*, not simply FuncVal. While we're here change runtime.nanotime to use clock_gettime with

Re: [PATCH] PR fortran/77978 -- STOP code fixes

2016-10-14 Thread Steve Kargl
On Fri, Oct 14, 2016 at 05:24:53PM -0700, Steve Kargl wrote: > For the code > > program foo > stop merge(667, 668, .true.) > end > > gfortran with either -std=f95 or -std=f2003 should reject > this code. My patch does not fix this issue, because it > would (1) require a complete

Re: [PATCH, libfortran] PR 48587 Newunit allocator

2016-10-14 Thread Bernhard Reutner-Fischer
On 14 October 2016 22:41:25 CEST, Janne Blomqvist wrote: >On Fri, Oct 14, 2016 at 8:01 PM, Bernhard Reutner-Fischer > wrote: >> On 13 October 2016 22:08:21 CEST, Jerry DeLisle > wrote: >>>On 10/13/2016 08:16 AM, Janne

Re: [ping * 2] remove optab functions for [us]divmod_optab in optabs.def

2016-10-14 Thread Prathamesh Kulkarni
On 13 October 2016 at 22:54, Bernd Schmidt wrote: > On 10/13/2016 07:18 PM, Prathamesh Kulkarni wrote: >> >> On 13 October 2016 at 16:56, Bernd Schmidt wrote: >>> >>> On 10/06/2016 07:43 AM, Prathamesh Kulkarni wrote: Pinging patch:

libgo patch committed: support SPARC64/ELF relocs

2016-10-14 Thread Ian Lance Taylor
This patch by James Clarke adds support for SPARC64/ELF relocs to the debug/elf package in libgo. This is a backport of https://golang.org/cl/30870 from the master library. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE

[PATCH] Add various DWARF5 constants to dwarf2.{h,def}

2016-10-14 Thread Jakub Jelinek
Hi! Now that DWARF5 public review draft has been released, I went through the document looking for double dagger marked constants and added them to dwarf2.{def,h}. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2016-10-14 Jakub Jelinek *

[PATCH] read-md.c: Move various state to within class rtx_reader

2016-10-14 Thread David Malcolm
On Wed, 2016-10-12 at 22:57 +0100, Richard Sandiford wrote: > Sorry, haven't had time to read the full series yet, but: > > David Malcolm writes: > > On Wed, 2016-10-05 at 17:51 +0200, Bernd Schmidt wrote: > > > On 10/05/2016 06:14 PM, David Malcolm wrote: > > > > The

libgo patch committed: just do flie/line lookup in C, move Func to Go

2016-10-14 Thread Ian Lance Taylor
In order to port stack backtraces to Go, we need the ability to look up file/line information for PC values without allocating memory. This libgo patch moves the handling of Func from C code to Go code, and simplifies the C code to just look up function/file/line/entry information for a PC.

[PATCH] Fix expansion ICE on store to CONST_DECL (PR middle-end/77959)

2016-10-14 Thread Jakub Jelinek
Hi! The following (invalid) testcase ICEs, because we try to store into CONST_DECL's FIELD. Normally in GIMPLE we have MEM_REF[] and writes to that expand gracefully into a MEM, but as soon as we use get_inner_reference in expand_assignment (even if the MEM is just reverse order, or we just want

[C++ PATCH] DR 1511 - const volatile variables and ODR

2016-10-14 Thread Jakub Jelinek
Hi! We weren't implementing this DR, in the past all non-extern const vars (and non-inline) at namespace scope had internal linkage, but now only non-volatile const var. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2016-10-14 Jakub Jelinek

[PATCH] Improve DWARF constant attribute langhooks

2016-10-14 Thread Jakub Jelinek
Hi! Before early dwarf changes, if we wanted to note some decl property so that some corresponding DWARF attribute can be emitted, we had to use some generic IL bit for that. Now a langhook can be used instead (hopefully for 7.x even with LTO), but having a single langhook for each such bit

[PATCH] Emit DW_AT_const_expr for constexpr variables

2016-10-14 Thread Jakub Jelinek
Hi! This relies on the previous langhook patch (which greatly simplifies it). I'm only handling variables for now, DW_AT_const_expr is just weird on functions/methods, it is supposed to appear only on DW_TAG_inlined_subroutine? Bootstrapped/regtested on x86_64-linux and i686-linux, ok for

[PATCH] Emit DW_AT_inline for C++17 inline variables

2016-10-14 Thread Jakub Jelinek
Hi! This also uses the infrastructure of the langhook patch I've sent earlier. It emits (if not strict dwarf) DW_AT_inline on explicit or implicit inline variables, and also tweaks dwarf2out so that for inline static data members we consider in-class declarations as definitions (emit

[PATCH] Avoid copies in std::scoped_allocator_adaptor piecewise construction

2016-10-14 Thread Jonathan Wakely
While looking at LWG 2511 I realised that we can prevent scoped_allocator_adaptor::construct(pair*, ...) from making any copies internally. The transformed tuples that get passed to the std::pair constructor (with additional allocator arguments) can be tuples of references even if the

[PATCH 5/8] nvptx mkoffload: pass -mgomp for OpenMP offloading

2016-10-14 Thread Alexander Monakov
This patch wires up use of alternative -mgomp multilib for OpenMP offloading via nvptx mkoffload. It makes OpenACC and OpenMP incompatible for simultaneous offloading compilation, so I've added a diagnostic for that. * config/nvptx/mkoffload.c (main): Check that either OpenACC or OpenMP

[PATCH 1/8] nvptx -msoft-stack

2016-10-14 Thread Alexander Monakov
This patch implements '-msoft-stack' code generation variant for NVPTX. The goal is to avoid relying on '.local' memory space for placement of automatic data, and instead have an explicitely-maintained stack pointer (which can be set up to point to preallocated global memory space). This allows

[PATCH 8/8] nvptx: handle OpenMP "omp target entrypoint"

2016-10-14 Thread Alexander Monakov
This patch implements emission of OpenMP target region entrypoints: the compiler emits the target function with '$impl' appended to the name, and under the original name it emits a short entry sequence that sets up shared memory arrays and calls the target function via 'gomp_nvptx_main' (which is

Re: [Patch 3/11] Implement TARGET_C_EXCESS_PRECISION for s390

2016-10-14 Thread James Greenhalgh
Hi, On Fri, Oct 07, 2016 at 10:34:25AM +0200, Andreas Krebbel wrote: > On 10/04/2016 03:42 PM, Joseph Myers wrote: > > On Tue, 4 Oct 2016, Andreas Krebbel wrote: > > > >>> (b) Handling EXCESS_PRECISION_TYPE_IMPLICIT like > >>> EXCESS_PRECISION_TYPE_FAST would accurately describe what the back

Re: [Patch 6/11] Migrate excess precision logic to use TARGET_EXCESS_PRECISION

2016-10-14 Thread James Greenhalgh
On Fri, Sep 30, 2016 at 05:32:01PM +, Joseph Myers wrote: > On Fri, 30 Sep 2016, James Greenhalgh wrote: > > >/* float.h needs to know this. */ > > + /* We already have the option -fno-fp-int-builtin-inexact to ensure > > + certain built-in functions follow TS 18661-1 semantics. It

[PATCH 0/8] NVPTX offloading to NVPTX: backend patches

2016-10-14 Thread Alexander Monakov
Hi, I'm resending the patch series with backend prerequisites for OpenMP offloading to the NVIDIA PTX ISA. The patches are rebased on trunk. Could a global reviewer have a look at patch 6 (new TARGET_SIMT_VF hook) please? Documentation changes in doc/invoke.texi have already been reviewed by

[PATCH 2/8] nvptx: implement predicated instructions

2016-10-14 Thread Alexander Monakov
This patch wires up generation of predicated instruction forms in nvptx.md and fixes their handling in nvptx.c. This is a prerequisite for the following patch. On its own it doesn't affect generated code because COND_EXEC instructions are created by if-conversion only after register allocation,

Re: [Patch 4/11] Implement TARGET_C_EXCESS_PRECISION for m68k

2016-10-14 Thread James Greenhalgh
On Fri, Sep 30, 2016 at 11:28:28AM -0600, Jeff Law wrote: > On 09/30/2016 11:01 AM, James Greenhalgh wrote: > > > >Hi, > > > >This patch ports the logic from m68k's TARGET_FLT_EVAL_METHOD to the new > >target hook TARGET_C_EXCESS_PRECISION. > > > >Patch tested by building an m68k-none-elf

Re: [PATCH, libfortran] PR 48587 Newunit allocator

2016-10-14 Thread Bernhard Reutner-Fischer
On 13 October 2016 22:08:21 CEST, Jerry DeLisle wrote: >On 10/13/2016 08:16 AM, Janne Blomqvist wrote: >> >> Regtested on x86_64-pc-linux-gnu. Ok for trunk? >> > >Yes, OK, clever! Thanks! Is 32 something a typical program uses? I'd have started at 8 and had not doubled

[PATCH 3/8] nvptx -muniform-simt

2016-10-14 Thread Alexander Monakov
This patch implements -muniform-simt code generation option, which is used to emit code for OpenMP offloading. The goal is to emit code that can either execute "normally", or can execute in a way that keeps all lanes in a given warp active, their local state synchronized, and observable effects

[PATCH 4/8] nvptx -mgomp

2016-10-14 Thread Alexander Monakov
This patch adds option -mgomp which enables -msoft-stack plus -muniform-simt, and wires up the corresponding multilib variant. This codegen convention is used for OpenMP offloading. * config/nvptx/nvptx.c (diagnose_openacc_conflict): New. Use it... (nvptx_option_override):

[PATCH 7/8] nvptx backend: new insns for OpenMP SIMD-via-SIMT

2016-10-14 Thread Alexander Monakov
This patch implements in nvptx.md a few new instruction patterns that are used for OpenMP SIMD code. * config/nvptx/nvptx-protos.h (nvptx_shuffle_kind): Move enum declaration from nvptx.c. (nvptx_gen_shuffle): Declare. * config/nvptx/nvptx.c (nvptx_shuffle_kind):

[PATCH 6/8] new target hook: TARGET_SIMT_VF

2016-10-14 Thread Alexander Monakov
This patch adds a new target hook and implements it in a straightforward manner on NVPTX to indicate that the target is running in SIMT fashion with 32 threads in a synchronous group ("warp"). For use in OpenMP transforms. * config/nvptx/nvptx.c (nvptx_simt_vf): New.

  1   2   >