[PATCH, AArch64] Fix abitest for ilp32

2015-01-13 Thread Hurugalawadi, Naveen
Hi, Please find attached the patch that fixes abitest for ilp32. testfunc_ptr is a 32bit pointer in ILP32 but is being loaded as 64bit. Hence some of the func-ret testcases FAIL's for ILP32. Please review the patch and let us know if its okay? Regression tested on aarch64-elf. Thanks, Naveen

Drop workaround for old binutils linker plugin bug

2015-01-13 Thread Jan Hubicka
Hi, this workaround actually triggers bug in quite recent golds, so it seems to be good motivation to finally drop it. The bug is long fixed. Bootstrapped/regtested x86_64-linux, will commit it shortly. Honza * tree-profile.c (init_ic_make_global_vars): Drop workaround for

PR 64481 (bootstrap miscompare)

2015-01-13 Thread Jan Hubicka
Hi, in December I conditoinally disabled expensive sanity checking in inliner. This triggeres bootstrap miscompare because caches are getting out of sync. This patch fixes the problem found by sanity check - the node growth cache was removed from use in badness calculation by Richard a while ago,

Re: [PATCH 0/6, nds32] Committed: Have -mcmodel option to support varied code model of programs.

2015-01-13 Thread Chung-Ju Wu
2015-01-14 6:22 GMT+08:00 Joseph Myers jos...@codesourcery.com: On Tue, 13 Jan 2015, Chung-Ju Wu wrote: To fix this issue, we are going to use -mcmodel=X options, which probably gives more flexibility to support varied code model on code generation. The -mgp-direct option now becomes

[PATCH] Fix PR c++/16160

2015-01-13 Thread Patrick Palka
This patch fixes the above PR where it was reported that the C++ frontend does not reject the malformed class declaration struct X5; Instead of rejecting it, the FE treats this declaration as if it were a forward declaration of a template specialization, i.e. as if it were written

Patch ping...

2015-01-13 Thread Jan Hubicka
Hi, I would like to ping the patch to fix divergence between a type and its main variant introduced by C++ FE. https://gcc.gnu.org/ml/gcc-patches/2014-12/msg01202.html Honza

Re: [PATCH/expand] PR64011 Adjust bitsize when partial overflow happen for big-endian

2015-01-13 Thread Jeff Law
On 01/13/15 15:42, Joseph Myers wrote: On Tue, 13 Jan 2015, Jeff Law wrote: In many ways having the compiler or assembler spitting out an error here is preferable to silently compiling the code. That would also help explain why As usual, an error is incorrect in such a case that only has

Re: [PATCH] Reenable CSE of non-volatile inline asm (PR rtl-optimization/63637)

2015-01-13 Thread Jeff Law
On 01/13/15 17:03, Segher Boessenkool wrote: On Tue, Jan 13, 2015 at 03:17:08PM -0700, Jeff Law wrote: And finally there is the case of non-volatile asm with memory clobber with no memory stores in between the two - the posted (safer) patch will not allow to CSE the two, while in theory we

[PATCH, nds32] Committed: Remove some features that are not available yet in nds32 port of GNU binutils package.

2015-01-13 Thread Chung-Ju Wu
Hi, all, The nds32 target supports two features, fp-as-gp and ex9, designed for code size optimizations. They are majorly performed by linker so that compiler is merely to give some hints or directives with -mforce-fp-as-gp, -mforbid-fp-as-gp, and -mex9 options. However, those two features are

Re: [committed] Update copyright years, part 2

2015-01-13 Thread Richard Sandiford
Jakub Jelinek ja...@redhat.com writes: Patch too large to attach uncompressed, this has been created with update-copyright.py --this-year. Note, I had to temporarily move away gcc/jit/docs/conf.py, the python script dies on that and leaves almost all files unchanged. Thanks for doing the

Re: shift/extract SHIFT_COUNT_TRUNCATED combine bug

2015-01-13 Thread Segher Boessenkool
On Tue, Jan 13, 2015 at 10:51:27AM +0100, Richard Biener wrote: IMHO SHIFT_COUNT_TRUNCATED should be removed and instead backends should provide shift patterns with a (and:QI ...) for the shift amount which simply will omit that operation if suitable. Note that that catches less though, e.g.

[PATCH][AArch64 Intrinsics] Replace temporary assembler for vst1_lane

2015-01-13 Thread Alan Lawrence
Nowadays, just storing the (bigendian-corrected) vector element to the address, generates exactly the same assembler for all cases except {float,int,uint}64x1_t, where st1 {v0.d}[0], [x0] becomes str d0, [x0] This is not a problem, and the change will be much better for optimization through

Re: [committed] Update copyright years, part 2

2015-01-13 Thread Jakub Jelinek
On Tue, Jan 13, 2015 at 05:06:35PM +, Richard Sandiford wrote: Jakub Jelinek ja...@redhat.com writes: Patch too large to attach uncompressed, this has been created with update-copyright.py --this-year. Note, I had to temporarily move away gcc/jit/docs/conf.py, the python script dies

Re: [PATCH] [AArch64, NEON] Improve vpmaxX vpminX intrinsics

2015-01-13 Thread Tejas Belagod
On 09/12/14 08:17, Yangfei (Felix) wrote: On 28 November 2014 at 09:23, Yangfei (Felix) felix.y...@huawei.com wrote: Hi, This patch converts vpmaxX vpminX intrinsics to use builtin functions instead of the previous inline assembly syntax. Regtested with aarch64-linux-gnu on QEMU. Also

Re: [PATCH] add option to emit more array bounds warnigs

2015-01-13 Thread Martin Uecker
Mon, 12 Jan 2015 11:00:44 -0700 Jeff Law l...@redhat.com: On 11/11/14 23:13, Martin Uecker wrote: ... * gcc/tree-vrp.c (check_array_ref): Emit more warnings for warn_array_bounds = 2. * gcc/testsuite/gcc.dg/Warray-bounds-11.c: New test-case. * gcc/c-family/c.opt: New option

[PATCH] Allow MIPS call-saved-{4-6}.c tests to correctly run for micromips

2015-01-13 Thread Andrew Bennett
Hi, The call-saved-{4-6}.c tests in the mips testsuite fail for micromips. The reason is that micromips uses the swm and lwm instructions to save/restore the call-saved registers rather than using the sw and lw instructions. The swm and lwm instructions only list the range of registers to

RE: [PATCH] Fix for PR64081 in RTL loop unroller

2015-01-13 Thread Zamyatin, Igor
Is it really sufficient here to verify that all the defs are on latch predecessors, what about the case where there is a predecessor without a def. How do you guarantee domination in that case? ISTM that given the structure for the code you're writing that you'd want to verify that in

Re: [PATCH] Allow MIPS call-saved-{4-6}.c tests to correctly run for micromips

2015-01-13 Thread Maciej W. Rozycki
On Tue, 13 Jan 2015, Andrew Bennett wrote: The call-saved-{4-6}.c tests in the mips testsuite fail for micromips. The reason is that micromips uses the swm and lwm instructions to save/restore the call-saved registers rather than using the sw and lw instructions. The swm and lwm

Re: [PATCH] Reenable CSE of non-volatile inline asm (PR rtl-optimization/63637)

2015-01-13 Thread Jeff Law
On 01/13/15 09:18, Jakub Jelinek wrote: Hi! My PR60663 fix unfortunately stopped CSE of all inline-asms, even when they e.g. only have the clobbers added by default. This patch attempts to restore the old behavior, with the exceptions: 1) as always, asm volatile is not CSEd 2) inline-asm with

Re: [PATCH] add option to emit more array bounds warnigs

2015-01-13 Thread Jeff Law
On 01/13/15 17:40, Martin Uecker wrote: Jeff Law l...@redhat.com: On 01/13/15 10:34, Martin Uecker wrote: Mon, 12 Jan 2015 11:00:44 -0700 Jeff Law l...@redhat.com: On 11/11/14 23:13, Martin Uecker wrote: ... Has this patch been bootstrapped and regression tested, if so on what platform.

Re: [PATCH] PR59448 - Promote consume to acquire

2015-01-13 Thread Jeff Law
On 01/13/15 15:56, Andrew MacLeod wrote: On 01/13/2015 02:06 PM, Andrew MacLeod wrote: On 01/13/2015 01:38 PM, Torvald Riegel wrote: On Tue, 2015-01-13 at 10:11 -0500, Andrew MacLeod wrote: On 01/13/2015 09:59 AM, Richard Biener wrote: On Tue, Jan 13, 2015 at 3:56 PM, Andrew MacLeod

Re: [PATCH][rtlanal.c][BE][1/2] Fix vector load/stores to not use ld1/st1

2015-01-13 Thread Jeff Law
On 01/10/15 06:05, Richard Sandiford wrote: Sorry for the slow response. Jeff has approved the patch in the meantime, but I didn't want to go ahead and apply it while there was still disagreement... Thanks. I didn't realize there was a disagreement when I approved. Let's continue to hash this

Re: [PATCH] [AArch64, NEON] Improve vpmaxX vpminX intrinsics

2015-01-13 Thread Yangfei (Felix)
On 09/12/14 08:17, Yangfei (Felix) wrote: On 28 November 2014 at 09:23, Yangfei (Felix) felix.y...@huawei.com wrote: Hi, This patch converts vpmaxX vpminX intrinsics to use builtin functions instead of the previous inline assembly syntax. Regtested with aarch64-linux-gnu on

Re: [PATCH] Correct target selector in -mfentry tests

2015-01-13 Thread Jeff Law
On 01/13/15 14:27, H.J. Lu wrote: -fprofile -mfentry works with PIE if gcrt1.o is compiled with -fPIC. A glibc has been filed, PR 17836, and a glibc patch has been submitted. OK for trunk? Thanks. H.J. -- * gcc.target/i386/fentry-override.c: Properly place {} in target

Re: [PATCH] Allow MIPS call-saved-{4-6}.c tests to correctly run for micromips

2015-01-13 Thread Richard Sandiford
Maciej W. Rozycki ma...@linux-mips.org writes: On Tue, 13 Jan 2015, Matthew Fortune wrote: I have tested this for both mips and micromips, and the tests now pass successfully. The ChangeLog and patch are below. Hmm, instead of trying to avoid testing microMIPS code generation

Re: [PATCH][rtlanal.c][BE][1/2] Fix vector load/stores to not use ld1/st1

2015-01-13 Thread Jeff Law
On 01/13/15 11:55, Eric Botcazou wrote: (1) we have a non-paradoxical subreg; (2) both (reg:ymode xregno) and (reg:xmode xregno) occupy full hard registers (no padding or unused upper bits); (3) (reg:ymode xregno) and (reg:xmode xregno) store the same number of bytes (X) in each

Re: [PATCH] add option to emit more array bounds warnigs

2015-01-13 Thread Martin Uecker
Jeff Law l...@redhat.com: On 01/13/15 17:40, Martin Uecker wrote: Jeff Law l...@redhat.com: On 01/13/15 10:34, Martin Uecker wrote: Mon, 12 Jan 2015 11:00:44 -0700 Jeff Law l...@redhat.com: On 11/11/14 23:13, Martin Uecker wrote: ... Has this patch been bootstrapped and

Re: [PATCH] PR59448 - Promote consume to acquire

2015-01-13 Thread Andrew MacLeod
On 01/13/2015 01:38 PM, Torvald Riegel wrote: On Tue, 2015-01-13 at 10:11 -0500, Andrew MacLeod wrote: On 01/13/2015 09:59 AM, Richard Biener wrote: On Tue, Jan 13, 2015 at 3:56 PM, Andrew MacLeod amacl...@redhat.com wrote: Lengthy discussion :

Re: [PATCH] Reenable CSE of non-volatile inline asm (PR rtl-optimization/63637)

2015-01-13 Thread Jakub Jelinek
On Tue, Jan 13, 2015 at 12:45:27PM -0700, Jeff Law wrote: On 01/13/15 09:38, Segher Boessenkool wrote: On Tue, Jan 13, 2015 at 05:18:19PM +0100, Jakub Jelinek wrote: 3) on request from Richard (which Segher on IRC argues against), memory clobber also prevents CSE; As extend.texi used to

Re: PR54442 build_qualified_type produces a non-canonical type

2015-01-13 Thread Paolo Carlini
Hi, On 06/09/2014 04:46 PM, Jason Merrill wrote: On 06/09/2014 10:32 AM, Marc Glisse wrote: On Mon, 9 Jun 2014, Jason Merrill wrote: On 06/09/2014 10:18 AM, Marc Glisse wrote: I doubt the patch can be wrong, but it may be that this is a situation that is not supposed to happen and should be

Re: [PATCH] PR59448 - Promote consume to acquire

2015-01-13 Thread Jeff Law
On 01/13/15 11:38, Torvald Riegel wrote: On Tue, 2015-01-13 at 10:11 -0500, Andrew MacLeod wrote: On 01/13/2015 09:59 AM, Richard Biener wrote: On Tue, Jan 13, 2015 at 3:56 PM, Andrew MacLeod amacl...@redhat.com wrote: Lengthy discussion : https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59448

Re: [PATCH] Fix ICE with -fgnu-tm and pragma ivdep (PR middle-end/64391)

2015-01-13 Thread Jeff Law
On 01/13/15 09:28, Marek Polacek wrote: We ICE on this testcase, because the usage of #pragma GCC ivdep pulls in the ANNOTATE internal functions which don't have underlying fndecls, hence we segv on a NULL_TREE. This patch makes get_attrs_for be prepared for such a scenario. The callers of

Re: [testsuite] PATCH: Add check_effective_target_pie

2015-01-13 Thread Jeff Law
On 01/13/15 05:52, H.J. Lu wrote: On Mon, Jan 12, 2015 at 03:04:20PM -0700, Jeff Law wrote: On 01/12/15 14:51, Magnus Granberg wrote: måndag 12 januari 2015 12.11.17 skrev H.J. Lu: On Mon, Jan 12, 2015 at 12:03 PM, Jeff Law l...@redhat.com wrote: On 01/12/15 12:59, H.J. Lu wrote: I don't

Re: [testsuite] PATCH: Check if -pg available

2015-01-13 Thread Jeff Law
On 01/13/15 05:54, H.J. Lu wrote: On Mon, Jan 12, 2015 at 03:04:20PM -0700, Jeff Law wrote: On 01/12/15 14:51, Magnus Granberg wrote: måndag 12 januari 2015 12.11.17 skrev H.J. Lu: On Mon, Jan 12, 2015 at 12:03 PM, Jeff Law l...@redhat.com wrote: On 01/12/15 12:59, H.J. Lu wrote: I don't

Re: [PATCH][rtlanal.c][BE][1/2] Fix vector load/stores to not use ld1/st1

2015-01-13 Thread Eric Botcazou
Sorry for the slow response. Jeff has approved the patch in the meantime, but I didn't want to go ahead and apply it while there was still disagreement... I still think that it isn't appropriate to short-circuit the main computation as the patch does, but I don't want to block it after

Re: [PATCH] Allow MIPS call-saved-{4-6}.c tests to correctly run for micromips

2015-01-13 Thread Richard Sandiford
Maciej W. Rozycki ma...@linux-mips.org writes: On Tue, 13 Jan 2015, Andrew Bennett wrote: The call-saved-{4-6}.c tests in the mips testsuite fail for micromips. The reason is that micromips uses the swm and lwm instructions to save/restore the call-saved registers rather than using the sw

RE: [PATCH] Allow MIPS call-saved-{4-6}.c tests to correctly run for micromips

2015-01-13 Thread Matthew Fortune
Richard Sandiford rdsandif...@googlemail.com writes: Maciej W. Rozycki ma...@linux-mips.org writes: On Tue, 13 Jan 2015, Andrew Bennett wrote: The call-saved-{4-6}.c tests in the mips testsuite fail for micromips. The reason is that micromips uses the swm and lwm instructions to

Re: [PATCH] PR59448 - Promote consume to acquire

2015-01-13 Thread Torvald Riegel
On Tue, 2015-01-13 at 10:11 -0500, Andrew MacLeod wrote: On 01/13/2015 09:59 AM, Richard Biener wrote: On Tue, Jan 13, 2015 at 3:56 PM, Andrew MacLeod amacl...@redhat.com wrote: Lengthy discussion : https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59448 Basically we can generate incorrect

Re: [PATCH] Fix for PR64081 in RTL loop unroller

2015-01-13 Thread Jeff Law
On 01/13/15 11:01, Zamyatin, Igor wrote: Is it really sufficient here to verify that all the defs are on latch predecessors, what about the case where there is a predecessor without a def. How do you guarantee domination in that case? ISTM that given the structure for the code you're writing

Re: [PATCH] add option to emit more array bounds warnigs

2015-01-13 Thread Jeff Law
On 01/13/15 10:34, Martin Uecker wrote: Mon, 12 Jan 2015 11:00:44 -0700 Jeff Law l...@redhat.com: On 11/11/14 23:13, Martin Uecker wrote: ... * gcc/tree-vrp.c (check_array_ref): Emit more warnings for warn_array_bounds = 2. * gcc/testsuite/gcc.dg/Warray-bounds-11.c: New

Re: [PATCH] Fix REE for vector modes (PR rtl-optimization/64286, take 2)

2015-01-13 Thread Jeff Law
On 01/13/15 09:11, Jakub Jelinek wrote: On Mon, Jan 12, 2015 at 02:29:53PM -0700, Jeff Law wrote: On 01/12/15 12:59, Jakub Jelinek wrote: Hi! As mentioned in the PR, giving up for all vector mode extensions is unnecessary, but unlike scalar integer extensions, where the low part of the

Re: [PATCH] Reenable CSE of non-volatile inline asm (PR rtl-optimization/63637)

2015-01-13 Thread Jeff Law
On 01/13/15 09:38, Segher Boessenkool wrote: On Tue, Jan 13, 2015 at 05:18:19PM +0100, Jakub Jelinek wrote: 3) on request from Richard (which Segher on IRC argues against), memory clobber also prevents CSE; As extend.texi used to say: If your assembler instructions access memory in an

Re: [PATCH] add option to emit more array bounds warnigs

2015-01-13 Thread Martin Uecker
Jeff Law l...@redhat.com: On 01/13/15 10:34, Martin Uecker wrote: Mon, 12 Jan 2015 11:00:44 -0700 Jeff Law l...@redhat.com: On 11/11/14 23:13, Martin Uecker wrote: ... Has this patch been bootstrapped and regression tested, if so on what platform. x86_64-unknown-linux-gnu

Re: Housekeeping work in backends.html

2015-01-13 Thread Anthony Green
Eric Botcazou ebotca...@adacore.com writes: Some ports are missing (lm32, moxie, nios2, nvptx, rl78, rx) so the relevant maintainers are CCed (see 6.3.9 Anatomy of a Target Back End in the doc). I think I got this right | Characteristics Target | HMSLQNFICBD

Re: [patch 1/2][ARM]: New CPU support for Marvell Whitney

2015-01-13 Thread Xingxing Pan
On 09/01/2015 19:22, Kyrill Tkachov wrote: Hi Xingxing, On 19/12/14 11:01, Xingxing Pan wrote: +/* Return true if vector element size is byte. */ Minor nit: two spaces after full stop and before */ Same in other places in the patch. +bool +marvell_whitney_vector_element_size_is_byte (rtx

Re: [PATCH 4/4] Wire X-Gene 1 up in the ARM (32bit) backend as a AArch32-capable core.

2015-01-13 Thread Ramana Radhakrishnan
On 12/01/15 20:15, Philipp Tomsich wrote: --- gcc/ChangeLog-2014| 10 ++ gcc/config/arm/arm-cores.def | 1 + gcc/config/arm/arm-tables.opt | 3 +++ gcc/config/arm/arm-tune.md| 3 ++- gcc/config/arm/arm.c | 22 ++

Re: [RFC PATCH Fortran] make enum_9/10.f90 testcases work under FreeBSD ARM

2015-01-13 Thread Ramana Radhakrishnan
On Sun, Jan 11, 2015 at 9:55 PM, Andreas Tobler andreast-l...@fgznet.ch wrote: Hi, I have here a possible way to make the enum_9.f90 and the enum_10.f90 work under arm*-*-freebsd*. The solution for enum_9.f90 is straight forward. But the one for enum_10.f90 requires a reordering of the

Re: [PATCH 3/4] Change the type of the prefetch-instructions to 'prefetch'.

2015-01-13 Thread Ramana Radhakrishnan
On 12/01/15 20:15, Philipp Tomsich wrote: --- gcc/config/aarch64/aarch64.md | 2 +- gcc/config/arm/types.md | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md index 1f6b1b6..98f4f30 100644 ---

Re: [PATCH, aarch64] Add prefetch support

2015-01-13 Thread Andrew Pinski
On Tue, Jan 13, 2015 at 6:13 AM, Marcus Shawcroft marcus.shawcr...@gmail.com wrote: On 11 January 2015 at 02:37, Andrew Pinski pins...@gmail.com wrote: On Tue, Nov 11, 2014 at 6:47 AM, Marcus Shawcroft marcus.shawcr...@gmail.com wrote: On 30 October 2014 08:54, Gopalasubramanian, Ganesh

Re: [PATCH, aarch64] Add prefetch support

2015-01-13 Thread Dr. Philipp Tomsich
Great. I should have an update patch-set ready tested later tonight. Best, Phil. On 13 Jan 2015, at 15:18, Andrew Pinski pins...@gmail.com wrote: On Tue, Jan 13, 2015 at 6:13 AM, Marcus Shawcroft marcus.shawcr...@gmail.com wrote: On 11 January 2015 at 02:37, Andrew Pinski

Re: [PATCH 2/4] Pipeline model for APM XGene-1.

2015-01-13 Thread Richard Earnshaw
On 13/01/15 13:46, Marcus Shawcroft wrote: On 12 January 2015 at 20:15, Philipp Tomsich philipp.toms...@theobroma-systems.com wrote: --- gcc/config/aarch64/aarch64.md | 1 + gcc/config/arm/xgene1.md | 531 ++ 2 files changed, 532

Re: [PATCH, aarch64] Add prefetch support

2015-01-13 Thread Marcus Shawcroft
On 11 January 2015 at 02:37, Andrew Pinski pins...@gmail.com wrote: On Tue, Nov 11, 2014 at 6:47 AM, Marcus Shawcroft marcus.shawcr...@gmail.com wrote: On 30 October 2014 08:54, Gopalasubramanian, Ganesh ganesh.gopalasubraman...@amd.com wrote: 2014-10-30 Ganesh Gopalasubramanian

Re: [testsuite] PATCH: Add check_effective_target_pie

2015-01-13 Thread H.J. Lu
On Mon, Jan 12, 2015 at 03:04:20PM -0700, Jeff Law wrote: On 01/12/15 14:51, Magnus Granberg wrote: måndag 12 januari 2015 12.11.17 skrev H.J. Lu: On Mon, Jan 12, 2015 at 12:03 PM, Jeff Law l...@redhat.com wrote: On 01/12/15 12:59, H.J. Lu wrote: I don't know if -pg will work PIE on any

[PATCH] PR59448 - Promote consume to acquire

2015-01-13 Thread Andrew MacLeod
Lengthy discussion : https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59448 Basically we can generate incorrect code for an atomic consume operation in some circumstances. The general feeling seems to be that we should simply promote all consume operations to an acquire operation until there is

[patch] libstdc++/64571 export fstream functions using new std::string

2015-01-13 Thread Jonathan Wakely
The existing tests for these functions are compile-only so didn't catch that I forgot to export these new symbols. I'll add a better test next week. Tested x86_64-linux, committed to trunk. commit d428e75af04d995451a917ef7c9caed6b8cee737 Author: Jonathan Wakely jwak...@redhat.com Date: Tue Jan

Re: [PATCH] PR59448 - Promote consume to acquire

2015-01-13 Thread Richard Biener
On Tue, Jan 13, 2015 at 3:56 PM, Andrew MacLeod amacl...@redhat.com wrote: Lengthy discussion : https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59448 Basically we can generate incorrect code for an atomic consume operation in some circumstances. The general feeling seems to be that we should

Re: [PATCH][ARM] FreeBSD ARM support, EABI, v3

2015-01-13 Thread Ramana Radhakrishnan
On Thu, Jan 8, 2015 at 8:51 PM, Andreas Tobler andreast-l...@fgznet.ch wrote: On 08.01.15 17:27, Richard Earnshaw wrote: On 29/12/14 18:44, Andreas Tobler wrote: All, here is the third attempt to support ARM with FreeBSD. In the meantime we found another issue in the unwinder where I had

Re: [PATCH] Fix PR64461, Incorrect code on coldfire targets

2015-01-13 Thread Andreas Schwab
Jeff Law l...@redhat.com writes: For fun I've got an m68k bootstrap of the trunk running. I don't expect it to finish for at least a week or so, assuming it runs to completion. The last time I did that it took about 10 days (with all languages enabled, running in Aranym on a moderately fast

Re: flatten expr.h (version 2)

2015-01-13 Thread Prathamesh Kulkarni
On 13 January 2015 at 15:34, Richard Biener rguent...@suse.de wrote: On Sun, 11 Jan 2015, Prathamesh Kulkarni wrote: Hi, This is a revamped expr.h flattening flattening patch rebased on tree.h and tree-core.h flattening patch (r219402). It depends upon the following patch to get committed.

Re: [Fortran, Patch] PR60334 - Segmentation fault on character pointer assignments

2015-01-13 Thread Andre Vehreschild
Hi Paul, thanks for the reviewed and the valued comments. Just for completeness I have attached the patch with the changes requested. Bootstraps and regtests ok on x86_64-linux-gnu. Regards, Andre On Mon, 12 Jan 2015 22:07:29 +0100 Paul Richard Thomas paul.richard.tho...@gmail.com

Re: shift/extract SHIFT_COUNT_TRUNCATED combine bug

2015-01-13 Thread Richard Biener
On Mon, Jan 12, 2015 at 11:12 PM, Jeff Law l...@redhat.com wrote: On 04/08/14 14:07, Mike Stump wrote: Something broke in the compiler to cause combine to incorrectly optimize: (insn 12 11 13 3 (set (reg:SI 604 [ D.6102 ]) (lshiftrt:SI (subreg/s/u:SI (reg/v:DI 601 [ x ]) 0)

Re: [patch, arm] Minor optimization on thumb2 tail call

2015-01-13 Thread Ramana Radhakrishnan
On 19/11/14 02:43, Joey Ye wrote: Current thumb2 -Os generates suboptimal code for following tail call case: int f4(int b, int a, int c, int d); int g(int a, int b, int c, int d) { return f4(b, a, c, d); } arm-none-eabi-gcc -Os -mthumb -mcpu=cortex-m3 test.c push {r4, lr} mov r4, r1 mov r1,

Re: flatten expr.h (version 2)

2015-01-13 Thread Richard Biener
On Sun, 11 Jan 2015, Prathamesh Kulkarni wrote: Hi, This is a revamped expr.h flattening flattening patch rebased on tree.h and tree-core.h flattening patch (r219402). It depends upon the following patch to get committed. https://gcc.gnu.org/ml/gcc-patches/2015-01/msg00565.html Changes:

Re: [Patch, AArch64, Testsuite] Check for expected MOVI vectorization.

2015-01-13 Thread Marcus Shawcroft
On 9 January 2015 at 16:31, Tejas Belagod tejas.bela...@arm.com wrote: gcc/testsuite: * gcc.target/aarch64/vect-movi.c: Check for vectorization for 64-bit and 128-bit. OK /Marcus

Re: [gomp4] Replace enum omp_clause_map_kind with enum gomp_map_kind (was: Including a file from include/ in gcc/*.h)

2015-01-13 Thread Thomas Schwinge
Hi! On Mon, 12 Jan 2015 17:39:16 +0100, Jakub Jelinek ja...@redhat.com wrote: On Mon, Jan 12, 2015 at 05:32:14PM +0100, Thomas Schwinge wrote: I have now committed the patch to gomp-4_0-branch in the following form. The issues raised above remain to be resolved. (I'll try to address those

Re: [PATCH] Fix PR64436: broken logic to process bitwise ORs in bswap pass

2015-01-13 Thread Richard Biener
On Mon, 12 Jan 2015, Thomas Preud'homme wrote: Hi all, To identify if a set of loads, shift, cast, mask (bitwise and) and bitwise OR is equivalent to a load or byteswap, the bswap pass assign a number to each byte loaded according to its significance (1 for lsb, 2 for next least

[AArch64] Allow stack pointer as first input to a subtraction

2015-01-13 Thread Richard Sandiford
Several sub-based patterns allowed the stack pointer to be the destination but not the first source. This looked like an oversight; in all the patterns changed here (but not for example in *sub_mul_imm_mode), the instruction allows the stack pointer to appear in both positions. Tested on

Re: [AARCH64][PR63424][4.9]Backport Fix PR63424 by adding sumaxminv2di3 pattern

2015-01-13 Thread Marcus Shawcroft
On 7 January 2015 at 14:01, Renlin Li renlin...@arm.com wrote: Is it Okay for branch 4.9? gcc/ChangeLog: 2014-11-19 Renlin Li renlin...@arm.com PR target/63424 * config/aarch64/aarch64-simd.md (sumaxminv2di3): New. gcc/testsuite/ChangeLog: 2014-11-19 Renlin Li renlin...@arm.com

[testsuite] PATCH: Check if -pg available

2015-01-13 Thread H.J. Lu
On Mon, Jan 12, 2015 at 03:04:20PM -0700, Jeff Law wrote: On 01/12/15 14:51, Magnus Granberg wrote: måndag 12 januari 2015 12.11.17 skrev H.J. Lu: On Mon, Jan 12, 2015 at 12:03 PM, Jeff Law l...@redhat.com wrote: On 01/12/15 12:59, H.J. Lu wrote: I don't know if -pg will work PIE on any

Re: [Fortran, Patch] Cosmetics

2015-01-13 Thread Andre Vehreschild
Hi, is this patch commited now? I don't have the rights to do so myself. - Andre On Sun, 28 Dec 2014 17:17:50 +0100 FX fxcoud...@gmail.com wrote: 2014-12-28 Andre Vehreschild ve...@gmx.de * trans-decl.c (gfc_finish_var_decl): Fixed displaced comment. * trans-stmt.c

Re: [PATCH] Fix PR64415

2015-01-13 Thread Richard Biener
On Tue, 13 Jan 2015, Jakub Jelinek wrote: On Tue, Jan 13, 2015 at 02:04:26PM +0100, Richard Biener wrote: The following removes -fvar-tracking-assignments from being eligible to the optimization attribute/pragma which fixes LTO operation for mixed inputs (LTO just drops debug stmts if the

Re: [AArch64] Allow stack pointer as first input to a subtraction

2015-01-13 Thread Marcus Shawcroft
On 13 January 2015 at 10:47, Richard Sandiford richard.sandif...@arm.com wrote: Several sub-based patterns allowed the stack pointer to be the destination but not the first source. This looked like an oversight; in all the patterns changed here (but not for example in *sub_mul_imm_mode), the

Re: [PATCH/AARCH64] Correctly handle stores of zero in fusion_load_store

2015-01-13 Thread Marcus Shawcroft
On 13 January 2015 at 04:48, Andrew Pinski pins...@gmail.com wrote: ChangeLog: * config/aarch64/aarch64.c (fusion_load_store): Check dest mode instead of src mode. * gcc.target/aarch64/store-pair-1.c: New testcase. OK, thanks /Marcus

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

2015-01-13 Thread H.J. Lu
On Mon, Jan 12, 2015 at 11:50:41PM +, Joseph Myers wrote: On Mon, 12 Jan 2015, H.J. Lu wrote: +if test x$enable_default_pie = xyes; then + AC_MSG_CHECKING(if $target supports default PIE) + enable_default_pie=no + case $target in +i?86*-*-linux* | x86_64*-*-linux*) +

Re: [testsuite] PATCH: Check if -pg available

2015-01-13 Thread H.J. Lu
On Tue, Jan 13, 2015 at 04:54:32AM -0800, H.J. Lu wrote: On Mon, Jan 12, 2015 at 03:04:20PM -0700, Jeff Law wrote: On 01/12/15 14:51, Magnus Granberg wrote: måndag 12 januari 2015 12.11.17 skrev H.J. Lu: On Mon, Jan 12, 2015 at 12:03 PM, Jeff Law l...@redhat.com wrote: On 01/12/15 12:59,

[PATCH] Fix PR64373

2015-01-13 Thread Richard Biener
The following patch guards LTO against PARM_DECLs without DECL_CONTEXT. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2015-02-13 Richard Biener rguent...@suse.de PR lto/64373 * lto-streamer-out.c (tree_is_indexable): Guard for NULL

Re: [PATCH] Fix PR64415

2015-01-13 Thread Jakub Jelinek
On Tue, Jan 13, 2015 at 02:04:26PM +0100, Richard Biener wrote: The following removes -fvar-tracking-assignments from being eligible to the optimization attribute/pragma which fixes LTO operation for mixed inputs (LTO just drops debug stmts if the flag is false). In theory we could also fix

[PATCH] Fix PR64406

2015-01-13 Thread Richard Biener
When a optimization pass in the loop pipeline moves stmts between loops or removes loops we have to reset the SCEV cache to not have stale CHREC_LOOPs. This patch does it for loop distribution for which I have a testcase. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard.

[PATCH][ARM] PR 64149: Remove -mlra/-mno-lra option for ARM.

2015-01-13 Thread Matthew Wahab
Hello, The LRA register alloator is enabled by default for the ARM backend and -mno-lra should no longer be used. This patch removes the -mlra/-mno-lra option from the ARM backend. arm-none-linux-gnueabihf passes gcc-check with no new failures. Matthew 2015-01-13 Matthew Wahab

Re: [PATCH 2/4] Pipeline model for APM XGene-1.

2015-01-13 Thread Marcus Shawcroft
On 12 January 2015 at 20:15, Philipp Tomsich philipp.toms...@theobroma-systems.com wrote: --- gcc/config/aarch64/aarch64.md | 1 + gcc/config/arm/xgene1.md | 531 ++ 2 files changed, 532 insertions(+) create mode 100644

Re: [PATCH] Fix up computed goto on POINTERS_EXTEND_UNSIGNED targets (PR middle-end/63974)

2015-01-13 Thread H.J. Lu
On Mon, Jan 12, 2015 at 12:19 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! The 991213-3.c testcase ICEs on aarch64-linux with -mabi=ilp32 since wide-int merge. The problem is that x = convert_memory_address (Pmode, x) is used twice on a VOIDmode CONST_INT, which is wrong. For non-VOIDmode

Re: [PATCH/AARCH64] Disable load/store pair peephole for volatile mem

2015-01-13 Thread Marcus Shawcroft
On 10 December 2014 at 02:18, Andrew Pinski pins...@gmail.com wrote: Hi, As mentioned in https://gcc.gnu.org/ml/gcc-patches/2014-12/msg00609.html, the load/store pair peepholes currently accept volatile mem which can cause wrong code as the architecture does not define which part of the

Re: [testsuite] PATCH: Add check_effective_target_pie

2015-01-13 Thread H.J. Lu
On Mon, Jan 12, 2015 at 03:04:20PM -0700, Jeff Law wrote: On 01/12/15 14:51, Magnus Granberg wrote: måndag 12 januari 2015 12.11.17 skrev H.J. Lu: On Mon, Jan 12, 2015 at 12:03 PM, Jeff Law l...@redhat.com wrote: On 01/12/15 12:59, H.J. Lu wrote: I don't know if -pg will work PIE on any

Re: [PATCH 1/4] Core definition for APM XGene-1 and associated cost-table.

2015-01-13 Thread Marcus Shawcroft
On 12 January 2015 at 20:15, Philipp Tomsich philipp.toms...@theobroma-systems.com wrote: +2014-11-19 Philipp Tomsich philipp.toms...@theobroma-systems.com + + * config/aarch64/aarch64-cores.def (xgene1): Update/add the + xgene1 (APM XGene-1) core definition. + *

[PATCH] Fix PRs 64493 and 64495

2015-01-13 Thread Richard Biener
The following fixes a bug in outer loop reduction vectorization which happens to use a bogus vectorized stmt for the inner loop exit PHI. Bootstrap and regtest in progress on x86_64-unknown-linux-gnu. Richard. 2015-01-13 Richard Biener rguent...@suse.de PR tree-optimization/64493

Re: [PATCH] Fix PR64415

2015-01-13 Thread Jakub Jelinek
On Tue, Jan 13, 2015 at 02:26:39PM +0100, Richard Biener wrote: The following seems to work (for the testcase). Testing coverage of this mode will of course be bad. LGTM. 2015-01-13 Richard Biener rguent...@suse.de PR lto/64415 * tree-inline.c (insert_debug_decl_map): Check

Re: [PATCH] Fix PR64404

2015-01-13 Thread Richard Biener
On Mon, 12 Jan 2015, Richard Biener wrote: I am testing the following patch to fix a latent bug in the vectorizer dealing with redundant DRs. Bootstrap and regtest pending on x86_64-unknown-linux-gnu. Which shows the patch is bogus. Instead we are not prepared to handle this situation.

[PATCH] Fix PR64415

2015-01-13 Thread Richard Biener
The following removes -fvar-tracking-assignments from being eligible to the optimization attribute/pragma which fixes LTO operation for mixed inputs (LTO just drops debug stmts if the flag is false). In theory we could also fix inlining to do that when inlining debug stmts into a non-VTA

Re: [PATCH] rs6000: Make rs6000_split_logical handle inverted 2nd operand (PR64358)

2015-01-13 Thread David Edelsohn
On Mon, Jan 12, 2015 at 6:52 PM, Pat Haugen pthau...@linux.vnet.ibm.com wrote: Following backport tested on 4.8/4.9 with no new regressions. Ok to commit to those branches? -Pat 2015-01-12 Pat Haugen pthau...@us.ibm.com Backport from mainline 2014-12-20 Segher

Re: [PATCH] PR59448 - Promote consume to acquire

2015-01-13 Thread Andrew MacLeod
On 01/13/2015 09:59 AM, Richard Biener wrote: On Tue, Jan 13, 2015 at 3:56 PM, Andrew MacLeod amacl...@redhat.com wrote: Lengthy discussion : https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59448 Basically we can generate incorrect code for an atomic consume operation in some circumstances. The

[[ARM/AArch64][testsuite] 09/36] Add vsubhn, vraddhn and vrsubhn tests. Split vaddhn.c into vXXXhn.inc and vaddhn.c to share code with other new tests.

2015-01-13 Thread Christophe Lyon
* gcc.target/aarch64/advsimd-intrinsics/vXXXhn.inc: New file. * gcc.target/aarch64/advsimd-intrinsics/vraddhn.c: New file. * gcc.target/aarch64/advsimd-intrinsics/vrsubhn.c: New file. * gcc.target/aarch64/advsimd-intrinsics/vsubhn.c: New file. *

[[ARM/AArch64][testsuite] 12/36] Add vmlal_n and vmlsl_n tests.

2015-01-13 Thread Christophe Lyon
* gcc.target/aarch64/advsimd-intrinsics/vmlXl_n.inc: New file. * gcc.target/aarch64/advsimd-intrinsics/vmlal_n.c: New file. * gcc.target/aarch64/advsimd-intrinsics/vmlsl_n.c: New file. diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vmlXl_n.inc

[[ARM/AArch64][testsuite] 07/36] Add vmla_lane and vmls_lane tests.

2015-01-13 Thread Christophe Lyon
* gcc.target/aarch64/advsimd-intrinsics/vmlX_lane.inc: New file. * gcc.target/aarch64/advsimd-intrinsics/vmla_lane.c: New file. * gcc.target/aarch64/advsimd-intrinsics/vmls_lane.c: New file. diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vmlX_lane.inc

[[ARM/AArch64][testsuite] 10/36] Add vmlal and vmlsl tests.

2015-01-13 Thread Christophe Lyon
* gcc.target/aarch64/advsimd-intrinsics/vmlXl.inc: New file. * gcc.target/aarch64/advsimd-intrinsics/vmlal.c: New file. * gcc.target/aarch64/advsimd-intrinsics/vmlsl.c: New file. diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vmlXl.inc

[[ARM/AArch64][testsuite] 23/36] Add vmul_lane tests.

2015-01-13 Thread Christophe Lyon
* gcc.target/aarch64/advsimd-intrinsics/vmul_lane.c: New file. diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vmul_lane.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vmul_lane.c new file mode 100644 index 000..978cd9b --- /dev/null +++

[[ARM/AArch64][testsuite] 24/36] Add vmul_n tests.

2015-01-13 Thread Christophe Lyon
* gcc.target/aarch64/advsimd-intrinsics/vmul_n.c: New file. diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vmul_n.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vmul_n.c new file mode 100644 index 000..be0ee65 --- /dev/null +++

[[ARM/AArch64][testsuite] 25/36] Add vmull tests.

2015-01-13 Thread Christophe Lyon
* gcc.target/aarch64/advsimd-intrinsics/vmull.c: New file. diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vmull.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vmull.c new file mode 100644 index 000..3fdd51e --- /dev/null +++

[[ARM/AArch64][testsuite] 02/36] Be more verbose, and actually confirm that a test was checked.

2015-01-13 Thread Christophe Lyon
* gcc.target/aarch64/advsimd-intrinsics/arm-neon-ref.h (CHECK): Add trace. (CHECK_FP): Likewise. (CHECK_CUMULATIVE_SAT): Likewise. diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/arm-neon-ref.h

[[ARM/AArch64][testsuite] 26/36] Add vmull_lane tests.

2015-01-13 Thread Christophe Lyon
* gcc.target/aarch64/advsimd-intrinsics/vmull_lane.c: New file. diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vmull_lane.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vmull_lane.c new file mode 100644 index 000..d3aa879 --- /dev/null +++

[[ARM/AArch64][testsuite] 35/36] Add vqdmull_lane tests.

2015-01-13 Thread Christophe Lyon
* gcc.target/aarch64/advsimd-intrinsics/vqdmull_lane.c: New file. diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqdmull_lane.c b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqdmull_lane.c new file mode 100644 index 000..12f2a6b --- /dev/null +++

  1   2   >