Re: [PATCH, rs6000] Fix PR72827 (ada bootstrap failure)

2016-08-31 Thread Eric Botcazou
> DSE should really detect this is happening and not do the wrong thing. > Maybe add an assert somewhere? Much easier to debug, that way. That sounds fragile, functions are allowed to fiddle with the frame pointer in the prologue or epilogue (but of course not in the body). I think that DSE is

[ARM] Fix RTL checking failure in Thumb mode

2016-08-31 Thread Eric Botcazou
Hi, compiling any non-trivial program in Thumb mode yields with RTL checking: eric@arcturus:~/build/gcc/arm-eabi> gcc/xgcc -Bgcc -S t.c -mthumb t.c:4:5: internal compiler error: RTL check: expected code 'const_int', have 'reg' in thumb1_size_rtx_costs, at config/arm/arm.c:9130 int foo (int a, i

Re: [patch, libgfortran] PR77393 [7 Regression] Revision r237735 changed the behavior of F0.0

2016-08-31 Thread Janne Blomqvist
On Wed, Aug 31, 2016 at 6:04 AM, Jerry DeLisle wrote: > Hi all, > > The attached patch fixes the problem by adding a new helper function to > determine the buffer size needed for F0 editing depending on the kind. In this > new function there are some constants presented which document the limits

Re: PR35503 - warn for restrict pointer

2016-08-31 Thread Tom de Vries
On 30/08/16 16:54, Tom de Vries wrote: On 26/08/16 13:39, Prathamesh Kulkarni wrote: Hi, The following patch adds option -Wrestrict that warns when an argument is passed to a restrict qualified parameter and it aliases with another argument. eg: int foo (const char *__restrict buf, const char *

Re: [ARM] Fix RTL checking failure in Thumb mode

2016-08-31 Thread Kyrill Tkachov
On 31/08/16 08:31, Eric Botcazou wrote: Hi, compiling any non-trivial program in Thumb mode yields with RTL checking: eric@arcturus:~/build/gcc/arm-eabi> gcc/xgcc -Bgcc -S t.c -mthumb t.c:4:5: internal compiler error: RTL check: expected code 'const_int', have 'reg' in thumb1_size_rtx_costs, a

libgo/runtime: Fix signal stack size for ia64

2016-08-31 Thread Andreas Schwab
On ia64, MINSIGSTKSZ is 128K. Andreas. * libgo/runtime/runtime.c (runtime_mpreinit): Increase stack size to 128K. --- libgo/runtime/runtime.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libgo/runtime/runtime.c b/libgo/runtime/runtime.c index c7d33bc..e8eb9

Re: [PATCH] selftest.c: avoid explicit "selftest::" qualifiers

2016-08-31 Thread Bernd Schmidt
On 08/31/2016 02:31 AM, David Malcolm wrote: The following follow-up patch removes them, moving the "namespace selftest {" to the top of the file so it covers everything. LGTM. Bernd

Re: [PATCH, rs6000] Fix PR72827 (ada bootstrap failure)

2016-08-31 Thread Segher Boessenkool
On Wed, Aug 31, 2016 at 09:08:37AM +0200, Eric Botcazou wrote: > > DSE should really detect this is happening and not do the wrong thing. > > Maybe add an assert somewhere? Much easier to debug, that way. > > That sounds fragile, functions are allowed to fiddle with the frame pointer > in > the

Re: Implement -Wimplicit-fallthrough (version 7)

2016-08-31 Thread Marek Polacek
On Tue, Aug 30, 2016 at 01:08:22PM -0400, Jason Merrill wrote: > On Tue, Aug 30, 2016 at 10:25 AM, Marek Polacek wrote: > >> > @@ -10585,15 +10610,26 @@ cp_parser_statement (cp_parser* parser, tree > >> > in_statement_expr, > >> > } > >> >/* Look for an expression-statement instea

Re: [PATCH] C: fixits for modernizing structure member designators

2016-08-31 Thread Marek Polacek
On Tue, Aug 30, 2016 at 11:22:21AM -0400, David Malcolm wrote: > On Tue, 2016-08-30 at 16:50 +0200, Marek Polacek wrote: > > On Tue, Aug 30, 2016 at 04:40:57PM +0200, Bernd Schmidt wrote: > > > On 08/30/2016 04:38 PM, David Malcolm wrote: > > > > > > > In conjunction with the not-yet-in-trunk -fdi

Re: [MPX] Fix for PR77267

2016-08-31 Thread Ilya Enkovich
2016-08-30 21:53 GMT+03:00 Alexander Ivchenko : > Would something like that count? > > I did not do the warning thing, cause the problem only appears when > you provide the -Wl,-as-needed option to the linker. > (In all other cases warning would be redundant). Are we able to check > that on runtime

[patch] Fix PR fortran/72743

2016-08-31 Thread Chung-Lin Tang
Hi Richard, Martin, this issue is actually sort of like PR 70856, basically the same ICE after IPA-ICF, due to DECL_PT_UIDs not consistent after reaching for the ultimate_alias_target(). The reason this wasn't covered by the PR70856 fix is that, in this case, the DECL_PT_UID was not set in origina

Re: [ARM][PR target/77281] Fix an invalid check for vectors of, the same floating-point constants.

2016-08-31 Thread Richard Earnshaw (lists)
On 30/08/16 14:49, Matthew Wahab wrote: > Ping. > > On 19/08/16 15:47, Richard Earnshaw (lists) wrote: >> On 19/08/16 15:06, Matthew Wahab wrote: >>> On 19/08/16 14:30, Richard Earnshaw (lists) wrote: On 19/08/16 12:48, Matthew Wahab wrote: > 2016-08-19 Matthew Wahab > >

Re: [PATCH] Fix detection of setrlimit in libstdc++ testsuite

2016-08-31 Thread Maxim Kuvyrkov
> On Apr 5, 2016, at 2:20 PM, Jonathan Wakely wrote: > >> This patch fixes an obscure cross-testing problem that crashed (OOMed) our >> boards at Linaro. Several tests in libstdc++ (e.g., [1]) limit themselves >> to some reasonable amount of RAM and then try to allocate 32 gigs. >> Unfortuna

[PATCH, PING] DWARF: space-optimize loc. descr. for integer literals on 32-bit targets

2016-08-31 Thread Pierre-Marie de Rodat
Hello, Ping for the patch submitted at . Also, here’s the update that Trevor suggested. Thanks! -- Pierre-Marie de Rodat >From da6753ad98127778e8923db63ce51b52a853ed17 Mon Sep 17 00:00:00 2001 From: Pierre-Marie de Rodat Date: Fri, 2

Re: [patch, libstdc++] std::shuffle: Generate two swap positions at a time if possible

2016-08-31 Thread Jonathan Wakely
On 03/05/16 16:42 +0200, Eelis van der Weegen wrote: Ah, thanks, I forgot to re-attach when I sent to include the libstdc++ list. On 2016-05-03 14:38, Jonathan Wakely wrote: ENOPATCH On 1 May 2016 at 15:21, Eelis wrote: Sorry, forgot to include the libstdc++ list. On 2016-05-01 16:18, Eelis

Re: [PATCH] Fix detection of setrlimit in libstdc++ testsuite

2016-08-31 Thread Jonathan Wakely
On 31/08/16 14:22 +0300, Maxim Kuvyrkov wrote: On Apr 5, 2016, at 2:20 PM, Jonathan Wakely wrote: This patch fixes an obscure cross-testing problem that crashed (OOMed) our boards at Linaro. Several tests in libstdc++ (e.g., [1]) limit themselves to some reasonable amount of RAM and then tr

Re: [MPX] Fix for PR77267

2016-08-31 Thread Alexander Ivchenko
2016-08-31 12:18 GMT+03:00 Ilya Enkovich : > 2016-08-30 21:53 GMT+03:00 Alexander Ivchenko : >> Would something like that count? >> >> I did not do the warning thing, cause the problem only appears when >> you provide the -Wl,-as-needed option to the linker. >> (In all other cases warning would be

Re: [PATCH] C: fixits for modernizing structure member designators

2016-08-31 Thread David Malcolm
On Wed, 2016-08-31 at 11:09 +0200, Marek Polacek wrote: > On Tue, Aug 30, 2016 at 11:22:21AM -0400, David Malcolm wrote: > > On Tue, 2016-08-30 at 16:50 +0200, Marek Polacek wrote: > > > On Tue, Aug 30, 2016 at 04:40:57PM +0200, Bernd Schmidt wrote: > > > > On 08/30/2016 04:38 PM, David Malcolm wro

Re: libgo/runtime: Fix signal stack size for ia64

2016-08-31 Thread Ian Lance Taylor
On Wed, Aug 31, 2016 at 1:24 AM, Andreas Schwab wrote: > On ia64, MINSIGSTKSZ is 128K. > > Andreas. > > * libgo/runtime/runtime.c (runtime_mpreinit): Increase stack size to > 128K. > --- > libgo/runtime/runtime.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --g

Fwd: libgo/runtime: Fix signal stack size for ia64

2016-08-31 Thread Ian Lance Taylor
Forgot to send to gofrontend-dev. Ian -- Forwarded message -- From: Ian Lance Taylor Date: Wed, Aug 31, 2016 at 6:59 AM Subject: Re: libgo/runtime: Fix signal stack size for ia64 To: Andreas Schwab Cc: gcc-patches On Wed, Aug 31, 2016 at 1:24 AM, Andreas Schwab wrote: > On

Re: [PATCH, rs6000] Fix PR72827 (ada bootstrap failure)

2016-08-31 Thread Bill Schmidt
On 8/31/16 1:19 AM, Segher Boessenkool wrote: > Hi Bill, > > On Tue, Aug 30, 2016 at 08:23:46PM -0500, Bill Schmidt wrote: >> The ada bootstrap failure reported in >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72827 >> occurs because of a latent bug in the powerpc back end. The immediate cause

Re: [PATCH 2/4] Improvements to typed_splay_tree

2016-08-31 Thread Bernd Schmidt
On 08/24/2016 03:28 AM, David Malcolm wrote: +/* Helper type for typed_splay_tree::foreach. */ + +template +struct closure Is this in the global namespace? If so, something more specific than "closure" might be more appropriate. Or move the struct into the typed_splay_tree class definition.

Re: libgo/runtime: Fix signal stack size for ia64

2016-08-31 Thread Andreas Schwab
On Aug 31 2016, Ian Lance Taylor wrote: > Go really work on ia64?). http://gcc.gnu.org/ml/gcc-testresults/2016-08/msg03154.html Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 "And now for something completely diff

Re: [PATCH] Fixes accidental renaming of gdb.py file (i.e. libstdc++.so.6.0.22-gdb.py)

2016-08-31 Thread Gerald Pfeifer
Michael, I noticed you did not copy the libstc++ list, so let me do this for you. (If any further ping is necessary, please include that list.) Jonathan, if you approve, Michael does not have commit access. Gerald On Thu, 16 Jul 2015, Michael Darling wrote: > Ping. I don't have write access.

Re: [PATCH] Fixes accidental renaming of gdb.py file (i.e. libstdc++.so.6.0.22-gdb.py)

2016-08-31 Thread Jonathan Wakely
On 31/08/16 16:52 +0200, Gerald Pfeifer wrote: Michael, I noticed you did not copy the libstc++ list, so let me do this for you. (If any further ping is necessary, please include that list.) Thanks, Gerald. Jonathan, if you approve, Michael does not have commit access. I found the problem

[PATCH] Fix some warnings/errors that appear when enabling -Wnarrowing when building gcc

2016-08-31 Thread Eric Gallager
In https://gcc.gnu.org/ml/gcc-patches/2016-08/msg01526.html I tried enabling -Wnarrowing when building GCC and produced a log of the resulting (uniq-ed) warnings/errors. The attached patch here fixes some of them by using the 'U' suffix to make certain constants unsigned so they don't become negati

Re: [PATCH, rs6000] Fix PR72827 (ada bootstrap failure)

2016-08-31 Thread Eric Botcazou
> I'm not sure I'm the right person to do that, as I don't really have any > familiarity with the DSE code. I can't even prove to myself that this code > is alloca-safe; it doesn't look like it. No FUD, please ;-) The code is alloca-safe, alloca is about moving the stack pointer, not the frame

[ARM] Fix broken sibcall with longcall, APCS frame and VFP

2016-08-31 Thread Eric Botcazou
Hi, this is a regression present on the mainline and 6 branch and introduced by: 2014-04-25 Jiong Wang * config/arm/predicates.md (call_insn_operand): Add long_call check. * config/arm/arm.md (sibcall, sibcall_value): Force the address to reg for long_call. * c

Re: [ARM] Fix broken sibcall with longcall, APCS frame and VFP

2016-08-31 Thread Richard Earnshaw (lists)
On 31/08/16 16:35, Eric Botcazou wrote: > Hi, > > this is a regression present on the mainline and 6 branch and introduced by: > > 2014-04-25 Jiong Wang > > * config/arm/predicates.md (call_insn_operand): Add long_call check. > * config/arm/arm.md (sibcall, sibcall_value): Force t

[x86] Disable STV pass if -mstackrealign is enabled.

2016-08-31 Thread Eric Botcazou
Hi, the new STV pass generates SSE instructions in 32-bit mode very late in the pipeline and doesn't bother about realigning the stack, so it wreaks havoc on OSes where you need to realign the stack, e.g. Windows, but I guess Solaris is equally affected. Therefore the attached patch disables i

Re: PR35503 - warn for restrict pointer (-Wrestrict)

2016-08-31 Thread Prathamesh Kulkarni
On 31 August 2016 at 03:45, Mike Stump wrote: > On Aug 30, 2016, at 4:57 AM, Prathamesh Kulkarni > wrote: >> >> On 30 August 2016 at 17:11, Eric Gallager wrote: >>> On 8/29/16, Jason Merrill wrote: On Mon, Aug 29, 2016 at 10:28 AM, Marek Polacek wrote: > On Mon, Aug 29, 2016 at 09:20

Re: PR35503 - warn for restrict pointer

2016-08-31 Thread Prathamesh Kulkarni
On 30 August 2016 at 18:49, David Malcolm wrote: > On Tue, 2016-08-30 at 17:08 +0530, Prathamesh Kulkarni wrote: >> On 30 August 2016 at 05:34, David Malcolm >> wrote: >> > On Mon, 2016-08-29 at 20:01 -0400, David Malcolm wrote: >> > > On Mon, 2016-08-29 at 19:55 -0400, David Malcolm wrote: >> >

Re: Fix bogus warning with -Wlogical-not-parentheses (PR c/77292)

2016-08-31 Thread Martin Sebor
On 08/26/2016 07:44 AM, Marek Polacek wrote: On Thu, Aug 25, 2016 at 11:35:53AM -0600, Martin Sebor wrote: +foo (int a, int b) +{ + int r = 0; + r += !a == (a < b); + r += !a == (a > b); + r += !a == (a >= b); + r += !a == (a <= b); + r += !a == (a != b); + r += !a == (a == b); + r += !a

Re: [PATCH, rs6000] Fix PR72827 (ada bootstrap failure)

2016-08-31 Thread Bill Schmidt
> On Aug 31, 2016, at 10:31 AM, Eric Botcazou wrote: > >> I'm not sure I'm the right person to do that, as I don't really have any >> familiarity with the DSE code. I can't even prove to myself that this code >> is alloca-safe; it doesn't look like it. > > No FUD, please ;-) The code is alloc

Re: [PATCH] Fixups for Martin's gimple-ssa-sprintf.c patch

2016-08-31 Thread David Malcolm
On Wed, 2016-08-31 at 10:23 -0600, Martin Sebor wrote: > On 08/25/2016 10:30 AM, Martin Sebor wrote: > > On 08/25/2016 10:23 AM, David Malcolm wrote: > > > Martin: here are the fixups for your patch I needed to apply to > > > make > > > it work with mine. I couldn't actually get any of your existi

Re: [PATCH] Fixups for Martin's gimple-ssa-sprintf.c patch

2016-08-31 Thread Martin Sebor
On 08/25/2016 10:30 AM, Martin Sebor wrote: On 08/25/2016 10:23 AM, David Malcolm wrote: Martin: here are the fixups for your patch I needed to apply to make it work with mine. I couldn't actually get any of your existing test cases to emit locations within the string literals, due to them all

Re: [v3 PATCH, RFC] Implement LWG 2729 for tuple

2016-08-31 Thread Ville Voutilainen
On 31 August 2016 at 18:40, Ville Voutilainen wrote: > Now tested with the full testsuite on Linux-PPC64, test in the patch > amended slightly. > New patch attached. I added some more torture to the new test and re-indented it. diff --git a/libstdc++-v3/include/std/tuple b/libstdc++-v3/include/st

Re: [PATCH] Fixups for Martin's gimple-ssa-sprintf.c patch

2016-08-31 Thread Martin Sebor
On 08/31/2016 10:26 AM, David Malcolm wrote: On Wed, 2016-08-31 at 10:23 -0600, Martin Sebor wrote: On 08/25/2016 10:30 AM, Martin Sebor wrote: On 08/25/2016 10:23 AM, David Malcolm wrote: Martin: here are the fixups for your patch I needed to apply to make it work with mine. I couldn't actua

Re: [PATCH] Define feature-test macro for std::enable_shared_from_this

2016-08-31 Thread Jonathan Wakely
On 04/08/16 13:33 +0100, Jonathan Wakely wrote: On 03/08/16 20:11 +0100, Jonathan Wakely wrote: Another feature we already support, so just define the macro. * include/bits/shared_ptr_base.h (__cpp_lib_enable_shared_from_this): Define feature-test macro. * testsuite/20_u

Re: Implement C _FloatN, _FloatNx types [version 6]

2016-08-31 Thread James Greenhalgh
On Fri, Aug 19, 2016 at 04:23:55PM +, Joseph Myers wrote: > On Fri, 19 Aug 2016, Szabolcs Nagy wrote: > > > On 17/08/16 21:17, Joseph Myers wrote: > > > Although there is HFmode support for ARM and AArch64, use of that for > > > _Float16 is not enabled. Supporting _Float16 would require addit

Re: [PATCH] libstdc++/77334 move assign RB trees of non-copyable types

2016-08-31 Thread Jonathan Wakely
On 30/08/16 11:17 +0100, Jonathan Wakely wrote: On 27/08/16 21:29 +0200, François Dumont wrote: On 23/08/2016 15:17, Jonathan Wakely wrote: This fixes move assignment of RB trees to tag dispatch so the branch that copies nodes doesn't need to be well-formed. PR libstdc++/77334 * include/bi

[PATCH] Constrain std::shared_ptr assignment and resetting

2016-08-31 Thread Jonathan Wakely
The standard subtly requires us to constrain the assignment operators, due to "Effects: Equivalent to [a constrained expression]". Constraining reset() is not required, but a private discussion started by Alisdair Meredith has convinced me that it's useful (and apparently libc++ already does this

[gomp4] vector reductions

2016-08-31 Thread Nathan Sidwell
This patch changes the implementation of vector reductions. Currently we emit a sequence of shuffle/op pairs. The size of the sequence depends on the vector size. This changes the implementation to emit a loop, the number of iterations of which depends on the vector size. the goal here is a

Re: Implement C _FloatN, _FloatNx types [version 6]

2016-08-31 Thread Joseph Myers
On Wed, 31 Aug 2016, James Greenhalgh wrote: > My concern with this is that the use of comparisons of FLT_EVAL_METHOD > against 0 that Szabolcs is referring to is common and can have performance > implications. In glibc for example, > > static FLOAT > overflow_value (int negative) > { >

match.pd: Revert a * (1 << b) relaxation

2016-08-31 Thread Marc Glisse
Hello, this patch was bootstrapped and regtested on powerpc64le-unknown-linux-gnu. Already committed, since this is simply reverting part of my patch. -- Marc GlisseIndex: gcc/ChangeLog === --- gcc/ChangeLog (revision 239901) +++

[patch] Fix segfault in param_change_prob

2016-08-31 Thread Eric Botcazou
Hi, the attached Ada testcase triggers a segfault in param_change_prob because the parameter is VIEW_CONVERT_EXPR (or * ipa-inline-analysis.c (param_change_prob): Strip VIEW_CONVERT_EXPR. 2016-08-31 Eric Botcazou * gnat.dg/opt58.adb: New test. * gnat.dg/opt58_pkg.a

[committed] Fix ICE with !$omp atomic (PR fortran/77374)

2016-08-31 Thread Jakub Jelinek
Hi! The resolve_omp_atomic code relied on gfc_resolve_blocks not actually changing the kinds and number of statements, which is apparently no longer the case, there are various changes where those can change, e.g. after diagnosing an error EXEC_ASSIGN can be changed into EXEC_NOP, or for the F08 f

[committed] Fix ICE with !$omp parallel workshare (PR fortran/77352)

2016-08-31 Thread Jakub Jelinek
Hi! This is something I've fixed recently for a couple of construct, but left parallel workshare untouched. Bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk. 2016-08-31 Jakub Jelinek PR fortran/77352 * trans-openmp.c (gfc_trans_omp_parallel_workshare)

Re: [doc] doc/install.texi: www.opencsw.org now uses https.

2016-08-31 Thread Gerald Pfeifer
On Mon, 22 Aug 2016, Gerald Pfeifer wrote: > 2016-08-22 Gerald Pfeifer > > * doc/install.texi (Binaries): www.opencsw.org now uses https. > > Applied. > > I'll backport to GCC 6 as well, once Richi has unfrozen the branch. Done. Gerald

[committed] diagnostic-show-locus.c: handle fixits on lines outside the regular ranges

2016-08-31 Thread David Malcolm
The diagnostic_show_locus implementation determines the set of line spans that need printing based on the ranges within the rich_location (in layout::calculate_line_spans). Currently this doesn't take into account fix-it hints, and hence we fail to print fix-it hints that are on lines outside of t

[PATCH] Fix up -fsanitize=address ctor order (PR sanitizer/77396)

2016-08-31 Thread Jakub Jelinek
Hi! libasan currently has an assertion that __asan_before_dynamic_init is called only after registering at least one global var. I have no idea how to always guarantee that without making the code too ugly or registering dummy global vars, since the registration of globals is done at the end of f

Re: [x86] Disable STV pass if -mstackrealign is enabled.

2016-08-31 Thread Uros Bizjak
> the new STV pass generates SSE instructions in 32-bit mode very late in the > pipeline and doesn't bother about realigning the stack, so it wreaks havoc on > OSes where you need to realign the stack, e.g. Windows, but I guess Solaris is > equally affected. Therefore the attached patch disables i

[PATCH, AVX-512, committed] Fix detection of AVX512IFMA in host_detect_local_cpu

2016-08-31 Thread Ilya Verbin
Hi! I've committed this patch as obvious. gcc/ * config/i386/driver-i386.c (host_detect_local_cpu): Fix detection of AVX512IFMA. Index: gcc/config/i386/driver-i386.c === --- gcc/config/i386/driver-i386.c (revision 239907) +++ gcc/

Re: [PATCH, rs6000] Fix PR72827 (ada bootstrap failure)

2016-08-31 Thread Bill Schmidt
> On Aug 31, 2016, at 9:00 AM, Bill Schmidt wrote: > > > On 8/31/16 1:19 AM, Segher Boessenkool wrote: >>> >> If (say) base=r1 offset=r0 this will now adjust r1? That cannot be good. > Mm, yeah, that wasn't well-thought. Was thinking 0, not r0. Will have > to avoid > that. Unfortunately th

Re: PR35503 - warn for restrict pointer

2016-08-31 Thread Fabien Chêne
2016-08-30 17:34 GMT+02:00 Prathamesh Kulkarni : > On 30 August 2016 at 20:24, Tom de Vries wrote: >> On 26/08/16 13:39, Prathamesh Kulkarni wrote: >>> >>> Hi, >>> The following patch adds option -Wrestrict that warns when an argument >>> is passed to a restrict qualified parameter and it aliases

[PATCH] PR target/77408: Add -mcopy-reloc to avoid copy relocation

2016-08-31 Thread H.J. Lu
On x86, copy relocation is used in executable to access external data defined in shared object as if it is defined locally. At run-time, dynamic linker copies symbol data from shared object to executable and its references from shared objects are resolved by GLOB_DAT relocation. Since the copy of s

Re: [MPX] Fix for PR77267

2016-08-31 Thread Ilya Enkovich
2016-08-31 16:37 GMT+03:00 Alexander Ivchenko : > 2016-08-31 12:18 GMT+03:00 Ilya Enkovich : >> 2016-08-30 21:53 GMT+03:00 Alexander Ivchenko : >>> Would something like that count? >>> >>> I did not do the warning thing, cause the problem only appears when >>> you provide the -Wl,-as-needed option

Re: [PING] Use correct location information for OpenACC shape and simple clauses in C/C++

2016-08-31 Thread Gerald Pfeifer
On Thu, 4 Aug 2016, David Malcolm wrote: > On Thu, 2016-08-04 at 16:54 +0200, Thomas Schwinge wrote: >> I think in your position as a maintainer for "diagnostic messages", >> you should be qualified to exercise that status to approve such a >> patch. :-) > I don't know exactly where the boundarie

[PATCH] PR fortran/77406

2016-08-31 Thread Steve Kargl
Consider the code interface s subroutine foo(*) end subroutine foo subroutine bar(*) end subroutine bar end interface s end gfortran currently ICE's, because she is ill-prepared to deal with alternate returns in an interface blocks. The attached patch fixes this p

[PATCH] Improvements to typed_splay_tree (v2)

2016-08-31 Thread David Malcolm
On Wed, 2016-08-31 at 16:12 +0200, Bernd Schmidt wrote: > On 08/24/2016 03:28 AM, David Malcolm wrote: > > +/* Helper type for typed_splay_tree::foreach. */ > > + > > +template > > +struct closure > > Is this in the global namespace? If so, something more specific than > "closure" might be more

[PATCH 1/3] rs6000: Fix for AIX, for r239866

2016-08-31 Thread Segher Boessenkool
This should fix r239866 for AIX. I missed two patterns that refer to LR as "register_operand" "l" instead of as reg:P LR_REGNO. David, could you please test if this fixes the problem for you? Bootstrapped and regression checked on powerpc64-linux -m32,-m64, for what that is worth :-) Segher

[PATCH 2/3] rs6000: Use LR_REGNO instead of constant 65

2016-08-31 Thread Segher Boessenkool
Many places still use 65 instead of the symbolic constant LR_REGNO. This fixes them all (I looked for the string "65" only, in config/rs6000/ only, I didn't read all code :-) ) I left it in *restore_world because Iain will remove it there soon. Bootstrapped and regression checked on powerpc64-li

[committed] C: Fix missing spaces in 'struct' fix-it hints

2016-08-31 Thread David Malcolm
In r237714 I added fix-it hints to the C frontend for missing "struct" keywords e.g.: spellcheck-typenames.c:69:1: error: unknown type name ‘foo_t’; use ‘struct’ keyword to refer to the type foo_t *foo_ptr; ^ struct However when using the (not yet in trunk) option -fdiagnostics-generate-

[PATCH 3/3] rs6000: Use LR_REGNO directly in the save/restore patterns

2016-08-31 Thread Segher Boessenkool
Various patterns use "register_operand" "l" (or "=l") although those patterns are only created refering to LR_REGNO directly. This patch changes those patterns to use the hard regs as well. Bootstrapped and regression checked on powerpc64-linux -m32,-m64. Segher 2016-08-31 Segher Boessenkool

[PATCH v2, rs6000] Fix PR72827 (ada bootstrap failure)

2016-08-31 Thread Bill Schmidt
Hi, After much discussion we've concluded that the first patch isn't salvageable. There are register assignments for which we can't fix up the addressing legally after reload using such tricks (for example: base=r31, offset=r0). This patch (suggested by Michael Meissner) instead prevents the pro

Re: PR35503 - warn for restrict pointer

2016-08-31 Thread Richard Biener
On Tue, 30 Aug 2016, Tom de Vries wrote: > On 30/08/16 17:34, Prathamesh Kulkarni wrote: > > On 30 August 2016 at 20:24, Tom de Vries wrote: > > > On 26/08/16 13:39, Prathamesh Kulkarni wrote: > > > > > > > > Hi, > > > > The following patch adds option -Wrestrict that warns when an argument > >