Re: Add a param to decide stack slot sharing at -O0

2013-10-09 Thread Eric Botcazou
Seems like a odd thing for a param. If the compile time increase is very small ( 1%?) I would just make the new threshold default. I don't understand the 'odd' here... params are exactly for this purpose, i.e. to avoid hardcoding magic numbers in the compiler, so please elaborate. -- Eric

[Committed] S/390: Use FPRs as GPR save slots

2013-10-09 Thread Andreas Krebbel
Hi, with the attached patch we use call-clobbered floating point registers as save slots for general purpose registers in leaf functions. Bootstrapped and regtested with various options and -march levels. Committed to mainline. Bye, -Andreas- 2013-10-09 Andreas Krebbel

[Committed] S/390: Cleanup of s390_frame_info

2013-10-09 Thread Andreas Krebbel
Just a cleanup of the s390_frame_info function. Committed to mainline after regression test was fine. Bye, -Andreas- 2013-10-09 Andreas Krebbel andreas.kreb...@de.ibm.com * config/s390/s390.c (s390_frame_info): Restructure function. --- gcc/config/s390/s390.c | 112

[Committed] S/390: Use fix stack slots for FPRs saved due to stdarg

2013-10-09 Thread Andreas Krebbel
Hi, for stdarg the floating point argument registers in the register save area must reside at an ABI-defined offset relative to the start of the save area. So far were not able to optimize away save instructions with the packed stack layout since this changed the offsets. With the attached

Re: Cleanup patches

2013-10-09 Thread Thomas Schwinge
Hi! On Tue, 8 Oct 2013 22:04:23 +0200, Jakub Jelinek ja...@redhat.com wrote: On Tue, Oct 08, 2013 at 09:17:35AM +0200, Thomas Schwinge wrote: Here are a few cleanup patches, mostly in the realm of OpenMP, so Jakub gets a CC. OK to commit? They look ok to me, but I'd prefer if they could

Re: Cleanup patches

2013-10-09 Thread Jakub Jelinek
On Wed, Oct 09, 2013 at 10:56:25AM +0200, Thomas Schwinge wrote: On Tue, 8 Oct 2013 22:04:23 +0200, Jakub Jelinek ja...@redhat.com wrote: On Tue, Oct 08, 2013 at 09:17:35AM +0200, Thomas Schwinge wrote: Here are a few cleanup patches, mostly in the realm of OpenMP, so Jakub gets a CC. OK

Re: Cleanup patches

2013-10-09 Thread Tobias Burnus
Thomas Schwinge wrote: Meanwhile, here's another series that I assumed had gotten lost, but now recovered thanks to Âgit fsckÂ. Also OK to commit? _OPENMP pre-processor checks, openmp_version Fortran instrinsic checks. Two remarks: * When Jakub's patch gets it, one needs to update the C/C++

[4.8] Backported fix for PR tree-optimization/58539

2013-10-09 Thread Jakub Jelinek
Hi! I've backported the following patch to 4.8 branch, bootstrapped/regtested on x86_64-linux and i686-linux, committed. 2013-10-09 Jakub Jelinek ja...@redhat.com Backport from mainline 2013-09-26 Richard Biener rguent...@suse.de PR tree-optimization/58539 *

[4.8] Fix OMP_THREAD_LIMIT libgomp handling

2013-10-09 Thread Jakub Jelinek
Hi! This is the minimal fix for the issue mentioned in http://gcc.gnu.org/ml/gcc-patches/2013-10/msg00333.html that is being fixed as part of gomp-4_0-branch merge to trunk, but for 4.8 I've applied this fix instead. The testcase also includes the follow-up fix from

Re: Cleanup patches

2013-10-09 Thread Thomas Schwinge
Hi! On Wed, 9 Oct 2013 11:04:29 +0200, Jakub Jelinek ja...@redhat.com wrote: The testing _OPENMP value against 201107 won't work after merge, it will be 201307 instead. The Fortran openmp_version will be still 201107 for now, as Fortran OpenMP 4.0 support isn't there yet. And: On Wed, 9 Oct

RE: Fix scheduler ix86_issue_rate and ix86_adjust_cost for modern x86 chips

2013-10-09 Thread Gopalasubramanian, Ganesh
Hi Honza, Yep, I think we need to merge only those autmatas tha are same for both: (define_automaton bdver3,bdver3_ieu,bdver3_load,bdver3_fp,bdver3_agu) probably can become (define_automaton bdver3,bdver3_fp) with the corresponding reservations using bdver3_ieu,bdver3_load,bdver3_agu

Re: [patch] Add tree-ssa-loop.h and friends.

2013-10-09 Thread Richard Biener
On Tue, Oct 8, 2013 at 4:15 PM, Andrew MacLeod amacl...@redhat.com wrote: On 10/08/2013 09:18 AM, Richard Biener wrote: On Tue, Oct 8, 2013 at 2:58 PM, Andrew MacLeod amacl...@redhat.com wrote: I just took a quick stab at it... I think its pretty involved and someone with better loop

Re: [PATCH i386 3/8] [AVX512] [2/n] Add AVX-512 patterns: Fix missing `v' constraint.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 2nd subpatch. It fixes missing `v' constraints. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec 2000

Re: [PATCH i386 3/8] [AVX512] [3/n] Add AVX-512 patterns: VF1 and VI iterators.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 3rd subpatch. It extends VF1 and VI iterators. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec 2000

Re: [PATCH i386 3/8] [AVX512] [4/n] Add AVX-512 patterns: V iterator.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 4th subpatch. It extends V iterator. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec 2000 2006 build

Re: [PATCH i386 3/8] [AVX512] [6/n] Add AVX-512 patterns: VI2 and VI124 iterators.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 6th subpatch. It extends VI2 and VI124 iterators. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec 2000

Re: [PATCH i386 3/8] [AVX512] [5/n] Add AVX-512 patterns: Introduce `multdiv' code iterator.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 5th subpatch. It introduces `multdiv' code iterator. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec

Re: [PATCH i386 3/8] [AVX512] [7/n] Add AVX-512 patterns: VI4 and VI8 iterators.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 7th subpatch. It extends VI4 and VI8 iterators. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec 2000

Re: [PATCH i386 3/8] [AVX512] [8/n] Add AVX-512 patterns: VI48 and VI48_AVX2 iterators.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 8th subpatch. It extends VI48 and VI48_AVX2 iterators. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec

Re: [PATCH i386 3/8] [AVX512] [10/n] Add AVX-512 patterns: VI248_AVX2_8_AVX512F and VI124_256_48_AVX512F iterators.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 10th subpatch. It introduces VI248_AVX2_8_AVX512F and VI124_256_48_512 iterators. Is it Ok? Testing: 1. Bootstrap pass. 2. make check

Re: [PATCH i386 3/8] [AVX512] [9/n] Add AVX-512 patterns: VI124_AVX2, VI8F iterators.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 9th subpatch. It extends VI124_AVX2_48 and VI8F iterators. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3.

Re: [PATCH i386 3/8] [AVX512] [11/n] Add AVX-512 patterns: FMA.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 11th subpatch. It introduces AVX-512 FMA instructions. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec

Re: [PATCH i386 3/8] [AVX512] [13/n] Add AVX-512 patterns: VI4_AVX iterator.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 13th subpatch. It introduces VI4_AVX iterator. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec 2000

Re: [PATCH i386 3/8] [AVX512] [12/n] Add AVX-512 patterns: V_512 and VI_512 iterators.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 12th subpatch. It introduces VF_512 and VI_512 iterators. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3.

Re: [PATCH i386 3/8] [AVX512] [15/n] Add AVX-512 patterns: VI48F_512 iterator.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 15th subpatch. It introduces VI48F_512 iterator. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec 2000

Re: [PATCH i386 3/8] [AVX512] [14/n] Add AVX-512 patterns: VI48F_256_512 iterator.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 14th subpatch. It introduces VI48F_256_512 iterator. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec

Re: [PATCH i386 3/8] [AVX512] [17/n] Add AVX-512 patterns: V8FI and V16FI iterators.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 17th subpatch. It introduces V8FI and V16FI iterators. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec

Re: [PATCH i386 3/8] [AVX512] [18/n] Add AVX-512 patterns: various RCPs and SQRTs.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 18th subpatch. It introduces various new insns. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec 2000

Re: [PATCH i386 3/8] [AVX512] [20/n] Add AVX-512 patterns: Misc.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 20th subpatch. It introduces last insns of AVX-512F. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec

Re: [PATCH i386 3/8] [AVX512] [19/n] Add AVX-512 patterns: Extracts and converts.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 19th subpatch. It extends extract and convert insn patterns. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no regressions.

Re: [PATCH i386 3/8] [AVX512] [16/n] Add AVX-512 patterns: VI48_512 and VI4F_128 iterators.

2013-10-09 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Here's 1st subpatch. It extends VI4F_128 and introduces VI48_512 iterator. Is it Ok? Testing: 1. Bootstrap pass. 2. make check shows no

Re: [PATCH][ARM]Replace gen_rtx_PLUS with plus_constant

2013-10-09 Thread Kyrill Tkachov
On 01/10/13 11:15, Marcus Shawcroft wrote: On 30 September 2013 14:23, Renlin Li renlin...@arm.com wrote: OK for trunk? Kind regards, Renlin Li gcc/ChangeLog: 2013-09-30 Renlin Li renlin...@arm.com * config/arm/arm.c (arm_output_mi_thunk): Use plus_constant. OK /Marcus Hi Renlin,

Re: [patch] Fix PR middle-end/58570

2013-10-09 Thread Richard Biener
On Tue, Oct 8, 2013 at 7:52 PM, Eric Botcazou ebotca...@adacore.com wrote: Probably because the actual accesses may overlap if we choose to perform a bigger access. Nope, simply because they share a byte. The same can happen if we for struct { char c1; char c2; } perform an HImode access in

Re: [PATCH][AArch64] Vneg NEON intrinsics modified

2013-10-09 Thread Marcus Shawcroft
On 8 October 2013 17:10, Alex Velenko alex.vele...@arm.com wrote: gcc/testsuite/ 2013-10-08 Alex Velenko alex.vele...@arm.com * gcc.target/aarch64/vneg_f.c: New testcase. * gcc.target/aarch64/vneg_s.c: New testcase. gcc/ 2013-10-08 Alex Velenko alex.vele...@arm.com

Re: [patch] The remainder of tree-flow.h refactored.

2013-10-09 Thread Richard Biener
On Wed, Oct 9, 2013 at 1:31 AM, Andrew MacLeod amacl...@redhat.com wrote: On 10/08/2013 07:44 AM, Andrew MacLeod wrote: On 10/08/2013 06:22 AM, Richard Biener wrote: graphite.h should be unnecessary with moving the pass struct like you did for other loop opts. Likewise tree-parloops.h

Re: Add a param to decide stack slot sharing at -O0

2013-10-09 Thread Richard Biener
On Tue, Oct 8, 2013 at 11:04 PM, Easwaran Raman era...@google.com wrote: In cfgexpand.c, variables in non-overlapping lexical scopes are assigned same stack locations at -O1 and above. At -O0, this is attempted only if the size of the stack objects is above a threshold (32). The rationale is

[C++ Patch] PR 58633 (Take 2)

2013-10-09 Thread Paolo Carlini
Hi, this is a completely different approach at fixing the bug, which overall I like better. In this case most of the patch touches cp_parser_decltype_expr: instead of using cp_parser_postfix_expression only for member access expressions, we accept all its valid return values (still

Re: [patch] Fix PR middle-end/58570

2013-10-09 Thread Eric Botcazou
In my opinion the MEM_EXPR is wrong, as it is supposed to be the tree equivalent of the memory access. At gimple level we handle accesses at bit-granularity so bit-accesses are fine. Not so at RTL level it seems. [this also shows we probably should lower bit-granular accesses at the

Re: [gomp4] Adjust some gcc.dg/autopar/ tests

2013-10-09 Thread Thomas Schwinge
Hi! On Tue, 8 Oct 2013 17:24:14 +0200, Jakub Jelinek ja...@redhat.com wrote: These tests were expecting 5 loopfn matches, 3 on the fn definition, one as GOMP_parallel_start argument and one called in between GOMP_parallel_start and GOMP_parallel_end. But the new API is to call GOMP_parallel

Re: [PATCH][AARCH64] Vdiv NEON intrinsic

2013-10-09 Thread Marcus Shawcroft
On 8 October 2013 17:25, Alex Velenko alex.vele...@arm.com wrote: gcc/testsuite/ 2013-09-10 Alex Velenko alex.vele...@arm.com * gcc.target/aarch64/vdiv_f.c: New testcase. gcc/ 2013-09-10 Alex Velenko alex.vele...@arm.com * config/aarch64/arm_neon.h

Re: [PATCH][AArch64] NEON vadd_f64 and vsub_f64 intrinsics modified

2013-10-09 Thread Marcus Shawcroft
On 8 October 2013 17:35, Alex Velenko alex.vele...@arm.com wrote: 2013-10-08 Alex Velenko alex.vele...@arm.com * gcc.target/aarch64/vadd_f64.c: New testcase. * gcc.target/aarch64/vsub_f64.c: New testcase. gcc/ 2013-10-08 Alex Velenko alex.vele...@arm.com *

Re: [patch] Fix PR middle-end/58570

2013-10-09 Thread Richard Biener
On Wed, Oct 9, 2013 at 1:36 PM, Eric Botcazou ebotca...@adacore.com wrote: In my opinion the MEM_EXPR is wrong, as it is supposed to be the tree equivalent of the memory access. At gimple level we handle accesses at bit-granularity so bit-accesses are fine. Not so at RTL level it seems.

Re: [PATCH][AArch64] NEON vclz intrinsic modified

2013-10-09 Thread Marcus Shawcroft
On 8 October 2013 17:45, Alex Velenko alex.vele...@arm.com wrote: 2013-10-08 Alex Velenko alex.vele...@arm.com * gcc.target/aarch64/vclz.c: New testcase. gcc/ 2013-10-08 Alex Velenko alex.vele...@arm.com * config/aarch64/arm_neon.h (vclz_s8): Asm replaced with

Patch to split out new warning flag for floating point conversion

2013-10-09 Thread Joshua J Cogliati
== Administrivia == This is my first patch. I have emailed in the signed copyright transfer documents already. == Description == This patch is a fix for Bug 53001 As required by the C and C++ standards, gcc automatically converts floating point numbers to lower precision or integer values.

Re: [patch] Fix PR middle-end/58570

2013-10-09 Thread Eric Botcazou
Well, ok. Please adjust the comment + /* Different fields of the same record type cannot overlap, unless they +are both bitfields and we are at the RTL level. */ to sth like ??? Bitfields can overlap at RTL level so punt if we end up at them. or sth

Re: Fix scheduler ix86_issue_rate and ix86_adjust_cost for modern x86 chips

2013-10-09 Thread Jan Hubicka
Before merging the insn reservations, I need to compare the latency values for bdver1 and bdver3. I know that they are different for some of the instructions. In that case, the merging should prop up another subset of latency differences. I would like to keep these insn reservations in

[committed] Fix up bb-slp-31.c testcase

2013-10-09 Thread Jakub Jelinek
Hi! I've noticed that this testcase doesn't clean up after itself. Fixed thusly, committed as obvious to trunk. 2013-10-09 Jakub Jelinek ja...@redhat.com * gcc.dg/vect/bb-slp-31.c: Add cleanup-tree-dump. --- gcc/testsuite/gcc.dg/vect/bb-slp-31.c.jj2013-08-30 14:38:40.0

Re: [C++ Patch] PR 58633

2013-10-09 Thread Jason Merrill
On 10/08/2013 07:03 PM, Paolo Carlini wrote: .. a curiosity: the cp_parser_commit_to_tentative_parse at the end of cp_parser_pseudo_destructor_name, which didn't exist in 4.6.x and we can consider the root of this issue, is also my fault:

[PATCH] Fix libstdc++/58659.cc test case.

2013-10-09 Thread Marcus Shawcroft
The test case add here: http://gcc.gnu.org/ml/gcc-patches/2013-10/msg00474.html Introduced an unprototyped call to abort() resulting in failures due to unexepected warnings in aarch64-none-elf cross testing. Committed to trunk as obvious. Cheers /Marcus 2013-10-09 Marcus Shawcroft

[PATCH][ARM][committed] Move comment to correct place below #includes

2013-10-09 Thread Kyrill Tkachov
Hi all, I've committed this trivial patch as r203322. It moves the comment for the arm_early_load_addr_dep from above the #includes in the file to above the function definition. Thanks, Kyrill 2013-10-09 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/aarch-common.c

Re: [C++ PATCH] PR58635

2013-10-09 Thread Jason Merrill
OK. Jason

Re: [C++ Patch] PR 58633

2013-10-09 Thread Paolo Carlini
Hi, On 10/09/2013 03:56 PM, Jason Merrill wrote: On 10/08/2013 07:03 PM, Paolo Carlini wrote: .. a curiosity: the cp_parser_commit_to_tentative_parse at the end of cp_parser_pseudo_destructor_name, which didn't exist in 4.6.x and we can consider the root of this issue, is also my fault:

Re: [gomp4] C++ OpenMP user defined reductions (take 2)

2013-10-09 Thread Jason Merrill
On 10/07/2013 07:02 AM, Jakub Jelinek wrote: duplicates anywhere, but during error diagnostics. Without those two decl.c hunks (either of them), pushdecl will sometimes return a different decl from the original or error_mark_node, and the original fndecl passed to it has ggc_free called on it,

Re: [PATCH] Fix libstdc++/58659.cc test case.

2013-10-09 Thread Jonathan Wakely
On 9 October 2013 15:26, Marcus Shawcroft wrote: The test case add here: http://gcc.gnu.org/ml/gcc-patches/2013-10/msg00474.html Introduced an unprototyped call to abort() resulting in failures due to unexepected warnings in aarch64-none-elf cross testing. Committed to trunk as obvious.

Re: Ping Re: [gomp4] Dumping gimple for offload.

2013-10-09 Thread Ilya Tocar
Ping. On 03 Oct 20:05, Ilya Tocar wrote: On 26 Sep 21:21, Ilya Tocar wrote: On 25 Sep 15:48, Richard Biener wrote: On Wed, Sep 25, 2013 at 3:29 PM, Ilya Tocar tocarip.in...@gmail.com wrote: On 24 Sep 11:02, Richard Biener wrote: On Mon, Sep 23, 2013 at 3:29 PM, Ilya Tocar

Re: [PATCH] Fix libstdc++/58659.cc test case.

2013-10-09 Thread Jakub Jelinek
On Wed, Oct 09, 2013 at 04:12:21PM +0100, Jonathan Wakely wrote: On 9 October 2013 15:26, Marcus Shawcroft wrote: The test case add here: http://gcc.gnu.org/ml/gcc-patches/2013-10/msg00474.html Introduced an unprototyped call to abort() resulting in failures due to unexepected

[patch] fix broken loop-doloop.c

2013-10-09 Thread Andrew MacLeod
Most of loop-doloop.c is wrapped by #ifdef HAVE_doloop_end ... #endif so on my bootstraps/builds, it doesn't exercise the code. It was pointed out to me the restructuring broke this file on some targets. This should fix it by calling the refactored get_max_loop_iterations() function. cross

Re: [patch] The remainder of tree-flow.h refactored.

2013-10-09 Thread Andrew MacLeod
On 10/08/2013 06:22 AM, Richard Biener wrote: unvisit_body isn't generic enough to warrant moving out of gimplify.c (the only user). Bah, now I remember.. so there *are* other users.. this routine is called from various front ends.. fortran, c-family and cp all call it. That is why I wanted

Re: [patch] The remainder of tree-flow.h refactored.

2013-10-09 Thread Diego Novillo
On Wed, Oct 9, 2013 at 11:37 AM, Andrew MacLeod amacl...@redhat.com wrote: bootstraps on x86_64-unknown-linux-gnu, regressions test are still running. OK? Sure.

Re: [PATCH]: Fix PR58542, Arguments of __atomic_* functions are converted in unsigned mode

2013-10-09 Thread Richard Henderson
On 10/08/2013 11:37 AM, Uros Bizjak wrote: As shown in the attached testcase, arguments of various __atomic builtins should be converted as signed, so the immediates get properly extended. 2013-10-08 Uros Bizjak ubiz...@gmail.com * optabs.c (maybe_emit_atomic_exchange): Convert

Re: [gomp4] C++ OpenMP user defined reductions (take 2)

2013-10-09 Thread Jakub Jelinek
On Wed, Oct 09, 2013 at 10:47:22AM -0400, Jason Merrill wrote: On 10/07/2013 07:02 AM, Jakub Jelinek wrote: duplicates anywhere, but during error diagnostics. Without those two decl.c hunks (either of them), pushdecl will sometimes return a different decl from the original or error_mark_node,

Re: [patch] The remainder of tree-flow.h refactored.

2013-10-09 Thread Richard Biener
Andrew MacLeod amacl...@redhat.com wrote: On 10/08/2013 06:22 AM, Richard Biener wrote: unvisit_body isn't generic enough to warrant moving out of gimplify.c (the only user). Bah, now I remember.. so there *are* other users.. this routine is called from various front ends.. fortran, c-family

Re: Add a param to decide stack slot sharing at -O0

2013-10-09 Thread Easwaran Raman
On Wed, Oct 9, 2013 at 4:11 AM, Richard Biener richard.guent...@gmail.com wrote: On Tue, Oct 8, 2013 at 11:04 PM, Easwaran Raman era...@google.com wrote: In cfgexpand.c, variables in non-overlapping lexical scopes are assigned same stack locations at -O1 and above. At -O0, this is attempted

[patch] shuffle a couple of functions.

2013-10-09 Thread Andrew MacLeod
This patch simply moves std_gimplify_va_arg_expr() and the related build_va_arg_indirect_ref to gimplify.c where I think it belongs. It also moves gimple_fold_indirect_ref out of gimplfy.c and into gimple-fold.c. bootstraps on x86_64-unknown-linux-gnu... regressions running. OK? Andrew *

Re: [patch] The remainder of tree-flow.h refactored.

2013-10-09 Thread Andrew MacLeod
On 10/09/2013 01:48 PM, Richard Biener wrote: Andrew MacLeod amacl...@redhat.com wrote: On 10/08/2013 06:22 AM, Richard Biener wrote: unvisit_body isn't generic enough to warrant moving out of gimplify.c (the only user). Bah, now I remember.. so there *are* other users.. this routine is

Re: [patch] The remainder of tree-flow.h refactored.

2013-10-09 Thread Andrew MacLeod
On 10/09/2013 02:15 PM, Andrew MacLeod wrote: On 10/09/2013 01:48 PM, Richard Biener wrote: Andrew MacLeod amacl...@redhat.com wrote: On 10/08/2013 06:22 AM, Richard Biener wrote: unvisit_body isn't generic enough to warrant moving out of gimplify.c (the only user). Bah, now I remember.. so

Re: [PATCH] Fix libstdc++/58659.cc test case.

2013-10-09 Thread Jonathan Wakely
On 9 October 2013 16:15, Jakub Jelinek wrote: On Wed, Oct 09, 2013 at 04:12:21PM +0100, Jonathan Wakely wrote: On 9 October 2013 15:26, Marcus Shawcroft wrote: The test case add here: http://gcc.gnu.org/ml/gcc-patches/2013-10/msg00474.html Introduced an unprototyped call to abort()

[Google 4.8 Patch] Generate gnu-pubnames for definitions only. Not declarations.

2013-10-09 Thread Sterling Augustine
Hi Cary, The enclosed patch stops declaration dies from generating gnu-pubnames. Declaration dies don't include enough information to be useful to gdb, so it has to scan all the entries in the index until it finds a definition. Not including declarations prevents this problem. Google ref:

Re: [PATCH] Enhance phiopt to handle BIT_AND_EXPR

2013-10-09 Thread Jeff Law
On 09/30/13 03:29, Zhenqiang Chen wrote: Hi, The patch enhances phiopt to handle cases like: if (a == 0 (...)) return 0; return a; Boot strap and no make check regression on X86-64 and ARM. Is it OK for trunk? Thanks! -Zhenqiang ChangeLog: 2013-09-30 Zhenqiang Chen

Re: [PATCH] Enhance phiopt to handle BIT_AND_EXPR

2013-10-09 Thread Jeff Law
On 10/08/13 21:39, Zhenqiang Chen wrote: I'll go ahead and pull the common bits into a single function and commit on Zhenqiang's behalf. Thank you! No problem. If you think you'll be contributing regularly, you might want to go ahead and ask for write-after-approval privileges. jeff

Re: [PATCH][RFC] fix reload causing ICE in subreg_get_info on m68k (PR58369)

2013-10-09 Thread Jeff Law
On 09/28/13 09:30, Mikael Pettersson wrote: This patch fixes PR58369, an ICE in subreg_get_info when compiling boost for m68k-linux. choose_reload_regs attempts to reload a DFmode (8-byte) reg, finds an XFmode (12-byte) reg in last_reg, and calls subreg_regno_offset with these two modes and a

Re: [PING] 3 patches waiting for approval/review

2013-10-09 Thread Jeff Law
On 08/21/13 03:21, Andreas Krebbel wrote: [RFC] Allow functions calling mcount before prologue to be leaf functions http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00993.html I don't think this is necessarily correct for all targets. ISTM the ability to consider a function calling mcount as a

Re: [PATCH]: Fix PR58542, Arguments of __atomic_* functions are converted in unsigned mode

2013-10-09 Thread Uros Bizjak
On Wed, Oct 9, 2013 at 6:23 PM, Richard Henderson r...@redhat.com wrote: This doesn't seem right at all. The bug is that I gets set to UINT64_MAX, right? Where's the incorrect conversion from int to __int128_t? Surely you can Please see Comment #5 of PR58542: --cut here-- The problem

Re: Patch to split out new warning flag for floating point conversion

2013-10-09 Thread Joseph S. Myers
On Wed, 9 Oct 2013, Joshua J Cogliati wrote: Because this changes -Wextra, when compiling with -Werror and -Wextra, some code will not compile now. The code in gcc that this occurred in was changed to use explicit casts. The patch would be shorter if I think those changes should be

Re: Patch to split out new warning flag for floating point conversion

2013-10-09 Thread Joseph S. Myers
Also note that this patch needs to add testcases to the testsuite (gcc/testsuite/c-c++-common/, probably) testing what cases generate warnings with the new option and what cases don't. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH i386 3/8] [AVX512] [2/n] Add AVX-512 patterns: Fix missing `v' constraint.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:24 AM, Kirill Yukhin wrote: Here's 2nd subpatch. It fixes missing `v' constraints. And one v constraint that shouldn't have been. Ok. r~

Re: [PATCH i386 3/8] [AVX512] [3/n] Add AVX-512 patterns: VF1 and VI iterators.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:24 AM, Kirill Yukhin wrote: Here's 3rd subpatch. It extends VF1 and VI iterators. Ok. r~

Re: [PATCH i386 3/8] [AVX512] [4/n] Add AVX-512 patterns: V iterator.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:25 AM, Kirill Yukhin wrote: Here's 4th subpatch. It extends V iterator. And much much more that's totally unrelated to changing V. That said, I didn't see anything wrong in there. Ok. r~

Re: [patch] The remainder of tree-flow.h refactored.

2013-10-09 Thread Andrew MacLeod
On 10/09/2013 02:18 PM, Andrew MacLeod wrote: On 10/09/2013 02:15 PM, Andrew MacLeod wrote: Fair enough. I'll adjust... the front end files which use that routine will just have to include gimplify.h Unless maybe we should expand the gimplify module to have a gimplfy-fe.[ch] which

Re: [PATCH i386 3/8] [AVX512] [5/n] Add AVX-512 patterns: Introduce `multdiv' code iterator.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:25 AM, Kirill Yukhin wrote: Here's 5th subpatch. It introduces `multdiv' code iterator. This is the sort of patch I like to see. It's the first one you've sent that's done exactly one thing. Congratulations. Ok. r~

Re: [PATCH i386 3/8] [AVX512] [6/n] Add AVX-512 patterns: VI2 and VI124 iterators.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:26 AM, Kirill Yukhin wrote: Here's 6th subpatch. It extends VI2 and VI124 iterators. Ok. r~

Re: [PATCH i386 3/8] [AVX512] [7/n] Add AVX-512 patterns: VI4 and VI8 iterators.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:26 AM, Kirill Yukhin wrote: Here's 7th subpatch. It extends VI4 and VI8 iterators. Ok. r~

Re: [PATCH i386 3/8] [AVX512] [8/n] Add AVX-512 patterns: VI48 and VI48_AVX2 iterators.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:27 AM, Kirill Yukhin wrote: Here's 8th subpatch. It extends VI48 and VI48_AVX2 iterators. Ok. r~

Re: [PATCH i386 3/8] [AVX512] [9/n] Add AVX-512 patterns: VI124_AVX2, VI8F iterators.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:27 AM, Kirill Yukhin wrote: Here's 9th subpatch. It extends VI124_AVX2_48 and VI8F iterators. Ok. r~

Re: [Google 4.8 Patch] Generate gnu-pubnames for definitions only. Not declarations.

2013-10-09 Thread Cary Coutant
2013-10-09 Sterling Augustine saugust...@google.com * dwarf2out.c (include_pubname_in_output): Add conditional on is_declaration_die and debug_generate_pubnames. OK for google 4.8 branch. Thanks! -cary

[Patch, Fortran] PR58226 - Avoid invalid mem access with compiler_options

2013-10-09 Thread Tobias Burnus
A rather obvious fix; the memory is freed by the caller (gfc_simplify_compiler_options). It is unlikely that the compiler has no arguments as the driver tends to send some, e.g. -mtune=generic -march=x86-64 on my system. However, it is better to be safe than sorry. Build and regtested on

Re: [PATCH i386 3/8] [AVX512] [10/n] Add AVX-512 patterns: VI248_AVX2_8_AVX512F and VI124_256_48_AVX512F iterators.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:27 AM, Kirill Yukhin wrote: Here's 10th subpatch. It introduces VI248_AVX2_8_AVX512F and VI124_256_48_512 iterators. Ok. r~

Re: [PATCH i386 3/8] [AVX512] [11/n] Add AVX-512 patterns: FMA.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:28 AM, Kirill Yukhin wrote: +;; CPUID bit AVX512F enables evex encoded scalar and 512-bit fma. It doesn't +;; care about FMA bit, so we enable fma for TARGET_AVX512F even when TARGET_FMA +;; and TARGET_FMA4 are both false. How do you force an evex encoding of the

Re: [PATCH i386 3/8] [AVX512] [12/n] Add AVX-512 patterns: V_512 and VI_512 iterators.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:28 AM, Kirill Yukhin wrote: Here's 12th subpatch. It introduces VF_512 and VI_512 iterators. Ok. r~

Re: [PATCH i386 3/8] [AVX512] [13/n] Add AVX-512 patterns: VI4_AVX iterator.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:29 AM, Kirill Yukhin wrote: Here's 13th subpatch. It introduces VI4_AVX iterator. Ok. r~

Re: [PATCH i386 3/8] [AVX512] [14/n] Add AVX-512 patterns: VI48F_256_512 iterator.

2013-10-09 Thread Richard Henderson
On 10/09/2013 03:29 AM, Kirill Yukhin wrote: Here's 14th subpatch. It introduces VI48F_256_512 iterator. Ok. r~

[RFC] [Testsuite,ARM] Neon intrinsics executable tests

2013-10-09 Thread Christophe Lyon
Hi, This patch is a first small sample of dejagnu-ization of my ARM Neon intrinsics tests. It's derived from my previous work at http://gitorious.org/arm-neon-tests/arm-neon-tests which supports all the ARM intrinsics, with executable tests. As I have to manually transform each test (to include

Go patch committed: Fix complex division of NaN / 0

2013-10-09 Thread Ian Lance Taylor
This patch to the Go frontend and libgo fixes complex division of NaN / 0. Go expects that to produce NaN. When using libgcc it could produce Inf. Specifically NaN+1i / 0+0i produced NaN+Infi, which by the rules of C99 Annex G is Inf. This may be correct for C, but it's not correct for Go.

Re: [Patch, Fortran] PR58226 - Avoid invalid mem access with compiler_options

2013-10-09 Thread Steve Kargl
On Wed, Oct 09, 2013 at 11:51:30PM +0200, Tobias Burnus wrote: A rather obvious fix; the memory is freed by the caller (gfc_simplify_compiler_options). It is unlikely that the compiler has no arguments as the driver tends to send some, e.g. -mtune=generic -march=x86-64 on my system.

[PATCH] Workaround errata for the PMC-Sierra RM7000 cpu.

2013-10-09 Thread Moore, Catherine
Hi Richard, This patch implements a workaround for errors on the PMC-Sierra RM7000 cpu while executing the dmult or dmultu instruction. The workaround is to insert three nops after the dmult/dmultu. Does this look okay to commit? Thanks, Catherine gcc/ 2013-10-09 Catherine Moore

[PATCH, rs6000] Fix variable permute control vectors for little endian

2013-10-09 Thread Bill Schmidt
Hi, This is a follow-up to the recent patch that fixed constant permute control vectors for little endian. When the control vector is constant, we can adjust the constant and use a vperm without increasing code size. When the control vector is unknown, however, we have to generate two additional

[GOOGLE] AutoFDO profile propagation should use all dominators

2013-10-09 Thread Dehao Chen
This patch updates the AutoFDO profile propagation of equivalence class: instead of looking just immediate dominators, traverse all dominators. This helps improving profile accuracy. Bootstrapped and passed regression test. OK for google-4_8 branch? Thanks, Dehao Index: gcc/auto-profile.c

Re: [GOOGLE] AutoFDO profile propagation should use all dominators

2013-10-09 Thread Xinliang David Li
ok. David On Wed, Oct 9, 2013 at 4:47 PM, Dehao Chen de...@google.com wrote: This patch updates the AutoFDO profile propagation of equivalence class: instead of looking just immediate dominators, traverse all dominators. This helps improving profile accuracy. Bootstrapped and passed

[GOOGLE] Represent the callsite with lineno only for AutoFDO

2013-10-09 Thread Dehao Chen
In legacy AutoFDO, callsite is represented as a (lineno, callee_name) pair because there could be multiple calls in one line. However, as we enhanced the debug info by assigning discriminators for each function call in the same line, callee_name is not needed when indexing the callsite. This

Go patch committed: A type conversion of a constant need not be const

2013-10-09 Thread Ian Lance Taylor
In Go a type conversion of a constant is not necessarily itself a constant, as in []byte(nil). This patch fixes the Go frontend to recognize that. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Will commit to 4.8 branch when it reopens. Ian diff -r

  1   2   >