Re: [v3 PATCH] Make sure the return value of malloc_allocator::allocate is zero-initialized so that checking it for non-zero works later.

2016-10-16 Thread Jonathan Wakely
On 15/10/16 23:44 +0300, Ville Voutilainen wrote: On 15 October 2016 at 22:44, Ville Voutilainen wrote: 2016-10-15 Ville Voutilainen Make sure the return value of malloc_allocator::allocate is zero-initialized so that

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

2016-10-16 Thread David Edelsohn
On Sat, Oct 15, 2016 at 2:16 PM, Segher Boessenkool wrote: > On Sat, Oct 15, 2016 at 07:55:47AM -0400, David Edelsohn wrote: >> Maybe rs6000 always should prefer inline save-restore when SWS is >> enabled, except for optimize_size? > > Yes, that would be a good

Re: [Ada] Set Always_Compatible_Rep to False everywhere

2016-10-16 Thread Andreas Schwab
On Okt 16 2016, Eric Botcazou wrote: > * system-aix.ads (Always_Compatible_Rep): Change to False. > * system-aix64.ads (Always_Compatible_Rep): Likewise. > * system-hpux-ia64.ads (Always_Compatible_Rep): Likewise. > * system-hpux.ads

[Ada] Set Always_Compatible_Rep to False everywhere

2016-10-16 Thread Eric Botcazou
It's not clear why this was set to True in some configuration files. Applied on the mainline. 2016-10-16 Eric Botcazou * system-aix.ads (Always_Compatible_Rep): Change to False. * system-aix64.ads (Always_Compatible_Rep): Likewise. *

PATCH to tidy up code in c-warn.c

2016-10-16 Thread Marek Polacek
Found when looking at something else. find_array_ref_with_const_idx_r would uselessly keep on looking for other array refs with constant indices, even though finding one is enough. So return when something is found, instead of just ignoring the subtrees. Bootstrapped/regtested on x86_64-linux,

RFC [3/3] divmod transform v2 - add test cases

2016-10-16 Thread Prathamesh Kulkarni
Hi, This patch adds test-cases for divmod transform. OK to commit ? Thanks, Prathamesh 2016-10-15 Prathamesh Kulkarni Kugan Vivekanandarajah Jim Wilson testsuite/ *

RFC [2/3] divmod transform v2 - override expand_divmod_libfunc for ARM port

2016-10-16 Thread Prathamesh Kulkarni
Hi, This patch overrides expand_divmod_libfunc hook for ARM port. I separated the SImode tests into separate file from DImode tests because certain arm configs (cortex-15) have hardware div insn for SImode but not for DImode, and for that config we want SImode tests to be disabled but not DImode

Re: RFC [1/3] divmod transform v2

2016-10-16 Thread Prathamesh Kulkarni
On 16 October 2016 at 11:29, Prathamesh Kulkarni wrote: > Hi, > After approval from Bernd Schmidt, I committed the patch to remove > optab functions for > sdivmod_optab and udivmod_optab in optabs.def, which removes the block > for divmod patch. > > This patch is

Re: [Patch, fortran] Implement inquire(iolength= ) for DTIO

2016-10-16 Thread Dominique d'Humières
Dear Jerry, The new test fails unless I replace 64 with 16 in 'if (rl.ne.64) call abort’. This seems consistent with your comment > The language seems a little obscure. I think the first sentence means > don't expect inquire to use a UDDTIO procedure and the second sentence > says when you use

[PATCH, libfortran] PR 48587 Newunit allocator, cleanup

2016-10-16 Thread Janne Blomqvist
Improve error message, and remove a redundant check, as the same check is done a bit earlier due to the PR 48587 patch. 2016-10-16 Janne Blomqvist PR libfortran/48587 * io/transfer.c (data_transfer_init): Improve error message, remove redundant check.

[SPARC] Do not create bogus CONST_VECTORs

2016-10-16 Thread Eric Botcazou
The compiler was creating bogus CONST_VECTORs under specific circumstances in sparc_expand_vector_init. Fixed by using the same test as the x86 back-end. Tested on SPARC/Solaris, applied on all active branches. 2016-10-16 Eric Botcazou * config/sparc/sparc.c

Re: [PATCH, C++] Warn on redefinition of builtin functions (PR c++/71973)

2016-10-16 Thread Bernd Edlinger
On 10/06/16 16:11, Bernd Edlinger wrote: > Hi! > > Currently C++ does not warn at all when built-in functions are > re-defined with a different signature, while C does warn on that > even without -Wall. > > Thus I'd like to propose a -Wall enabled warning for that in C++ only. > > Initially I

Re: [Patch, fortran] Implement inquire(iolength= ) for DTIO

2016-10-16 Thread Dominique d'Humières
Dear Jerry, > The new test fails unless I replace 64 with 16 in 'if (rl.ne.64) call abort’. > This seems consistent with your comment I have not been clear enough: I have to replace 64 with 16 even with your patch. Dominique

Re: [patch] Fix GC issue triggered by arithmetic overflow checking

2016-10-16 Thread Eric Botcazou
> 2016-10-11 Eric Botcazou > > * tree.h (build_complex_type): Add second parameter with default. > * tree.c (build_complex_type): Add NAMED second parameter and adjust > recursive call. Create a TYPE_DECL only if NAMED is true. >

Re: [Ada] Set Always_Compatible_Rep to False everywhere

2016-10-16 Thread Eric Botcazou
> Can this be refactored to avoid having to duplicate the whole file for > every target? I'm not sure, maybe entries could be omitted if they match the default though. -- Eric Botcazou

Re: [Patch, fortran] Implement inquire(iolength= ) for DTIO

2016-10-16 Thread Dominique d'Humières
Forget it! It seems that I did not apply the patch!-( Sorry for the noise. Dominique > Le 16 oct. 2016 à 18:42, Dominique d'Humières a écrit : > > Dear Jerry, > >> The new test fails unless I replace 64 with 16 in 'if (rl.ne.64) call >> abort’. This seems consistent with

Re: [PATCH] Fix PR55152

2016-10-16 Thread Andrew Pinski
On Tue, Oct 4, 2016 at 2:50 AM, Richard Biener wrote: > On Tue, 4 Oct 2016, Marc Glisse wrote: > >> On Tue, 4 Oct 2016, Richard Biener wrote: >> >> > Possibly. Though then for FP we also want - abs (a) -> copysign (a, -1). >> >> I thought this might fix PR 62055, but at least

[PATCH/AARCH64] Fix some testcases for AARCH64 ILP32

2016-10-16 Thread Andrew Pinski
Hi, These testcases use long for 64bit integer which means they will fail with -mabi=ilp32 on aarch64. This reduces the number of failures down for ILP32. OK? Bootstrapped and tested on aarch64-linux-gnu with no regressions (including with a multi-lib for ilp32). Thanks, Andrew Pinski

Re: [patch v2] Get rid of stack trampolines for nested functions (0/4)

2016-10-16 Thread Eric Botcazou
> this is the updated version of the patch initially posted at: > https://gcc.gnu.org/ml/gcc-patches/2016-06/msg02016.html > It takes into account Jeff's remarks, both on the code and the > documentation. > > As discussed, I'm going to split it into 4 parts: common infrastructure, Ada >