Re: [PATCH] PR target/48904 x86_64-knetbsd-gnu missing defs

2015-05-08 Thread Bernhard Reutner-Fischer
On 1 May 2015 at 01:23, Trevor Saunders wrote: > On Thu, Apr 30, 2015 at 11:58:09PM +0200, Bernhard Reutner-Fischer wrote: >> On April 30, 2015 5:53:02 PM GMT+02:00, Jeff Law wrote: >> >On 04/30/2015 01:58 AM, Bernhard Reutner-Fischer wrote: >> >> Hi, >> >> >> >> On 30 April 2015 at 07:00, Jeff L

Re: PATCH: PR target/48904: x86_64-knetbsd-gnu fails to build

2015-05-08 Thread Bernhard Reutner-Fischer
On 21 September 2012 at 21:11, H.J. Lu wrote: > Hi, > > This patch adds i386/knetbsd-gnu64.h for x86_64-knetbsd-gnu. OK to > install? I now installed this to trunk as r222903 after Jeff's approval. Thanks! > > Thanks. > > H.J. > --- > 2012-09-21 H.J. Lu > > PR target/48904 >

Re: Enhance std::hash for pointers

2015-05-08 Thread Richard Biener
On Wed, May 6, 2015 at 10:10 PM, François Dumont wrote: > Hi > > Following Marc Glisse comment #4 > on:https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65641 I would like to > propose this enhancement to the hash functor for pointers. It simply gets > rid of the irrelevant bits on pointers hash co

Re: [PATCH][tree-ssa-math-opts] Expand pow (x, CONST) using square roots when possible

2015-05-08 Thread Kyrill Tkachov
Ping. https://gcc.gnu.org/ml/gcc-patches/2015-05/msg00071.html Thanks, Kyrill On 01/05/15 17:02, Kyrill Tkachov wrote: Hi all, GCC has some logic to expand calls to pow (x, 0.75), pow (0.25) and pow (x, (int)k + 0.5) using square roots. So, for the above examples it would generate sqrt (x) *

Re: [PATCH 2/13] musl libc config

2015-05-08 Thread Kyrill Tkachov
On 07/05/15 19:02, Jeff Law wrote: On 05/06/2015 05:24 AM, Szabolcs Nagy wrote: On 29/04/15 00:30, Joseph Myers wrote: On Mon, 20 Apr 2015, Szabolcs Nagy wrote: * config/linux.opt (mmusl): New option. New -m options need documenting in invoke.texi. Patch v3. Now with documentatio

Re: [PATCH, ARM] attribute target (thumb,arm) [4/6] respin (4th)

2015-05-08 Thread Ramana Radhakrishnan
I'm still playing with the code, so this is a partial review. We should prevent inlining of ARM state functions into functions we know will be T16 if !TARGET_SOFT_FLOAT on the grounds that the architecture doesn't have floating point instruction encodings in the T16 ISA (Thumb1). We'll just c

Re: [PATCH 0/13] Add musl support to GCC

2015-05-08 Thread Kyrill Tkachov
On 07/05/15 19:07, Jeff Law wrote: On 05/06/2015 05:36 AM, Szabolcs Nagy wrote: On 30/04/15 00:18, Joseph Myers wrote: On Wed, 29 Apr 2015, Szabolcs Nagy wrote: only affects [u]int_fastN_t types (on 64bit systems for N=16,32 musl uses int but glibc uses long) i can fix glibc-stdint.h, but it

Re: [libgomp, testsuite] Support parallel testing in libgomp (PR libgomp/66005)

2015-05-08 Thread Thomas Schwinge
Hi! On Thu, 7 May 2015 13:39:40 +0200, Jakub Jelinek wrote: > On Thu, May 07, 2015 at 01:26:57PM +0200, Rainer Orth wrote: > > As reported in the PR, with the addition of all those OpenACC tests, > > libgomp make check times have skyrocketed since the testsuite is still > > run sequentially. ACK

Re: [PATCH, doc] fix match-and-simplify API doc errors

2015-05-08 Thread Richard Biener
On Fri, May 8, 2015 at 1:37 AM, Jim Wilson wrote: > I noticed this while reading the match-and-simplify docs. The > gimple_simplify API list has two built-in function cases with three > tree args. The last one is supposed to have four tree args for a > ternary function (3 inputs and one output).

genrecog: Address -Wsign-compare diagnostics (was: Mostly rewrite genrecog)

2015-05-08 Thread Thomas Schwinge
Hi! On Mon, 27 Apr 2015 11:20:30 +0100, Richard Sandiford wrote: > This patch [...] by replacing most of genrecog [...] OK to commit? Is it a bug that I'm seeing these warnings only in the stage 1 build with the bootstrap GCC 4.6 compiler, but not anymore later on? (I have not verified the C+

Re: PR 64454: Improve VRP for %

2015-05-08 Thread Richard Biener
On Mon, May 4, 2015 at 3:57 PM, Marc Glisse wrote: > On Mon, 4 May 2015, Richard Biener wrote: > >> On Sat, May 2, 2015 at 12:46 AM, Marc Glisse wrote: >>> >>> Hello, >>> >>> this patch tries to tighten a bit the range estimate for x%y. >>> slp-perm-7.c >>> started failing by vectorizing more tha

Re: [PATCH, ARM] attribute target (thumb,arm) [5/6] respin (4th)

2015-05-08 Thread Ramana Radhakrishnan
On 06/05/15 15:27, Christian Bruel wrote: Implements the hooks for #pragma GCC target A test included to check that macros were correctly defined/undefined on pragma regions. Thanks Christian Missing the hooks - this only appears to have the test. Ramana

Re: [PATCH] Simple optimization for MASK_STORE.

2015-05-08 Thread Richard Biener
On Wed, May 6, 2015 at 4:04 PM, Yuri Rumyantsev wrote: > Hi All, > > Here is a patch which gives us significant speed-up on HASWELL for > test containing masked stores. The main goal of that patch is attempt > to avoid HW hazard for maskmove instructions through inserting > additional check on zer

Re: [patch 3/28] fixincludes: Use automake-1.11.6 (across the tree)

2015-05-08 Thread Michael Haubenwallner
On 05/08/2015 12:10 AM, Bruce Korb wrote: > On 05/06/15 01:58, Michael Haubenwallner wrote: >> Trivial patch for fixincludes. > > A) sufficiently trivial that explicit permission ought not be required Agreed for the actual code change - more important is to notify the automake revbump. > B) it

Re: [PATCH][tree-ssa-math-opts] Expand pow (x, CONST) using square roots when possible

2015-05-08 Thread Richard Biener
On Fri, May 1, 2015 at 6:02 PM, Kyrill Tkachov wrote: > Hi all, > > GCC has some logic to expand calls to pow (x, 0.75), pow (0.25) and pow (x, > (int)k + 0.5) > using square roots. So, for the above examples it would generate sqrt (x) * > sqrt (sqrt (x)), > sqrt (sqrt (x)) and powi (x, k) * sqrt

Re: [patch 1/10] debug-early merge: Ada front-end

2015-05-08 Thread Eric Botcazou
> @@ -5204,28 +5199,6 @@ gnat_write_global_declarations (void) > types_used_by_var_decl_insert (t, dummy_global); > } > } > - > - /* Output debug information for all global type declarations first. This > - ensures that global types whose compilation hasn't been finalized >

Re: Remove mode argument from gen_rtx_SET

2015-05-08 Thread Franz Sirl
Am 2015-05-07 um 13:37 schrieb Richard Sandiford: One problem with the automatically-generated gen_rtx_FOO () macros is that they always have a mode parameter, even for codes like SET where the mode should always be VOIDmode. This inevitably leads to cases where a caller accidentally passes some

Re: [patch 1/10] debug-early merge: Ada front-end

2015-05-08 Thread Richard Biener
On Fri, May 8, 2015 at 12:26 PM, Eric Botcazou wrote: >> @@ -5204,28 +5199,6 @@ gnat_write_global_declarations (void) >> types_used_by_var_decl_insert (t, dummy_global); >> } >> } >> - >> - /* Output debug information for all global type declarations first. This >> - ensur

Re: [AArch64] Fix predicate and constraint mismatch in logical atomic operations

2015-05-08 Thread Richard Biener
On Tue, Nov 4, 2014 at 11:44 AM, Marcus Shawcroft wrote: > On 25 September 2014 04:45, Michael Collison > wrote: >> On certain patterns in atomics.md the constraint 'n' is used in combination >> with the predicate atomic_op_operand. The constraint is too general and >> allows constants that are d

[PATCH PR65447]Improve IV handling by grouping address type uses with same base and step

2015-05-08 Thread Bin Cheng
Hi, GCC's IVO currently handles every IV use independently, which is not right by learning from cases reported in PR65447. The rationale is: 1) Lots of address type IVs refer to the same memory object, share similar base and have same step. We should handle these IVs as a group in order to maximi

Re: [Patch, Fortran, PR58586, v3] ICE with derived type with allocatable component passed by value

2015-05-08 Thread Andre Vehreschild
Hi Mikael, thanks for the review. I still have some questions/remarks before commiting: On Thu, 07 May 2015 12:14:59 +0200 Mikael Morin wrote: > > @@ -2158,6 +2158,8 @@ build_function_decl (gfc_symbol * sym, bool global) > > gfc_set_decl_assembler_name (fndecl, gfc_sym_mangled_function_id

RE: [patch 0/27] RFC: Use automake-1.11.6 across the tree

2015-05-08 Thread Bernd Edlinger
Hi Michael, On Thu, 7 May 2015 18:52:52, Michael Haubenwallner wrote: > > Hi Bernd, > > On 05/06/2015 03:01 PM, Bernd Edlinger wrote: >> On Tue, 5 May 2015 18:03:15, Michael Haubenwallner wrote: >>> >>> Now that gcc-5 is out, what about an automake-1.11.6 update for gcc-6? >>> >>> BTW, the actual

Re: Fix logic error in Fortran OpenACC parsing

2015-05-08 Thread Ilmir Usmanov
Hi! On 06.05.2015 14:38, Thomas Schwinge wrote: Hi! On Tue, 5 May 2015 15:38:03 -0400, David Malcolm wrote: On Wed, 2015-04-29 at 14:10 +0200, Mikael Morin wrote: Le 29/04/2015 02:02, David Malcolm a écrit : diff --git a/gcc/fortran/parse.c b/gcc/fortran/parse.c index 2c7c554..30e4eab 10064

RE: [patch 1/28] top-level: Use automake-1.11.6

2015-05-08 Thread Bernd Edlinger
Hi, On Thu, 7 May 2015 15:25:14, Joseph S. Myers wrote: > > On Thu, 7 May 2015, Bernd Edlinger wrote: > >> But that is not the case for other tool scripts. I think these should >> be in-sync with the automake version that creates the configure scripts >> that make use of them. > > At least some o

Re: [patch] Implement ISO/IEC TS 18822 C++ File system TS

2015-05-08 Thread Rainer Orth
Jonathan Wakely writes: > I've committed the two changes attached (only tested on linux again). > > patch2.txt should fix the mingw-w64 errors above, as well as the > issues Daniel reported, and should fix the error on Solaris 10 > > Rainer, would you be able to test with > --enable-libstdcxx-fil

Re: [Patch, fortran, pr65894, v1] [6 Regression] severe regression in gfortran 6.0.0

2015-05-08 Thread Andre Vehreschild
Hi Mikael, at first I tried to fix this issue with the scalarizer, too, but I could not grasp how the scalarizer was working. Do you have any documentation, how it is meant to be? I mean, I have read the comments in the code, but those are sparse and the multitude of routines the scalarizer is spl

Re: [patch 7/10] debug-early merge: LTO

2015-05-08 Thread Richard Biener
On Fri, May 8, 2015 at 2:37 AM, Aldy Hernandez wrote: > Ok. Thanks, Richard.

Re: [patch 0/27] RFC: Use automake-1.11.6 across the tree

2015-05-08 Thread Andreas Schwab
Michael Haubenwallner writes: > This starts to become tricky here on my quite up-to-date Gentoo stable amd64 > box: > > The normal host compiler is: gcc version 4.8.4 configured to > --enable-languages=c,c++ > while the gnat compiler is: gnatgcc version 4.3.5 configured to > --enable-lang

Re: [patch 3/10] debug-early merge: C++ front-end

2015-05-08 Thread Richard Biener
On Fri, May 8, 2015 at 2:34 AM, Aldy Hernandez wrote: > Maybe you can split out the Java aliases stuff (that annoyed me multiple times when trying to refactor the FE - middle-end interface). It looks unrelated enough. Thanks, Richard.

Re: Remove mode argument from gen_rtx_SET

2015-05-08 Thread Segher Boessenkool
On Fri, May 08, 2015 at 12:32:30PM +0200, Franz Sirl wrote: > this patch (r222882 is fine, r222883 fails) breaks bootstrap for me on > x86_64-linux-gnu: i386.md has "set:BND" twice; replace that with just "set", and all should be fine. Maybe gen* should warn on this; maybe it already does. Seg

Re: [patch 0/10] debug-early merge

2015-05-08 Thread Richard Biener
On Fri, May 8, 2015 at 2:30 AM, Aldy Hernandez wrote: > Hi folks. > > I have divided the patches into 10 pieces. The patches are interdependent > and cannot be applied independently. I am merely dividing them up to aid > the relevant reviewers. > > As I've mentioned elsewhere, the patchset as po

Re: [patch 4/10] debug-early merge: Fortran front-end

2015-05-08 Thread Tobias Burnus
Aldy Hernandez wrote: > gcc/fortran/ > > * f95-lang.c (gfc_write_global_declarations): Remove. > (LANG_HOOKS_WRITE_GLOBALS): Remove. > (gfc_write_global_declarations): Move code from here to... > (gfc_be_parse_file): ...here. > Call global_decl_processing. > * tr

Re: [RFC] Elimination of zext/sext - type promotion pass

2015-05-08 Thread Richard Biener
On Fri, May 1, 2015 at 6:41 AM, Kugan wrote: > >>> Thanks for the comments. Here is a prototype patch that implements a >>> type promotion pass. This pass records SSA variables that will have >>> values in higher bits (than the original type precision) if promoted and >>> uses this information in

Re: Remove mode argument from gen_rtx_SET

2015-05-08 Thread Franz Sirl
Am 2015-05-08 um 13:57 schrieb Segher Boessenkool: On Fri, May 08, 2015 at 12:32:30PM +0200, Franz Sirl wrote: this patch (r222882 is fine, r222883 fails) breaks bootstrap for me on x86_64-linux-gnu: i386.md has "set:BND" twice; replace that with just "set", and all should be fine. Maybe gen*

Re: [PATCH 5/14][AArch64] Add basic fp16 support

2015-05-08 Thread Alan Lawrence
Joseph Myers wrote: > I'd think it would be desirable to share tests between ARM and AArch64 as far as possible (where applicable to both - so not the tests for the alternative format, and some of the gcc.target/arm/fp16-* tests using scan-assembler might need adapting to work for AArch64).

Re: [PATCH 0/14][ARM/AArch64] __FP16 support, vectors, intrinsics, testsuite

2015-05-08 Thread Alan Lawrence
Alan Lawrence wrote: This patch series adds support for ARM Neon float16x4_t and float16x8_t vector types and intrinsics, and the __fp16 type, on both ARM and AArch64, and extends the tests in Christophe Lyon's advsimd-intrinsics testsuite to cover these. (I chose to extend the existing tests r

Re: [PATCH 1/2][ARM] PR/63870: Add qualifier to check lane bounds in expand

2015-05-08 Thread Alan Lawrence
Alan Lawrence wrote: Ping (https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01422.html). These are required for float16 patches posted at https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01332.html . Bootstrapped + check-gcc on arm-none-linux-gnueabihf. Alan Lawrence wrote: This is based loosely u

[Patch, fortran] Fix elemental optional dummy argument handling

2015-05-08 Thread Mikael Morin
Hello, I found a (unrelated) bug while playing with Andre's PR65894 patch. The dummy argument can get out of sync with the actual argument when there is an (optional) argument missing. I plan to commit the attached fix as obvious later today (after testing). Mikael 2015-05-08 Mikael Morin

Re: [Patch, Fortran, PR58586, v3] ICE with derived type with allocatable component passed by value

2015-05-08 Thread Mikael Morin
Le 08/05/2015 12:54, Andre Vehreschild a écrit : > Hi Mikael, > > thanks for the review. I still have some questions/remarks before commiting: > >>> @@ -5898,8 +5900,21 @@ gfc_generate_function_code (gfc_namespace * ns) >>> >>>if (TREE_TYPE (DECL_RESULT (fndecl)) != void_type_node) >>>

[Patch, Fortran, 66035, v1] [5/6 Regression] gfortran ICE segfault

2015-05-08 Thread Andre Vehreschild
Hi all, please find attached a patch for 66035. An ICE occurred when in a structure constructor an allocatable component of type class was initialized with an existing class object. This was caused by - the size of the memory to allocate for the component was miscalculated, - the vptr was not se

Re: [Patch, Fortran, PR58586, v3] ICE with derived type with allocatable component passed by value

2015-05-08 Thread Andre Vehreschild
Hi Mikael, > > ?? I don't get you there? What do you mean? Do you think the > > alloc_comp_class_3/4.* are not correctly testing the issue? Any idea of how > > to test this better? I mean the pr is about this artificial constructs. I > > merely struck it in search of a pr about allocatable compone

[PATCH] rs6000: Fix peephole

2015-05-08 Thread Segher Boessenkool
This peephole transforms lis a,HI ; ori a,a,LO cmpw c,a,b ; beq c,... to xoris a,b,HI1 cmpwi c,a,LO1 ; beq c,... when a and c are dead after this. But it forgets to check that a and b are not the same reg, generating non-sensical code. This patch fixes that. Tested etc.; is this okay

Re: [PATCH 6/13] mips musl support

2015-05-08 Thread H.J. Lu
On Mon, Apr 27, 2015 at 7:40 AM, Szabolcs Nagy wrote: > > > On 21/04/15 15:59, Matthew Fortune wrote: >> Rich Felker writes: >>> On Tue, Apr 21, 2015 at 01:58:02PM +, Matthew Fortune wrote: There does however appear to be both soft and hard float variants > > Patch v2. > > Now all the AB

Re: [PATCH][tree-ssa-math-opts] Expand pow (x, CONST) using square roots when possible

2015-05-08 Thread Kyrill Tkachov
On 08/05/15 11:18, Richard Biener wrote: On Fri, May 1, 2015 at 6:02 PM, Kyrill Tkachov wrote: Hi all, GCC has some logic to expand calls to pow (x, 0.75), pow (0.25) and pow (x, (int)k + 0.5) using square roots. So, for the above examples it would generate sqrt (x) * sqrt (sqrt (x)), sqrt (s

Re: Remove mode argument from gen_rtx_SET

2015-05-08 Thread Richard Sandiford
Franz Sirl writes: > Am 2015-05-08 um 13:57 schrieb Segher Boessenkool: >> On Fri, May 08, 2015 at 12:32:30PM +0200, Franz Sirl wrote: >>> this patch (r222882 is fine, r222883 fails) breaks bootstrap for me on >>> x86_64-linux-gnu: >> >> i386.md has "set:BND" twice; replace that with just "set", a

Re: [Patch, fortran, pr65894, v1] [6 Regression] severe regression in gfortran 6.0.0

2015-05-08 Thread Mikael Morin
Le 08/05/2015 13:54, Andre Vehreschild a écrit : > Hi Mikael, > > at first I tried to fix this issue with the scalarizer, too, but I could not > grasp how the scalarizer was working. Do you have any documentation, how it is > meant to be? I mean, I have read the comments in the code, but those are

Re: [Patch, Fortran, PR58586, v3] ICE with derived type with allocatable component passed by value

2015-05-08 Thread Andre Vehreschild
Hi, so attached is a quick and dirty solution for the allocatable return value problem. I personally don't like it. It is making a special case from the assign a function result to a variable. May be you have a better idea how to do this in gfortran style. - Andre On Fri, 8 May 2015 15:31:46 +0

Re: [PATCH] rs6000: Fix peephole

2015-05-08 Thread David Edelsohn
On Fri, May 8, 2015 at 9:38 AM, Segher Boessenkool wrote: > This peephole transforms > > lis a,HI ; ori a,a,LO > cmpw c,a,b ; beq c,... > > to > > xoris a,b,HI1 > cmpwi c,a,LO1 ; beq c,... > > when a and c are dead after this. But it forgets to check that a and b > are not the same reg, g

Re: [PATCH] rs6000: Fix peephole

2015-05-08 Thread Segher Boessenkool
On Fri, May 08, 2015 at 10:18:46AM -0400, David Edelsohn wrote: > > 2015-05-08 Segher Boessenkool > > > > * config/rs6000/rs6000.md: Require operand inequality in one > > of the peepholes. > > Okay. > > Is there an artificial testcase? I don't have one. Peepholes require opti

Re: [PATCH 6/13] mips musl support

2015-05-08 Thread Szabolcs Nagy
On 08/05/15 14:56, H.J. Lu wrote: > On Mon, Apr 27, 2015 at 7:40 AM, Szabolcs Nagy wrote: >> On 21/04/15 15:59, Matthew Fortune wrote: >>> Rich Felker writes: On Tue, Apr 21, 2015 at 01:58:02PM +, Matthew Fortune wrote: > There does however appear to be both soft and hard float varia

RE: [PATCH 6/13] mips musl support

2015-05-08 Thread Matthew Fortune
H.J. Lu writes: > On Mon, Apr 27, 2015 at 7:40 AM, Szabolcs Nagy > wrote: > > > > > > On 21/04/15 15:59, Matthew Fortune wrote: > >> Rich Felker writes: > >>> On Tue, Apr 21, 2015 at 01:58:02PM +, Matthew Fortune wrote: > There does however appear to be both soft and hard float variants

[RFA] libiberty/mkstemps.c: Include if not available.

2015-05-08 Thread Joel Brobecker
Hello, Attempting to build libiberty on LynxOS-178 fails trying to compile mkstemps.c with the following error: mkstemps.c:84:18: error: storage size of 'tv' isn't known struct timeval tv; ^ This file would normally include to get the type's definition, but unfo

Re: [PATCH 6/13] mips musl support

2015-05-08 Thread Rich Felker
On Fri, May 08, 2015 at 02:25:11PM +, Matthew Fortune wrote: > H.J. Lu writes: > > On Mon, Apr 27, 2015 at 7:40 AM, Szabolcs Nagy > > wrote: > > > > > > > > > On 21/04/15 15:59, Matthew Fortune wrote: > > >> Rich Felker writes: > > >>> On Tue, Apr 21, 2015 at 01:58:02PM +, Matthew Fortun

Re: [PATCH 6/13] mips musl support

2015-05-08 Thread Szabolcs Nagy
On 08/05/15 15:25, Matthew Fortune wrote: > H.J. Lu writes: >> On Mon, Apr 27, 2015 at 7:40 AM, Szabolcs Nagy >> wrote: >>> >>> >>> On 21/04/15 15:59, Matthew Fortune wrote: Rich Felker writes: > On Tue, Apr 21, 2015 at 01:58:02PM +, Matthew Fortune wrote: >> There does howeve

Re: genrecog: Address -Wsign-compare diagnostics

2015-05-08 Thread Richard Sandiford
Thomas Schwinge writes: > Hi! > > On Mon, 27 Apr 2015 11:20:30 +0100, Richard Sandiford > wrote: >> This patch [...] by replacing most of genrecog [...] > > OK to commit? Looks good to me FWIW. Probably counts as obvious. Thanks, Richard

RE: [PATCH 6/13] mips musl support

2015-05-08 Thread Matthew Fortune
Szabolcs Nagy writes: > On 08/05/15 15:25, Matthew Fortune wrote: > > H.J. Lu writes: > >> On Mon, Apr 27, 2015 at 7:40 AM, Szabolcs Nagy > >> > >> wrote: > >>> > >>> > >>> On 21/04/15 15:59, Matthew Fortune wrote: > Rich Felker writes: > > On Tue, Apr 21, 2015 at 01:58:02PM +, Mat

Re: [patch 0/10] debug-early merge

2015-05-08 Thread David Malcolm
On Thu, 2015-05-07 at 17:30 -0700, Aldy Hernandez wrote: > Hi folks. > > I have divided the patches into 10 pieces. The patches are > interdependent and cannot be applied independently. I am merely > dividing them up to aid the relevant reviewers. > > As I've mentioned elsewhere, the patchset

Re: [PATCH 6/13] mips musl support

2015-05-08 Thread Rich Felker
On Fri, May 08, 2015 at 03:41:31PM +0100, Szabolcs Nagy wrote: > > I.e. as it stands this patch is not OK for backporting to GCC 5 > > without further discussion. > > > > There is also the perspective that we should be able to aim for > > an ABI variant agnostic dynamic linker at some point over t

Re: PR 64454: Improve VRP for %

2015-05-08 Thread Marc Glisse
Hello, here is a rewrite of the patch, using wide_int, and improving a bit the result. Same ChangeLog, tested again on x86_64-linux-gnu. -- Marc GlisseIndex: gcc/testsuite/gcc.dg/tree-ssa/vrp97.c === --- gcc/testsuite/gcc.dg/tree-

Re: [patch 0/10] debug-early merge

2015-05-08 Thread Aldy Hernandez
On 05/08/2015 07:41 AM, David Malcolm wrote: On Thu, 2015-05-07 at 17:30 -0700, Aldy Hernandez wrote: Hi folks. I have divided the patches into 10 pieces. The patches are interdependent and cannot be applied independently. I am merely dividing them up to aid the relevant reviewers. As I've m

[PATCH] Vectorize strided group loads

2015-05-08 Thread Richard Biener
Currently the vectorizer forces unrolling for grouped loads that have DR_STEP not constant, forcing the elements loaded with strided load support. The following patch enhances that machinery to deal with SLP used groups that have non-constant DR_STEP, avoiding the excessive unrolling (and (un-)pa

Re: [PATCH][tree-ssa-math-opts] Expand pow (x, CONST) using square roots when possible

2015-05-08 Thread Kyrill Tkachov
On 08/05/15 14:56, Kyrill Tkachov wrote: On 08/05/15 11:18, Richard Biener wrote: On Fri, May 1, 2015 at 6:02 PM, Kyrill Tkachov wrote: Hi all, GCC has some logic to expand calls to pow (x, 0.75), pow (0.25) and pow (x, (int)k + 0.5) using square roots. So, for the above examples it would ge

[PATCH][PR66013] Update address_taken after ifn_va_arg expansion

2015-05-08 Thread Tom de Vries
Hi, this patch fixes PR66013. I. Consider this test-case, with a va_list passed from f2 to f2_1: ... #include inline int __attribute__((always_inline)) f2_1 (va_list ap) { return va_arg (ap, int); } int f2 (int i, ...) { int res; va_list ap; va_start (ap, i); res = f2_1 (ap); v

Re: [PATCH 6/13] mips musl support

2015-05-08 Thread Szabolcs Nagy
On 08/05/15 15:46, Matthew Fortune wrote: > Szabolcs Nagy writes: >> if you think that's ok, i can now submit the patch with %{msoft-float:- >> sf} added to all abi variants. > > That's fine. Go ahead. > the patch for the record. Changelog: 2015-05-08 Gregor Richards Szabolcs N

Re: [PATCH 6/13] mips musl support

2015-05-08 Thread Kyrill Tkachov
On 08/05/15 16:13, Szabolcs Nagy wrote: On 08/05/15 15:46, Matthew Fortune wrote: Szabolcs Nagy writes: if you think that's ok, i can now submit the patch with %{msoft-float:- sf} added to all abi variants. That's fine. Go ahead. the patch for the record. I've committed this on Szabolcs'

Re: [patch 1/10] debug-early merge: Ada front-end

2015-05-08 Thread Aldy Hernandez
On 05/08/2015 03:35 AM, Richard Biener wrote: On Fri, May 8, 2015 at 12:26 PM, Eric Botcazou wrote: @@ -5204,28 +5199,6 @@ gnat_write_global_declarations (void) types_used_by_var_decl_insert (t, dummy_global); } } - - /* Output debug information for all global type declar

Re: [RFC 0/6] Flags outputs for asms

2015-05-08 Thread Richard Henderson
On 05/07/2015 06:20 PM, H. Peter Anvin wrote: > This is a separate issue which really shouldn't have anything to do with > this, but is there a specific reason why: > > void good1(int x, int y) > { > _Bool pf; > > asm("cmpl %2,%1" > : "=@ccp" (pf) > : "r" (x), "g" (y)); > > if

Re: [Patch, fortran, pr65894, v1] [6 Regression] severe regression in gfortran 6.0.0

2015-05-08 Thread Steve Kargl
On Fri, May 08, 2015 at 01:54:17PM +0200, Andre Vehreschild wrote: > > I do not have the privileges to do a review so I can't help you there. Good > luck finding a reviewer. > You probably understand this area of code as well as anyone else, and your contributions to gfortran over the last few m

Re: PR 64454: Improve VRP for %

2015-05-08 Thread Richard Biener
On Fri, May 8, 2015 at 4:59 PM, Marc Glisse wrote: > Hello, > > here is a rewrite of the patch, using wide_int, and improving a bit the > result. Same ChangeLog, tested again on x86_64-linux-gnu. Please use /* */ for comments. Otherwise ok! Thanks, Richard. > -- > Marc Glisse > Index: gcc/test

Re: [PATCH][PR66013] Update address_taken after ifn_va_arg expansion

2015-05-08 Thread Richard Biener
On Fri, May 8, 2015 at 5:13 PM, Tom de Vries wrote: > Hi, > > this patch fixes PR66013. > > > I. > > Consider this test-case, with a va_list passed from f2 to f2_1: > ... > #include > > inline int __attribute__((always_inline)) > f2_1 (va_list ap) > { > return va_arg (ap, int); > } > > int > f2

Re: [RFC 0/6] Flags outputs for asms

2015-05-08 Thread Jay Foad
On 8 May 2015 at 16:23, Richard Henderson wrote: > Yes, the i386 backend has not implemented conditional sibcalls. See: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60159 Jay.

Re: [RFC 0/6] Flags outputs for asms

2015-05-08 Thread Jeff Law
On 05/08/2015 09:23 AM, Richard Henderson wrote: On 05/07/2015 06:20 PM, H. Peter Anvin wrote: This is a separate issue which really shouldn't have anything to do with this, but is there a specific reason why: void good1(int x, int y) { _Bool pf; asm("cmpl %2,%1" : "=@ccp" (pf)

Re: [RFC 0/6] Flags outputs for asms

2015-05-08 Thread Richard Henderson
On 05/07/2015 06:15 PM, H. Peter Anvin wrote: > /* This case really should produce good code in both cases */ > > void good1(int x, int y) > { > _Bool pf; > > asm("cmpl %2,%1" > : "=@ccp" (pf) > : "r" (x), "g" (y)); > > if (pf) > beta(); > } > > void bad1(int x, int y) > {

Re: [patch 1/10] debug-early merge: Ada front-end

2015-05-08 Thread Aldy Hernandez
On 05/08/2015 03:26 AM, Eric Botcazou wrote: @@ -5204,28 +5199,6 @@ gnat_write_global_declarations (void) types_used_by_var_decl_insert (t, dummy_global); } } - - /* Output debug information for all global type declarations first. This - ensures that global types who

Re: [C/C++ PATCH] Implement -Wshift-negative-value (PR c/65179)

2015-05-08 Thread Steve Ellcey
On Thu, 2015-05-07 at 21:15 +0200, Marek Polacek wrote: > On Thu, May 07, 2015 at 12:00:20PM -0600, Jeff Law wrote: > > OK. Please install if you haven't already. > > I have not, so will do momentarily. Thanks, > > Marek Marek, This patch has broken the glibc build. I am not sure if th

Re: [C/C++ PATCH] Implement -Wshift-negative-value (PR c/65179)

2015-05-08 Thread Markus Trippelsdorf
On 2015.05.08 at 09:38 -0700, Steve Ellcey wrote: > > This patch has broken the glibc build. I am not sure if the problem is > a bug in your patch or a bug in the code used by glibc. Here is a > cutdown test case from glibc (timezone/scheck.c). This code compiled > before your patch but now it

RE: [patch 1/28] top-level: Use automake-1.11.6

2015-05-08 Thread Joseph Myers
On Fri, 8 May 2015, Bernd Edlinger wrote: > One example where there is an incompatibility is "missing": > > Formerly it had code that emulated the missing "flex" by > creating a dummy lex.yy.c from the hopefully installed > pre-compiled flex output file.  But the version from the > trunk does not

Re: Remove mode argument from gen_rtx_SET

2015-05-08 Thread DJ Delorie
> ; This pattern is identical to the truncsipsi2 pattern except > ; that it uses a SUBREG instead of a TRUNC. It is needed in > ; order to prevent reload from converting (set:SI (SUBREG:PSI (SI))) > ; into (SET:PSI (PSI)). > > I'm not sure what that's supposed to mean (what's an SI set of a PSI

[PATCH, alpha]: Remove dead (HOST_BITS_PER_WIDE_INT < 64) code

2015-05-08 Thread Uros Bizjak
... to prepare alpha for TARGET_SUPPORTS_WIDE_INT switch. 2015-05-08 Uros Bizjak * config/alpha/alpha.c (alpha_emit_set_const_1) (alpha_emit_set_long_const, alpha_extract_integer) (alpha_legitimate_constant_p, alpha_split_const_mov) (alpha_expand_block_clear, alpha_expand_zap_m

Re: [RFA] libiberty/mkstemps.c: Include if not available.

2015-05-08 Thread DJ Delorie
> * mkstemps.c: #include if HAVE_TIME_H is defined > but not HAVE_SYS_TIME_H. Ok.

Re: [PATCH 6/13] mips musl support

2015-05-08 Thread Joseph Myers
On Fri, 8 May 2015, Rich Felker wrote: > On Fri, May 08, 2015 at 03:41:31PM +0100, Szabolcs Nagy wrote: > > > I.e. as it stands this patch is not OK for backporting to GCC 5 > > > without further discussion. > > > > > > There is also the perspective that we should be able to aim for > > > an ABI

Re: [Patch, fortran] PR 37131, inline matmul

2015-05-08 Thread H.J. Lu
On Mon, May 4, 2015 at 11:25 PM, Thomas Koenig wrote: > Hello world, > > this is an update of the matmul inline patch. The only difference to > the last version is that it has the ubound simplification taken out. > > Any further comments? OK for trunk? > > Thomas > > 2015-05-05 Thomas K

Re: [C/C++ PATCH] Implement -Wshift-negative-value (PR c/65179)

2015-05-08 Thread Joseph Myers
On Fri, 8 May 2015, Steve Ellcey wrote: > On Thu, 2015-05-07 at 21:15 +0200, Marek Polacek wrote: > > On Thu, May 07, 2015 at 12:00:20PM -0600, Jeff Law wrote: > > > OK. Please install if you haven't already. > > > > I have not, so will do momentarily. Thanks, > > > > Marek > > Marek, >

Re: [PATCH 6/13] mips musl support

2015-05-08 Thread Rich Felker
On Fri, May 08, 2015 at 04:50:28PM +, Joseph Myers wrote: > On Fri, 8 May 2015, Rich Felker wrote: > > > On Fri, May 08, 2015 at 03:41:31PM +0100, Szabolcs Nagy wrote: > > > > I.e. as it stands this patch is not OK for backporting to GCC 5 > > > > without further discussion. > > > > > > > > T

Re: [RFA] libiberty/mkstemps.c: Include if not available.

2015-05-08 Thread Joel Brobecker
> > * mkstemps.c: #include if HAVE_TIME_H is defined > > but not HAVE_SYS_TIME_H. > > Ok. Thank you, DJ. Pushed to both GCC and binutils-gdb. -- Joel

Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-08 Thread H.J. Lu
On Thu, May 7, 2015 at 2:17 PM, Joseph Myers wrote: > On Fri, 6 Mar 2015, H.J. Lu wrote: > >> +# We don't want to compile the compiler with -fPIE, it make PCH fail. >> +COMPILER += @NO_PIE_CFLAGS@ >> + >> +# Link with -no-pie since we compile the compiler with -fno-PIE. >> +LINKER += @NO_PIE_FLAG@

[PATCH 2/5] Use NO_FPIE_AND_FPIC_SPEC in NO_SHARED_SPECS

2015-05-08 Thread H.J. Lu
OK for trunk? * config/mips/gnu-user.h (NO_SHARED_SPECS): Use NO_FPIE_AND_FPIC_SPEC. --- gcc/config/mips/gnu-user.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/config/mips/gnu-user.h b/gcc/config/mips/gnu-user.h index 28b00ed..dd4cf11 100644 --- a/gcc/c

[PATCH 3/5] Use FPIE_OR_FPIC_SPEC in CRIS_ASM_SUBTARGET_SPEC

2015-05-08 Thread H.J. Lu
OK for trunk? H.J. * config/cris/linux.h (CRIS_ASM_SUBTARGET_SPEC): Use FPIE_OR_FPIC_SPEC. --- gcc/config/cris/linux.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gcc/config/cris/linux.h b/gcc/config/cris/linux.h index bd57986..262aac5 100644 --- a/gc

[PATCH 5/5] Use FPIE_OR_FPIC_SPEC in ASM_SPEC

2015-05-08 Thread H.J. Lu
OK for trunk? H.J. * config/sparc/linux.h (ASM_SPEC): Use FPIE_OR_FPIC_SPEC. * config/sparc/linux64.h (ASM_SPEC): Likewise. --- gcc/config/sparc/linux.h | 2 +- gcc/config/sparc/linux64.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/gcc/config/sparc/li

[PATCH 4/5] Use FPIE_OR_FPIC_SPEC in ASM_SPEC

2015-05-08 Thread H.J. Lu
OK for trunk? H.J. * config/rs6000/sysv4.h (ASM_SPEC): Use FPIE_OR_FPIC_SPEC. --- gcc/config/rs6000/sysv4.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/config/rs6000/sysv4.h b/gcc/config/rs6000/sysv4.h index 9917c2f..1041648 100644 --- a/gcc/config/rs6000/sysv

Re: [RFC 0/6] Flags outputs for asms

2015-05-08 Thread H. Peter Anvin
On 05/08/2015 08:54 AM, Richard Henderson wrote: > > Anyway, I'll look into whether the branch around alpha can be optimized, but > I'd be shocked if I'd be able to do anything about the branch around beta. > True, there's nothing in between that will clobber the flags so it would be an > excellen

Re: [patch 8/10] debug-early merge: Objective-C front-end

2015-05-08 Thread Mike Stump
On May 7, 2015, at 5:38 PM, Aldy Hernandez wrote: Ok.

Re: [patch 9/10] debug-early merge: testsuite changes

2015-05-08 Thread Mike Stump
On May 7, 2015, at 5:39 PM, Aldy Hernandez wrote: So, I don’t feel there is anything in there for me to review, I’d like the front-end maintainer to review. dearly-testsuite.patch Description: Binary data

Question about patch for PR bootstrap/65150 (identical functions)

2015-05-08 Thread Steve Ellcey
Jan and Martin, I just noticed that your patch for PR bootstrap/65150 broke one of the MIPS tests (gcc.target/mips/branch-1.c). I can fix the test with no problem but I am wondering if the change I am seeing with your patch is intended or not. A cutdown version of the test is: void bar (void);

Re: [debug-early] fix -fdump-go-spec

2015-05-08 Thread Ian Lance Taylor
On Wed, Apr 29, 2015 at 5:56 PM, Aldy Hernandez wrote: > > Despite what Go thinks: > > /* The debug hooks are used to implement -fdump-go-spec because it > gives a simple and stable API for all the information we need to > dump. */ > > ...the debug hooks are not stable... :). Alas.

Re: [patch 5/10] debug-early merge: Go front-end

2015-05-08 Thread Ian Lance Taylor
This is fine if it works. Thanks. Ian On Thu, May 7, 2015 at 5:36 PM, Aldy Hernandez wrote: >

Re: [PATCH 1/6] Only resolve_asm_operand_names once

2015-05-08 Thread Jeff Law
On 05/07/2015 03:38 PM, Richard Henderson wrote: We do it in the front end already; no need to repeat. --- gcc/cfgexpand.c | 2 -- gcc/stmt.c | 7 --- 2 files changed, 4 insertions(+), 5 deletions(-) Any reason this shouldn't go into the tree immediately? Seems like it stands on it

Re: [PATCH 3/6] Canonicalize asm volatility earlier

2015-05-08 Thread Jeff Law
On 05/07/2015 03:38 PM, Richard Henderson wrote: If gimple_asm_volatile_p is correct, no point re-checking. This is also done by the C and C++ front ends, but not Ada. So we can't yet trust ASM_VOLATILE_P from the front end. --- gcc/cfgexpand.c | 11 +++ gcc/gimplify.c | 2 +- 2 fil

Re: [PATCH 5/6] i386: Add CCPmode

2015-05-08 Thread Jeff Law
On 05/07/2015 03:38 PM, Richard Henderson wrote: For testing parity coming out of asm flags. --- gcc/config/i386/i386-modes.def | 2 ++ gcc/config/i386/i386.c | 19 +++ 2 files changed, 13 insertions(+), 8 deletions(-) Seems like it ought to move forward now. Oh yea

Re: update docs for --enable-languages

2015-05-08 Thread Jeff Law
On 05/07/2015 04:15 PM, Jim Wilson wrote: ping https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01690.html OK. Must have missed it when it flew by -- sorry. jeff

  1   2   >