[PATCH] use gold/configure.tgt to calc supported targets

2015-04-20 Thread Mike Frysinger
Rather than maintain two lists, re-use gold's target file to determine whether the current target is supported. 2015-04-20 Mike Frysinger vap...@gentoo.org * configure.ac: Replace $target checks with gold/configure.tgt. * configure: Regenerate. --- configure| 20

RE: [PATCH][MIPS] Enable load-load/store-store bonding

2015-04-20 Thread Moore, Catherine
-Original Message- From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] Sent: Monday, April 20, 2015 3:10 PM To: Sameera Deshpande; Moore, Catherine Cc: Richard Sandiford; gcc-patches@gcc.gnu.org; echri...@gmail.com Subject: RE: [PATCH][MIPS] Enable load-load/store-store

Re: Announcing Segher Boessenkool as combine.c maintainer

2015-04-20 Thread Segher Boessenkool
On Mon, Apr 20, 2015 at 01:29:03PM -0600, Jeff Law wrote: Segher, can you please add yourself to the MAINTAINERS file for the additional role. Done as this. Cheers, Segher 2015-04-20 Segher Boessenkool seg...@kernel.crashing.org * MAINTAINERS (Various Maintainers): Add myself

Re: [PATCH 10/13] fixincludes

2015-04-20 Thread Jeff Law
On 04/20/2015 12:58 PM, Szabolcs Nagy wrote: No fixincludes are needed for musl. fixincludes/Changelog: 2015-04-16 Gregor Richards gregor.richa...@uwaterloo.ca * mkfixinc.sh: Add *-musl* with no fixes. OK. jeff

Re: [wwwdocs] Add porting_to.html for GCC 5 (again)

2015-04-20 Thread Gerald Pfeifer
On Wed, 14 Jan 2015, Marek Polacek wrote: A few months ago I posted the porting to document for GCC 5. But I never got around to commit it, so here it is again, this time with feewing. Thanks again for doing this. Below is my last set of changes: Break some paragraphs and longer sentences,

Re: [PATCH][AArch64] Increase static buffer size in aarch64_rewrite_selected_cpu

2015-04-20 Thread James Greenhalgh
On Mon, Apr 20, 2015 at 05:24:39PM +0100, Kyrill Tkachov wrote: Hi all, When trying to compile a testcase with -mcpu=cortex-a57+crypto+nocrc I got the weird assembler error: Assembler messages: Error: missing architectural extension Error: unrecognized option -mcpu=cortex-a57+crypto+no

Re: [RS6000] pr65810, powerpc64 alignment of r2

2015-04-20 Thread Alan Modra
On Mon, Apr 20, 2015 at 01:11:41PM -0400, David Edelsohn wrote: @@ -6510,6 +6518,8 @@ offsettable_ok_by_alignment (rtx op, HOST_WIDE_INT return false; dsize = GET_MODE_SIZE (mode); + if (dsize POWERPC64_TOC_POINTER_ALIGNMENT) +return false; Why do you immediately fail

Re: [PATCH] 65479 - sanitizer stack trace missing frames past #0 on powerpc64

2015-04-20 Thread Martin Sebor
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index b4052ef..18eede3 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,12 @@ +2015-04-19 Martin Sebormse...@redhat.com + +PR sanitizer/65479 +* gcc/testsuite/c-c++-common/asan/misalign-1.c

Re: [PATCH 1/13] libitm fixes for musl support

2015-04-20 Thread Jeff Law
On 04/20/2015 12:51 PM, Szabolcs Nagy wrote: This are minor correctness fixes required for musl. (fcntl.h is the standard header and always available on Linux, sys/fcntl.h is just a legacy alias, so use the standard one.) libitm/Changelog: 2015-04-16 Gregor Richards

[wwwdocs] PATCH for Re: [ANN] gcc-lua: Lua plugin for the GNU Compiler Collection

2015-04-20 Thread Gerald Pfeifer
Hi Peter, On Wed, 31 Oct 2012, Peter Colberg wrote: gcc‑lua extends the GNU Compiler Collection with the ability to run Lua scripts. The plugin provides an interface to register callback functions for plugin events, and inspect the abstract syntax tree of a translation unit. The plugin is

Re: [Patch] pr65779 - [5/6 Regression] undefined local symbol on powerpc

2015-04-20 Thread Alan Modra
On Mon, Apr 20, 2015 at 12:17:08PM -0600, Jeff Law wrote: So just one question. Why handle the split case differently? In the split case you effectively move the debug insn to the new block. In the !split case, you just delete the debug insn. The idea was that when split we have a new block

Re: [PING^2] [PATCH][5a/5] Postpone expanding va_arg until pass_stdarg

2015-04-20 Thread Bin.Cheng
On Thu, Apr 16, 2015 at 4:55 PM, Richard Biener rguent...@suse.de wrote: On Thu, 16 Apr 2015, Tom de Vries wrote: [stage1 ping^2] On 10-03-15 16:30, Tom de Vries wrote: [stage1 ping] On 22-02-15 14:13, Tom de Vries wrote: On 19-02-15 14:03, Richard Biener wrote: On Thu, 19 Feb 2015,

Re: Ping^3 : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-04-20 Thread Segher Boessenkool
On Tue, Apr 21, 2015 at 09:39:16AM +0800, Terry Guo wrote: Is this one ok to trunk? Probably, if you send the patch + changelog entry :-) Did you fix the comment? REG_USERVAR_P and HARD_REGISTER_P can be set for more than just register asm. Segher

Re: ira.c update_equiv_regs patch causes gcc/testsuite/gcc.target/arm/pr43920-2.c regression

2015-04-20 Thread Jeff Law
On 04/20/2015 01:09 AM, Shiva Chen wrote: Hi, Jeff Thanks for your advice. can_replace_by.patch is the new patch to handle both cases. pr43920-2.c.244r.jump2.ori is the original jump2 rtl dump pr43920-2.c.244r.jump2.patch_can_replace_by is the jump2 rtl dump after patch

[PATCH] Emit -Waddress warnings for comparing address of reference against NULL

2015-04-20 Thread Patrick Palka
Implementation is pretty straightforward. The only catch is that the middle-end doesn't actually assume that REFERENCE_TYPEs are non-NULL so code like int a = *(int *)0; if (a != 0) will warn that a will never be NULL yet the middle-end will fold the conditional to false instead of true

Ping^3 : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-04-20 Thread Terry Guo
Hi there, Is this one ok to trunk? BR, Terry On Wed, Apr 15, 2015 at 6:45 PM, Hale Wang hale.w...@arm.com wrote: Ping for trunk? Hale -Original Message- From: Richard Sandiford [mailto:rdsandif...@googlemail.com] Sent: Friday, February 27, 2015 4:04 AM To: Terry Guo Cc: Segher

RE: [PATCH][AArch64] Implement -m{cpu,tune,arch}=native using only /proc/cpuinfo

2015-04-20 Thread Kumar, Venkataramanan
Hi Kyrill, In AMD Seattle board, I see that CPU implementer is 0x41 and CPU part is 0xd07.CPU variant is 1 but you don’t do anything with that. It matches with cortex-a57 and its features. I will try a bootstrap test as well. Regards, Venkat. -Original Message- From:

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

2015-04-20 Thread Gregor Richards
To confirm: Yes, I've signed the FSF copyright assignment document, and yes we've discussed the changes relative to my own (which were mostly just cleanups and sanity anyway). I'm Gregor Richards and I approve this message. With valediction, - Gregor Richards On 04/20/2015 02:49 PM,

Re: Merge current set of OpenACC changes from gomp-4_0-branch

2015-04-20 Thread Gerald Pfeifer
On Mon, 20 Apr 2015, Thomas Schwinge wrote: Gerald, is it OK to commit the following to update GCC 5 changes' »New Languages and Language specific improvements« section? Sure thing. Gerald

Re: [AArch64][PR65375] Fix RTX cost for vector SET

2015-04-20 Thread James Greenhalgh
On Fri, Apr 17, 2015 at 12:19:14PM +0100, Kugan wrote: My point is that adding your patch while keeping the logic at the top which claims to catch ALL vector operations makes for less readable code. At the very least you'll need to update this comment: /* TODO: The cost

Re: [PATCH 8/13] sh musl support

2015-04-20 Thread Oleg Endo
On Mon, 2015-04-20 at 19:56 +0100, Szabolcs Nagy wrote: Set up dynamic linker name for sh. gcc/Changelog: 2015-04-16 Gregor Richards gregor.richa...@uwaterloo.ca * config/sh/linux.h (MUSL_DYNAMIC_LINKER): Define. The SH parts are OK for trunk. Cheers, Oleg

Re: [PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt

2015-04-20 Thread Richard Biener
On Wed, 15 Apr 2015, Tom de Vries wrote: On 03-04-15 14:39, Tom de Vries wrote: On 27-03-15 15:10, Tom de Vries wrote: Hi, this patch fixes PR65443, a todo in the parloops pass for function transform_to_exit_first_loop: ... TODO: the common case is that latch of the loop

Re: [patch, fortran] PR 37131

2015-04-20 Thread Mikael Morin
Le 19/04/2015 17:58, Thomas Koenig a écrit : Hello world, here is the first installation of the matmul inlining patch. This patch calculates c=MATMUL(a,b) using DO loops where there is no dependency between a and c/b and c loops, taking care of realloc on assignment and bounds checking

Re: [Patch] pr65779 - [5/6 Regression] undefined local symbol on powerpc

2015-04-20 Thread Jakub Jelinek
On Mon, Apr 20, 2015 at 10:30:32PM +0930, Alan Modra wrote: and there could be other debug insns for the same variable on that path, say saying that decl lives in some other register, or can be computed using an expression involving other registers, or memory etc. Say you could have Yes,

Update __atomic builtins documentation.

2015-04-20 Thread Matthew Wahab
Hello, The documentation for the __atomic builtins isn't clear about their expectations and behaviour. In particular, assumptions about the C11/C++11 restrictions on programs should be stated and the different behaviour of memory models in fences and in operations should be noted. The behaviour

Re: [patch,wwwdocs] Add gcc-5 caveats for avr.

2015-04-20 Thread Andi Kleen
Georg-Johann Lay a...@gjlay.de writes: +a href=http://gcc.gnu.org/onlinedocs/gcc/Spec-Files.html;spec file/a. +If the compiler is used together with AVR-LibC, this requires at +least GCCnbsp;5.2 and a version of AVR-LibC which implements Really 5.2? -Andi

Re: [PATCH][ARM][cleanup] Use IN_RANGE more often

2015-04-20 Thread Richard Earnshaw
On 20/04/15 12:03, Kyrill Tkachov wrote: On 18/04/15 15:18, Richard Earnshaw wrote: On 15/04/15 16:22, Kyrill Tkachov wrote: Hi all, This patch goes through the arm backend and replaces expressions of the form a = lo a = hi with IN_RANGE (a, lo, hi) which is that tiny bit smaller and

Re: [PATCH] Fix check for whether a function is a variadic function

2015-04-20 Thread Patrick Palka
On Sun, Apr 19, 2015 at 11:17 PM, Jason Merrill ja...@redhat.com wrote: On 04/19/2015 07:45 PM, Patrick Palka wrote: stdarg_p() apparently returns false for a variadic function that has no concrete parameters, e.g. void foo (...);. This patch fixes this issue by removing the predicate's

Re: [PATCH] Make split_block and create_basic_block type-safe

2015-04-20 Thread Richard Biener
On Mon, 23 Mar 2015, Richard Biener wrote: On Fri, 20 Mar 2015, David Malcolm wrote: On Thu, 2015-03-12 at 14:20 +0100, Richard Biener wrote: After noticing tree-parloop.c passing crap to split_block (a tree rather than a gimple or an rtx) I noticed those CFG functions simply take

Re: [PATCH][AArch64] Fix libstdc++ ABI baseline for aarch64-linux-gnu

2015-04-20 Thread Jakub Jelinek
On Mon, Apr 20, 2015 at 04:23:17PM +0300, Maxim Kuvyrkov wrote: I have been trying to figure out why I constantly get 2 extra TLS symbols in my aarch64-linux-gnu libstdc++ ABI tests, and it turned out to be due to support for non-TLS toolchain -- as discussed here [*]. However, as far as I

[committed] Fix GC ICE due to dwarf2out bug (PR debug/65807)

2015-04-20 Thread Jakub Jelinek
Hi! add_AT_wide is the only add_AT_* that doesn't clear or otherwise initialize dw_attr_val.val_entry field, so it contains random garbage, which isn't desirable when ggc walks it during collections. Supposedly this omission originates from the val_entry addition being added everywhere only

Re: [Patch] pr65779 - [5/6 Regression] undefined local symbol on powerpc

2015-04-20 Thread Alan Modra
On Mon, Apr 20, 2015 at 10:55:56AM +0200, Jakub Jelinek wrote: On Mon, Apr 20, 2015 at 06:12:26PM +0930, Alan Modra wrote: I had it that way in my first patch, then decided to try deleting.. I can certainly change it back even if only to do it the standard way for safety's sake, but I'm

Re: [PATCH, ping1] Fix removing of df problem in df_finish_pass

2015-04-20 Thread Kenneth Zadeck
As a dataflow maintainer, I approve this patch for the next release. However, you will have to get approval of a release manager to get it into 5.0. On 04/20/2015 04:22 AM, Thomas Preud'homme wrote: Ping? -Original Message- From: gcc-patches-ow...@gcc.gnu.org

Re: [Patch] pr65779 - [5/6 Regression] undefined local symbol on powerpc

2015-04-20 Thread Jakub Jelinek
On Mon, Apr 20, 2015 at 12:40:49PM +0930, Alan Modra wrote: with the log for the ubsan fails /src/gcc-5/gcc/testsuite/c-c++-common/ubsan/object-size-10.c:19:11: runtime error: index 128 out of bounds for type 'char [128]' /src/gcc-5/gcc/testsuite/c-c++-common/ubsan/object-size-10.c:19:11:

Re: [Patch] pr65779 - [5/6 Regression] undefined local symbol on powerpc

2015-04-20 Thread Alan Modra
On Mon, Apr 20, 2015 at 09:35:07AM +0200, Jakub Jelinek wrote: On Mon, Apr 20, 2015 at 12:40:49PM +0930, Alan Modra wrote: with the log for the ubsan fails /src/gcc-5/gcc/testsuite/c-c++-common/ubsan/object-size-10.c:19:11: runtime error: index 128 out of bounds for type 'char [128]'

[patch,wwwdocs] Add gcc-5 caveats for avr.

2015-04-20 Thread Georg-Johann Lay
Hi Gerald, this is the patch against GCC-5's release notes. Okay to install? Johann Index: gcc-5/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v retrieving revision 1.109 diff -u -p -r1.109 changes.html

Re: [PATCH][ARM][cleanup] Use IN_RANGE more often

2015-04-20 Thread Jakub Jelinek
On Mon, Apr 20, 2015 at 12:03:11PM +0100, Kyrill Tkachov wrote: The definition and comment on IN_RANGE in system.h is: /* A macro to determine whether a VALUE lies inclusively within a certain range without evaluating the VALUE more than once. This macro won't warn if the VALUE is

[PATCH][AArch64] Fix libstdc++ ABI baseline for aarch64-linux-gnu

2015-04-20 Thread Maxim Kuvyrkov
Hi, I have been trying to figure out why I constantly get 2 extra TLS symbols in my aarch64-linux-gnu libstdc++ ABI tests, and it turned out to be due to support for non-TLS toolchain -- as discussed here [*]. However, as far as I understand, aarch64-linux-gnu postdates NPTL implementation,

Re: ira.c update_equiv_regs patch causes gcc/testsuite/gcc.target/arm/pr43920-2.c regression

2015-04-20 Thread Shiva Chen
Hi, Jeff Thanks for your advice. can_replace_by.patch is the new patch to handle both cases. pr43920-2.c.244r.jump2.ori is the original jump2 rtl dump pr43920-2.c.244r.jump2.patch_can_replace_by is the jump2 rtl dump after patch can_replace_by.patch Could you help me to review the patch?

Re: [PATCH PR65767]Fix test case failure on arm-none-eabi

2015-04-20 Thread Jakub Jelinek
On Mon, Apr 20, 2015 at 09:17:21AM +0100, Ramana Radhakrishnan wrote: On Mon, Apr 20, 2015 at 7:50 AM, Bin Cheng bin.ch...@arm.com wrote: Hi, As comments at PR65767 and PR65718, we should use namespace other than std to avoid duplicated definition problem on arm-none-eabi. This patch fixes

Re: ping: [PATCH, ARM] attribute target (thumb,arm) [0-6]

2015-04-20 Thread Christian Bruel
Hello Ramana Can you respin this now that we are in stage1 again ? Ramana Attached the rebased, rechecked set of patches. Original with comments posted in https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02455.html https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02458.html

Re: [PATCH][expmed] Calculate mult-by-const cost properly in mult_by_coeff_cost

2015-04-20 Thread Kyrill Tkachov
On 15/04/15 16:41, Jeff Law wrote: On 04/14/2015 02:07 AM, Kyrill Tkachov wrote: Hi Jeff, Thanks for looking at this. On 13/04/15 19:18, Jeff Law wrote: On 03/16/2015 04:12 AM, Kyrill Tkachov wrote: Hi all, Eyeballing the mult_by_coeff_cost function I think it has a typo/bug. It's

Re: [PATCH 1/2] ?PR c++/61636

2015-04-20 Thread Marek Polacek
On Sat, Apr 18, 2015 at 06:53:28PM +0100, Adam Butcher wrote: Test like this? /* { dg-do run { target c++14 } } */ /* { dg-final { scan-assembler-not ... } } */ What is this dg-final supposed to do here? Marek

Re: [PATCH][expmed] Properly account for the cost and latency of shift+add ops when synthesizing mults

2015-04-20 Thread Kyrill Tkachov
Hi Jeff, On 17/04/15 20:38, Jeff Law wrote: On 04/14/2015 02:11 AM, Kyrill Tkachov wrote: Of course the effect on codegen of this patch depends a lot on the rtx costs in the backend. On aarch64 with -mcpu=cortex-a57 tuning I see the cost limit being exceeded in more cases and the expansion

Re: [PATCH] -Warray-bounds TLC

2015-04-20 Thread Richard Biener
On Fri, 17 Apr 2015, Steve Ellcey wrote: On Sat, 2015-04-18 at 00:15 +0200, Marc Glisse wrote: extern void bad (const char *__assertion) __attribute__ ((__noreturn__)); struct link_map { long int l_ns; }; extern struct link_namespaces { unsigned int _ns_nloaded; }

Re: patch to fix PR65805

2015-04-20 Thread Jakub Jelinek
On Sun, Apr 19, 2015 at 07:49:52PM -0400, Vladimir Makarov wrote: The followingpatch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65805 The problem occurred when SP was changed between the original insn and rematerialized one and the rematerialized insn contained a reg which will

Re: [PATCH PR65767]Fix test case failure on arm-none-eabi

2015-04-20 Thread Ramana Radhakrishnan
On Mon, Apr 20, 2015 at 7:50 AM, Bin Cheng bin.ch...@arm.com wrote: Hi, As comments at PR65767 and PR65718, we should use namespace other than std to avoid duplicated definition problem on arm-none-eabi. This patch fixes the issue. It is an obvious change, but I will wait for approval

[wwwdocs] gcc-5/changes.html

2015-04-20 Thread Gerald Pfeifer
Some minor changes I suggest based on going through the page. Applied, but happy to reconsider should others feel differently. Gerald Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v retrieving

RE: [PATCH, ping1] Fix removing of df problem in df_finish_pass

2015-04-20 Thread Thomas Preud'homme
Ping? -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme Sent: Tuesday, March 03, 2015 12:02 PM To: 'Bernhard Reutner-Fischer'; gcc-patches@gcc.gnu.org; 'Paolo Bonzini'; 'Seongbae Park'; 'Kenneth Zadeck'

[PATCH][doc] Improve pipeline description docs a bit

2015-04-20 Thread Kyrill Tkachov
Hi all, This patch attempts to improve the pipeline description documentation. It fixes some grammar errors,typos and clarifies some concepts. The sections on the syntactic constructs are formatted to have a small description, and example, description of syntax elements and some elaboration. Is

Re: [PATCH][ARM][cleanup] Use IN_RANGE more often

2015-04-20 Thread Kyrill Tkachov
On 18/04/15 15:18, Richard Earnshaw wrote: On 15/04/15 16:22, Kyrill Tkachov wrote: Hi all, This patch goes through the arm backend and replaces expressions of the form a = lo a = hi with IN_RANGE (a, lo, hi) which is that tiny bit smaller and easier to read in my opinion. I guess there's

Re: [wwwdocs] Update changes.html with libstdc++ changes

2015-04-20 Thread Gerald Pfeifer
On Wed, 8 Apr 2015, Jonathan Wakely wrote: The only drawback of this, and some similar cases, is that we now risk referring to older versions on a release branch. Yes, I realised that problem when making the change and linking to the versions that were current at the time. One option would be

Re: [wwwdocs] Update Fortran secrion in 4.8/changes.html

2015-04-20 Thread Gerald Pfeifer
On Mon, 19 Nov 2012, Tobias Burnus wrote: There is one sentence (preceding my patch) which I don't quite understand (specifically around the to): ...which diagnose when code to is inserted for automatic (re)allocation of a variable during assignment. Let me try to explain what the

Re: [PATCH][expr.c] PR 65358 Avoid clobbering partial argument during sibcall

2015-04-20 Thread Kyrill Tkachov
Hi Jeff, On 17/04/15 18:26, Jeff Law wrote: On 03/19/2015 08:39 AM, Kyrill Tkachov wrote: Hi all, This patch fixes PR 65358. For details look at the excellent write-up by Honggyu in bugzilla. The problem is that we're trying to pass a struct partially on the stack and partially in regs during

[PATCH][AArch64] PR target/65491: Classify V1TF vectors as AAPCS64 short vectors rather than composite types

2015-04-20 Thread Kyrill Tkachov
Hi all, The ICE in the PR happens when we pass a 1x(128-bit float) vector as an argument. The aarch64 backend erroneously classifies it as a composite type when in fact it is a short vector according to AAPCS64 (section 4.1.2 from

Re: [PATCH, i386, Darwin RFT]: Remove reload_in_progress checks

2015-04-20 Thread Uros Bizjak
On Mon, Apr 20, 2015 at 12:00 PM, Iain Sandoe i...@codesourcery.com wrote: After having fixed the typo, regtesting went without regression. I have done a bootstrap on i686-darwin10 with the amended patch - slow machine, so testing still in progress (but looks OK so far), NOTE: that there

[RFC][wwwdocs] bug report reopen policy

2015-04-20 Thread Tom de Vries
Hi, In PR64683 comment 11, Ian mentioned: ... This bug may have the same symptoms but it has a completely different cause. Next time, please do not reopen the bug unless you are certain it has the same cause. Please open a new bug instead. Thanks. ... I couldn't find a similar rule in the

[patch] Improve libstdc++ documentation on using atomics.

2015-04-20 Thread Jonathan Wakely
Currently we don't mention libatomic anywhere in the libstdc++ manual, even though it might be needed for std::atomic. This fixes that and makes a few other drive-by improvements. Committed to trunk. This would be suitable for all active branches, so I might backport it once the gcc-5-branch

Re: [Patch] pr65779 - [5/6 Regression] undefined local symbol on powerpc

2015-04-20 Thread Jakub Jelinek
On Mon, Apr 20, 2015 at 06:12:26PM +0930, Alan Modra wrote: I had it that way in my first patch, then decided to try deleting.. I can certainly change it back even if only to do it the standard way for safety's sake, but I'm curious as to why they can't be deleted in this special case. My

Re: [PATCH, i386, Darwin RFT]: Remove reload_in_progress checks

2015-04-20 Thread Iain Sandoe
On 20 Apr 2015, at 10:47, Dominique d'Humières wrote: After having fixed the typo, regtesting went without regression. I have done a bootstrap on i686-darwin10 with the amended patch - slow machine, so testing still in progress (but looks OK so far), NOTE: that there some references to

Re: [PATCH, i386, Darwin RFT]: Remove reload_in_progress checks

2015-04-20 Thread Dominique d'Humières
After having fixed the typo, regtesting went without regression. Dominique Le 19 avr. 2015 à 20:35, Uros Bizjak ubiz...@gmail.com a écrit : Hello! Attached patch removes reload_in_progress checks for x86 (LRA enabled) target. AFAICS, reload_in_progress is never set during the

[PATCH] Simplify gimple_build interface, more closely match fold_buildN

2015-04-20 Thread Richard Biener
This changes gimple_build to follow fold_buildN behavior - combine stmts only from the sequence(s) we are currently building. This avoids possible issues with a straight-forward transitioning to gimple_build in passes if they do not keep SSA form up-to-date. Bootstrap and regtest in progress on

Re: Merge current set of OpenACC changes from gomp-4_0-branch

2015-04-20 Thread Thomas Schwinge
Hi! On Thu, 15 Jan 2015 21:20:07 +0100, I wrote: In r219682, I have committed to trunk our current set of OpenACC changes, which we had prepared on gomp-4_0-branch. Thanks to everyone who has been contributing! Note that this is an experimental feature, incomplete, and subject to change

Re: [PATCH] Fix check for whether a function is a variadic function

2015-04-20 Thread Jason Merrill
On 04/20/2015 09:02 AM, Patrick Palka wrote: But never mind, I'm getting hundreds of failures from C tests that define main without a parameter list e.g. int main () { ... }. This is an issue not worth fixing.. Yep, that's what I was wondering. I think it makes sense to fix this testcase in

[PATCH][committed] Add missed '%' to error_at message

2015-04-20 Thread Ilya Verbin
This patch adds missed '%' to the error_at () message in c[p]_parser_omp_target_update. Committed as obvious. gcc/c/ * c-parser.c (c_parser_oacc_enter_exit_data): Remove excess semicolon. (c_parser_omp_target_update): Add missed % to error_at (). gcc/cp/ * parser.c

Re: [Obvious][AArch64] Delete unused aarch64_simd_emit_pair_result_insn.

2015-04-20 Thread Alan Lawrence
Oops, missed off the patch actually pushed. Attached now. Cheers, Alan Alan Lawrence wrote: Bootstrapped on aarch64-none-linux-gnu. Pushed as r34. gcc/ChangeLog: * config/aarch64/aarch64.c (aarch64_simd_emit_pair_result_insn): Delete. * config/aarch64/aarch64-protos.h

Re: [PATCH, rs6000] Force lvx and stvx for prologue saves and epilogue restores of Altivec regs

2015-04-20 Thread Bill Schmidt
Hi, I've added an assert to this patch to detect a related condition. Prior to this patch, the prologue code was causing us to incorrectly call the expand-time routine rs6000_emit_vsx_le_store(). This was harmless, but we should have caught this sooner. The patch removes this problem, but we

[Obvious][AArch64] Delete unused aarch64_simd_emit_pair_result_insn.

2015-04-20 Thread Alan Lawrence
Bootstrapped on aarch64-none-linux-gnu. Pushed as r34. gcc/ChangeLog: * config/aarch64/aarch64.c (aarch64_simd_emit_pair_result_insn): Delete. * config/aarch64/aarch64-protos.h (aarch64_simd_emit_pair_result_insn): Delete.

[PATCH][combine] Do not call rtx costs on potentially unrecognisable rtxes in combine

2015-04-20 Thread Kyrill Tkachov
Hi all, I'm trying to reduce the cases where the midend calls the backend rtx costs on bogus rtl for which the backend doesn't have patterns or ways of handling. Having to handle these kinds of rtxes sanely bloats those functions and makes them harder to maintain. One of the cases where this

Re: [RS6000] pr65810, powerpc64 alignment of r2

2015-04-20 Thread Jakub Jelinek
On Mon, Apr 20, 2015 at 11:23:17PM +0930, Alan Modra wrote: This fixes a thinko in offsettable_ok_by_alignment. It's not the absolute placement that matters, but the toc-pointer relative offset. So alignment of r2 also needs to be taken into account. Bootstrapped and regression tested

Re: [PATCH][committed] Add missed '%' to error_at message

2015-04-20 Thread Jakub Jelinek
On Mon, Apr 20, 2015 at 05:56:16PM +0300, Ilya Verbin wrote: This patch adds missed '%' to the error_at () message in c[p]_parser_omp_target_update. Committed as obvious. Please commit also to gcc-5-branch. The Swedish translation is the only one that has this message (with the bug in it)

Re: [PATCH][expmed] Properly account for the cost and latency of shift+add ops when synthesizing mults

2015-04-20 Thread Jeff Law
On 04/20/2015 05:09 AM, Kyrill Tkachov wrote: Hi Jeff, On 17/04/15 20:38, Jeff Law wrote: On 04/14/2015 02:11 AM, Kyrill Tkachov wrote: Of course the effect on codegen of this patch depends a lot on the rtx costs in the backend. On aarch64 with -mcpu=cortex-a57 tuning I see the cost limit

Re: [PATCH][expmed] Properly account for the cost and latency of shift+add ops when synthesizing mults

2015-04-20 Thread Kyrill Tkachov
On 20/04/15 16:06, Jeff Law wrote: On 04/20/2015 05:09 AM, Kyrill Tkachov wrote: Hi Jeff, On 17/04/15 20:38, Jeff Law wrote: On 04/14/2015 02:11 AM, Kyrill Tkachov wrote: Of course the effect on codegen of this patch depends a lot on the rtx costs in the backend. On aarch64 with

[RS6000] pr65810, powerpc64 alignment of r2

2015-04-20 Thread Alan Modra
This fixes a thinko in offsettable_ok_by_alignment. It's not the absolute placement that matters, but the toc-pointer relative offset. So alignment of r2 also needs to be taken into account. Bootstrapped and regression tested powerpc64-linux. OK for mainline and gcc-5 branch? Without the dead

[PATCH][OpenMP] Fix resolve_device with -foffload=disable

2015-04-20 Thread Ilya Verbin
Hi! Currently if a compiler is configured with enabled offloading, the 'devices' array in libgomp is filled properly with a number of available devices. However, if a program is compiled with -foffload=disable, the resolve_device function returns a pointer to the device, and host-fallback is not

Re: RFA (stor-layout): PATCH for c++/65734 (attribute aligned and templates)

2015-04-20 Thread Jakub Jelinek
On Tue, Apr 14, 2015 at 12:06:16PM -0400, Jason Merrill wrote: With C++ templates and attribute ((aligned)), you can have TYPE_ALIGN and TYPE_USER_ALIGN set on a type before you know its size, so layout_type and kin need to respect them if they are already set. Tested x86_64-pc-linux-gnu.

Re: [PATCH][expmed] Calculate mult-by-const cost properly in mult_by_coeff_cost

2015-04-20 Thread Jeff Law
On 04/20/2015 03:27 AM, Kyrill Tkachov wrote: On 15/04/15 16:41, Jeff Law wrote: On 04/14/2015 02:07 AM, Kyrill Tkachov wrote: Hi Jeff, Thanks for looking at this. On 13/04/15 19:18, Jeff Law wrote: On 03/16/2015 04:12 AM, Kyrill Tkachov wrote: Hi all, Eyeballing the mult_by_coeff_cost

Re: [PATCH][expr.c] PR 65358 Avoid clobbering partial argument during sibcall

2015-04-20 Thread Jeff Law
On 04/20/2015 02:25 AM, Kyrill Tkachov wrote: Hi Jeff, Hmmm, so what happens if the difference is 0? I'd be a bit worried about that case for the PA (for example). So how about asserting that the INTVAL is = 0 prior to returning so that we catch that case if it ever occurs? INTVAL being =

Re: [Patch, fortran] PR65792 unitialized structure constructor array subcomponent

2015-04-20 Thread Dominique d'Humières
I have retested a clean tree with only the patches for pr 65792 [first patch] and Andre’s one for pr59678: i.e., without any patch from pr61831, and I still see the conflict between the two patches. Dominique Le 19 avr. 2015 à 10:39, Dominique d'Humières domi...@lps.ens.fr a écrit : Snip

Re: [Patch] pr65779 - [5/6 Regression] undefined local symbol on powerpc

2015-04-20 Thread Jeff Law
On 04/19/2015 09:10 PM, Alan Modra wrote: This patch removes bogus debug info left around by shrink-wrapping, which on some powerpc targets with just the right register allocation led to assembly errors. Bootstrapped and regression tested powerpc64-linux and x86_64-linux. I did see some

Re: [patch,wwwdocs] Add gcc-5 caveats for avr.

2015-04-20 Thread Jakub Jelinek
On Mon, Apr 20, 2015 at 06:21:30PM +0200, Georg-Johann Lay wrote: Am 04/20/2015 um 03:40 PM schrieb Andi Kleen: Georg-Johann Lay a...@gjlay.de writes: +a href=http://gcc.gnu.org/onlinedocs/gcc/Spec-Files.html;spec file/a. +If the compiler is used together with AVR-LibC, this

[PATCH][AArch64] Properly cost MNEG/[SU]MNEGL patterns

2015-04-20 Thread Kyrill Tkachov
Hi all, Currently we do not handle the MNEG patterns properly in rtx costs. These instructions are similar to the MSUB ones. This patch handles them by catching the NEG at the appropriate position, extracting its operands and letting the rest of the aarch64_rtx_mult_cost function handle the

Re: [PATCH] 65479 - sanitizer stack trace missing frames past #0 on powerpc64

2015-04-20 Thread Jeff Law
On 04/19/2015 07:48 PM, Martin Sebor wrote: The attached patch resolves the failures in a number of address sanitizer tests on powerpc64*-*-*-* discussed in bug 65479 (the failures in c-c++-common/asan/swapcontext-test-1.c reported in pr65643 remain unresolved). The patch has been tested on

Re: [PATCH][committed] Add missed '%' to error_at message

2015-04-20 Thread Ilya Verbin
On Mon, Apr 20, 2015 at 17:03:27 +0200, Jakub Jelinek wrote: On Mon, Apr 20, 2015 at 05:56:16PM +0300, Ilya Verbin wrote: This patch adds missed '%' to the error_at () message in c[p]_parser_omp_target_update. Committed as obvious. Please commit also to gcc-5-branch. The Swedish

Re: [RS6000] pr65810, powerpc64 alignment of r2

2015-04-20 Thread David Edelsohn
On Mon, Apr 20, 2015 at 9:53 AM, Alan Modra amo...@gmail.com wrote: This fixes a thinko in offsettable_ok_by_alignment. It's not the absolute placement that matters, but the toc-pointer relative offset. So alignment of r2 also needs to be taken into account. Bootstrapped and regression

Re: Handle redirection blocks with clobbers

2015-04-20 Thread Jakub Jelinek
On Mon, Apr 20, 2015 at 11:14:03AM -0600, Jeff Law wrote: while (!gsi_end_p (gsi) (gimple_code (gsi_stmt (gsi)) == GIMPLE_LABEL || is_gimple_debug (gsi_stmt (gsi)) - || gimple_nop_p (gsi_stmt (gsi + || gimple_nop_p (gsi_stmt (gsi)) + ||

[PATCH][AArch64] Use extend_arith rtx cost appropriately

2015-04-20 Thread Kyrill Tkachov
Hi all, When calculating the rtx costs of an arithmetic operation combined with zero or sign extension of its operand we should use the extend_arith cost rather than the arith_shift cost. Bootstrapped and tested on aarch64-linux. Ok for trunk? Thanks, Kyrill 2015-04-20 Kyrylo Tkachov

[PATCH][DRIVER] Wrong C++ include paths when configuring with --with-sysroot=/

2015-04-20 Thread Pavel Kopyl
Hi all, To build a GCC-4.9.2 ARM cross-compiler for my setting I need to configure it with --with-sysroot=/ --with-gxx-include-dir=/usr/include/c++/4.9.2. But I found that gcc driver removes the leading slash from resulting paths: `gcc -print-prog-name=cc1plus` -v ... ignoring nonexistent

Handle redirection blocks with clobbers

2015-04-20 Thread Jeff Law
PR 65658 shows a case where we fail to thread jumps in a block that is trivially threadable and would generate no code if threaded. That in turn results in inefficient code and a false positive from -Wuninitialized. The problem is the problem block has a clobber statement and

[PATCH][ARM] Handle UNSPEC_VOLATILE in rtx costs and don't recurse inside the unspec

2015-04-20 Thread Kyrill Tkachov
Hi all, A pet project of mine is to get to the point where backend rtx costs functions won't have to handle rtxes that don't match down to any patterns/expanders we have. Or at least limit such cases. A case dealt with in this patch is QImode PLUS. We don't actually generate or handle these

[PATCH][AArch64] Implement -m{cpu,tune,arch}=native using only /proc/cpuinfo

2015-04-20 Thread Kyrill Tkachov
Hi all, This is an attempt to add native CPU detection to AArch64 GNU/Linux targets. Similar to other ports we use SPEC rewriting to rewrite -m{cpu,tune,arch}=native options into the appropriate CPU/architecture and the architecture extension options when appropriate (i.e. +crypto/+crc etc).

[PATCH][AArch64] Properly handle SHIFT ops and EXTEND in aarch64_rtx_mult_cost

2015-04-20 Thread Kyrill Tkachov
Hi all, The aarch64_rtx_mult_cost helper is supposed to handle multiplication costs as well as PLUS/MINUS operations combined with multiplication or shift operations. The shift operations may contain an extension. Currently we do not handle all these cases properly. We also don't handle other

Re: [PATCH] Add new target h8300-*-linux

2015-04-20 Thread Jeff Law
On 04/19/2015 10:51 PM, Yoshinori Sato wrote: + if (TARGET_H8300H (TARGET_H8300S || TARGET_H8300SX)) +{ + target_flags ^= MASK_H8300H; +} I'm a bit concerned by this. Why did you need to make this change? The flag is exclusion, but it's set both. Hmmm, IIRC the port has many

Re: [patch,wwwdocs] Add gcc-5 caveats for avr.

2015-04-20 Thread Georg-Johann Lay
Am 04/20/2015 um 03:40 PM schrieb Andi Kleen: Georg-Johann Lay a...@gjlay.de writes: +a href=http://gcc.gnu.org/onlinedocs/gcc/Spec-Files.html;spec file/a. +If the compiler is used together with AVR-LibC, this requires at +least GCCnbsp;5.2 and a version of AVR-LibC

[PATCH][AArch64] Increase static buffer size in aarch64_rewrite_selected_cpu

2015-04-20 Thread Kyrill Tkachov
Hi all, When trying to compile a testcase with -mcpu=cortex-a57+crypto+nocrc I got the weird assembler error: Assembler messages: Error: missing architectural extension Error: unrecognized option -mcpu=cortex-a57+crypto+no The problem is the aarch64_rewrite_selected_cpu that is used to rewrite

Re: [PATCH][MIPS] Enable load-load/store-store bonding

2015-04-20 Thread Mike Stump
With FUSION you might get farther. See the arm port as I recall. The quick overview, FUSION allows instructions that are not contiguous to be paired up and fused together. it was built for load/load store/store combining. On Apr 19, 2015, at 10:09 PM, sameera sameera.deshpa...@imgtec.com

Re: [PATCH, i386, Darwin RFT]: Remove reload_in_progress checks

2015-04-20 Thread Mike Stump
On Apr 20, 2015, at 3:16 AM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Apr 20, 2015 at 12:00 PM, Iain Sandoe i...@codesourcery.com wrote: After having fixed the typo, regtesting went without regression. I have done a bootstrap on i686-darwin10 with the amended patch - slow machine, so

[PATCH 0/13] Add musl support to GCC

2015-04-20 Thread Szabolcs Nagy
This patch set adds musl libc[0] support to GCC. The patches were originally developed by Gregor Richards[1][2], who I believe have already done the FSF copyright assignment and agrees with the changes I made (please verify). I only did minor cleanups to make the patches better suited for

[PATCH 3/13] aarch64 musl support

2015-04-20 Thread Szabolcs Nagy
Set up dynamic linker name for aarch64. gcc/Changelog: 2015-04-16 Gregor Richards gregor.richa...@uwaterloo.ca Szabolcs Nagy szabolcs.n...@arm.com * config/aarch64/aarch64-linux.h (MUSL_DYNAMIC_LINKER): Define. diff --git a/gcc/config/aarch64/aarch64-linux.h

[PATCH 11/13] unwind fix for musl

2015-04-20 Thread Szabolcs Nagy
dl_iterate_phdr depends on USE_PT_GNU_EH_FRAME. I think USE_PT_GNU_EH_FRAME could be enabled more generally (whenever libc provides dl_iterate_phdr), but I only made a conservative change. libgcc/Changelog: 2015-04-16 Gregor Richards gregor.richa...@uwaterloo.ca Szabolcs Nagy

  1   2   >