Re: [2/9] Add a REG_NREGS macro

2015-05-19 Thread Richard Sandiford
Jeff Law l...@redhat.com writes: On 05/18/2015 12:13 PM, Richard Sandiford wrote: This patch adds a REG_NREGS macro that by the end of the series will simply be an rtx field access. The definition in this patch is just a placeholder (and so I haven't tried to avoid the double evaluation of

Ref: Package with Reg #: BK272062313GDM..

2015-05-19 Thread Mr. Rowland Dewey
Good Day, Your package arrived successfully. Awaiting your reply for onward delivery. Sincerely, Mr. Rowland Dewey (Cargo Unit Director)

Re: [PATCH] Fix memory orders description in atomic ops built-ins docs.

2015-05-19 Thread Jeff Law
On 05/15/2015 10:22 AM, Torvald Riegel wrote: This patch improves the documentation of the built-ins for atomic operations. First, to be consistent with the C++11 standard and make it easier for users to relate the built-ins to the C++11 specification, the patch changes 'memory model' to

Re: [4/9] Add a dedicated rtx union member for REGs

2015-05-19 Thread Richard Sandiford
Jeff Law l...@redhat.com writes: On 05/18/2015 12:19 PM, Richard Sandiford wrote: This patch replaces the current REG i0 format with a dedicated structure, so that we can make use of the extra 32 bits in the i field. Of the places that iterate on formats and do something for 'i's, most

[SH][committed] Fix gcc.target/sh/pr54236-2.c failures

2015-05-19 Thread Oleg Endo
Hi, Since a recent change to the tree optimizers https://gcc.gnu.org/ml/gcc-patches/2015-05/msg00089.html some related SH patterns stopped working. The attached patch fixes this. Tested briefly with 'make all' and with make -k check-gcc RUNTESTFLAGS=sh.exp=pr54236* --target_board=sh-sim

[PATCH] S390: SUpport -mtune=native and -march=native.

2015-05-19 Thread Dominik Vogt
The attached patch activates the -mtune=native and -march=native command line options on S390 and System z. Most of the code is a modified copy of gcc/config/mips/driver-native.c. ChangeLog attached. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog: 2015-05-05 Dominik Vogt

Re: [PATCH] fortran/66057 -- detect malformed GENERIC statement

2015-05-19 Thread FX
Yes, the patch appears to catch those as well. Then: OK to commit. Thanks! FX

Re: [PATCH] move check-gcc parallelize value into C front end

2015-05-19 Thread Jeff Law
On 05/17/2015 09:42 PM, Jim Wilson wrote: Every check_$lang_parallelize setting is in the language specific Make-lang.in file except for the C front-end check_gcc_parallelize setting which is in the toplevel Makefile.in file. This seems to be an oversight, as it was only 3 years ago that the c/

Re: [4/9] Add a dedicated rtx union member for REGs

2015-05-19 Thread Jeff Law
On 05/19/2015 01:00 AM, Richard Sandiford wrote: Just to confirm, this doesn't change the size of a REG object, right? If it doesn't change the size, then it's OK. It doesn't change the size for LP64 hosts. It makes it 32 bits bigger for ILP32 hosts. See

[RFC] COMDAT Safe Module Level Multi versioning

2015-05-19 Thread Sriraman Tallam
We have the following problem with selectively compiling modules with -misa options and I have provided a solution to solve this. I would like to hear what you think. Multi versioning at module granularity is done by compiling a subset of modules with advanced ISA instructions, supported on

Re: Improve error message for vector conversion

2015-05-19 Thread Jeff Law
On 05/15/2015 03:59 AM, Marc Glisse wrote: Hello, this patch was regtested on ppc64le-redhat-linux. 2015-05-15 Marc Glisse marc.gli...@inria.fr gcc/ * convert.c (convert_to_integer, convert_to_vector): Include the types in the error message. gcc/testsuite/ * gcc.dg/simd-1.c:

Re: [PATCH, RFC] New memory usage statistics infrastructure

2015-05-19 Thread Jeff Law
On 05/15/2015 08:38 AM, Martin Liška wrote: Hello. Following patch attempts to rewrite memory reports for GCC's internal allocations so that it uses a new template type. The type shares parts which are currently duplicated, adds support for special 'counters' and introduces new support for

Re: [gomp4] bootstrap broken, function enclosing_target_ctx defined but not used

2015-05-19 Thread Tom de Vries
[ moved to gcc-patches ml ] On 18-05-15 17:31, Tom de Vries wrote: Thomas, In ran into this bootstrap failure with branch gomp-4_0-branch: ... src/gcc-gomp-4_0-branch/gcc/omp-low.c:2897:1: error: 'omp_context* enclosing_target_ctx(omp_context*)' defined but not used [-Werror=unused-function]

Re: [match-and-simplify] fix incorrect code-gen in 'for' pattern

2015-05-19 Thread Marek Polacek
On Tue, May 19, 2015 at 10:33:08AM +0200, Richard Biener wrote: Would that be a good idea ? For symmetry, I thought (for op (list) op2 (op)) should be supported too. Hmm, but is this really a useful extension? To me it just complicates the syntax for the occasional reader.

Re: [PATCH 02/13] optabs: Fix vec_perm - V16QI middle end lowering.

2015-05-19 Thread Andreas Krebbel
On 05/18/2015 07:35 PM, Richard Henderson wrote: On 05/11/2015 06:23 AM, Andreas Krebbel wrote: @@ -6784,14 +6784,18 @@ expand_vec_perm (machine_mode mode, rtx v0, rtx v1, rtx sel, rtx target) { /* Multiply each element by its byte size. */ machine_mode selmode =

Re: [PATCH 01/13] recog: Increased max number of alternatives - v2

2015-05-19 Thread Andreas Krebbel
On 05/18/2015 04:19 PM, Richard Biener wrote: On Mon, May 18, 2015 at 3:41 PM, Andreas Krebbel kreb...@linux.vnet.ibm.com wrote: The new version also changes the type for the alternative_mask to unsigned HOST_WIDE_INT. Bootstrapped without regressions on x86-64. Ok to apply? Please use

Re: [5/9] Create sensible dummy registers

2015-05-19 Thread Eric Botcazou
Some pieces of code create a temporary REG or MEM and only fill it in later when they're testing the cost of a particular rtx. This patch makes sure that even the dummy REG or MEM is valid, rather than force the gen_* code to handle garbage values. gcc/ * caller-save.c

[PATCH] Fix PR66185

2015-05-19 Thread Richard Biener
I forgot to roll back SLP child state when ending up building vectors from scalars. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-05-19 Richard Biener rguent...@suse.de PR tree-optimization/66185 * tree-vect-slp.c (vect_build_slp_tree): Properly

Re: Refactor gimple_expr_type

2015-05-19 Thread Richard Biener
On Tue, May 19, 2015 at 12:04 AM, Aditya K hiradi...@msn.com wrote: Date: Mon, 18 May 2015 12:08:58 +0200 Subject: Re: Refactor gimple_expr_type From: richard.guent...@gmail.com To: hiradi...@msn.com CC: tbsau...@tbsaunde.org;

Re: [CHKP, PATCH] Fix LTO cgraph merge for instrumented functions

2015-05-19 Thread Ilya Enkovich
Ping 2015-05-05 11:06 GMT+03:00 Ilya Enkovich enkovich@gmail.com: Ping 2015-04-14 12:14 GMT+03:00 Ilya Enkovich enkovich@gmail.com: On 10 Apr 03:15, Jan Hubicka wrote: References are not streamed out for nodes which are referenced in a partition but don't belong to it

Re: [PATCH 01/13] recog: Increased max number of alternatives - v2

2015-05-19 Thread Richard Biener
On Tue, May 19, 2015 at 10:40 AM, Andreas Krebbel kreb...@linux.vnet.ibm.com wrote: On 05/18/2015 04:19 PM, Richard Biener wrote: On Mon, May 18, 2015 at 3:41 PM, Andreas Krebbel kreb...@linux.vnet.ibm.com wrote: The new version also changes the type for the alternative_mask to unsigned

Re: [match-and-simplify] fix incorrect code-gen in 'for' pattern

2015-05-19 Thread Richard Biener
On Mon, 18 May 2015, Prathamesh Kulkarni wrote: On 18 May 2015 at 14:12, Richard Biener rguent...@suse.de wrote: On Sat, 16 May 2015, Prathamesh Kulkarni wrote: Hi, genmatch generates incorrect code for following (artificial) pattern: (for op (plus) op2 (op) (simplify

Re: [Patch, fortran, pr65548, 2nd take, v5] [5/6 Regression] gfc_conv_procedure_call

2015-05-19 Thread Andre Vehreschild
Hi all, find attached latest version to fix 65548. Bootstraps and regtests ok on x86_64-linux-gnu/f21. - Andre -- Andre Vehreschild * Email: vehre ad gmx dot de pr65548_5.clog Description: Binary data diff --git a/gcc/fortran/trans-stmt.c b/gcc/fortran/trans-stmt.c index 814bdde..6d565ae

Re: [PATCH] Fix match.pd narrowing opt (PR tree-optimization/66187)

2015-05-19 Thread Richard Biener
On Mon, 18 May 2015, Jakub Jelinek wrote: Hi! As the testcases show, for signed types we really should use SIGNED rather than UNSIGNED as tree_int_cst_min_precision argument, that function doesn't really do the desired thing with UNSIGNED for negative values and with -fwrapv we just want

Re: [match-and-simplify] report error for invalid operator-lists

2015-05-19 Thread Richard Biener
On Tue, 19 May 2015, Prathamesh Kulkarni wrote: Hi, genmatch segfaults on: (define_operator_list op (plus)) The above syntax is invalid, and it segfaults on: fatal_error (token, operator list is empty); because token is NULL. The patch puts a check for CPP_CLOSE_PAREN after parsing

Re: [CHKP, PATCH] Fix instrumented indirect calls with propagated pointers

2015-05-19 Thread Ilya Enkovich
Ping 2015-05-05 11:05 GMT+03:00 Ilya Enkovich enkovich@gmail.com: Ping 2015-04-14 17:35 GMT+03:00 Ilya Enkovich enkovich@gmail.com: On 10 Apr 03:27, Jan Hubicka wrote: + /* We might propagate instrumented function pointer into + not instrumented function and vice versa. In

Re: [RFC] COMDAT Safe Module Level Multi versioning

2015-05-19 Thread Richard Biener
On Tue, May 19, 2015 at 8:16 AM, Sriraman Tallam tmsri...@google.com wrote: We have the following problem with selectively compiling modules with -misa options and I have provided a solution to solve this. I would like to hear what you think. Multi versioning at module granularity is done by

Re: Check canonical types in verify_type

2015-05-19 Thread Richard Biener
On Mon, 18 May 2015, Jan Hubicka wrote: + if (!comp_type_attributes (t1, t2)) + return false; Because I think the TBAA does not care about attribute lists. I suppose this is kind-of harmless because of: I think that was about attributes like 'packed', but those

Re: [match-and-simplify] fix incorrect code-gen in 'for' pattern

2015-05-19 Thread Richard Biener
On Tue, 19 May 2015, Prathamesh Kulkarni wrote: On 18 May 2015 at 20:17, Prathamesh Kulkarni prathamesh.kulka...@linaro.org wrote: On 18 May 2015 at 14:12, Richard Biener rguent...@suse.de wrote: On Sat, 16 May 2015, Prathamesh Kulkarni wrote: Hi, genmatch generates incorrect code for

[PATCH] Fix PR66165

2015-05-19 Thread Richard Biener
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-05-19 Richard Biener rguent...@suse.de PR tree-optimization/66165 * tree-vect-slp.c (vect_supported_load_permutation_p): Add guard for no load permutation. * gcc.dg/torture/pr66165.c:

Re: Fix PR48052: loop not vectorized if index is unsigned int

2015-05-19 Thread Bin.Cheng
On Wed, May 6, 2015 at 7:02 PM, Richard Biener richard.guent...@gmail.com wrote: On Mon, May 4, 2015 at 9:47 PM, Abderrazek Zaafrani az.zaafr...@gmail.com wrote: This is an old thread and we are still running into similar issues: Code is not being vectorized on 64-bit target due to scev not

Re: [PATCH] Handle multiple vector sizes in BB vectorization

2015-05-19 Thread Rainer Orth
Richard Biener rguent...@suse.de writes: Well, not really - but at least don't fail vectorization because of that but allow it to proceed the build up from scalar pieces path. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. The testcase FAILs on Solaris/SPARC: FAIL:

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

2015-05-19 Thread James Greenhalgh
On Mon, Apr 20, 2015 at 11:16:02AM +0100, Kyrylo Tkachov wrote: 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

Re: [PATCH] Fix PR66168: ICE due to incorrect invariant register info

2015-05-19 Thread Steven Bosscher
On Tue, May 19, 2015 at 12:17 PM, Thomas Preud'homme wrote: 2015-05-18 Thomas Preud'homme PR rtl-optimization/66168 * loop-invariant.c (move_invariant_reg): Set inv-reg to destination of inv-insn when moving an invariant without introducing a temporary

[PATCH][wwwdocs] Mention native CPU detection in aarch64 notes for GCC 6

2015-05-19 Thread Kyrill Tkachov
Hi all, This patch adds a mention of the new native cpu detection feature in aarch64 GNU/Linux. Gerald, this is a patch against htdocs/gcc-6/changes.html and I thought I had seen the 'changes' link in gcc.gnu.org earlier but I don't see it now (there's only a release criteria link). Is this a

Re: [Patch, fortran, PR44672, v6] [F08] ALLOCATE with SOURCE and no array-spec

2015-05-19 Thread Andre Vehreschild
Hi all, update based on latest 65548 (v5) patch and current trunk. Description and issue addressed unchanged (see cite below). Bootstrapped and regtested on x86_64-linux-gnu/f21. Any volunteers to review? The initial version dates back to March 30. 2015. Not a single comment so far! - Andre

[gomp4] Lack of OpenACC NVPTX devices is not an error during scanning

2015-05-19 Thread Julian Brown
Hi, This patch fixes an oversight whereby if the CUDA libraries are available for some reason on a system that doesn't actually contain an nVidia card, an OpenACC program will raise an error if the NVPTX backend is picked as a default instead of falling back to some other device instead. OK for

Re: [gomp4] Lack of OpenACC NVPTX devices is not an error during scanning

2015-05-19 Thread Jakub Jelinek
On Tue, May 19, 2015 at 11:36:58AM +0100, Julian Brown wrote: This patch fixes an oversight whereby if the CUDA libraries are available for some reason on a system that doesn't actually contain an nVidia card, an OpenACC program will raise an error if the NVPTX backend is picked as a default

[PATCH] Fix PR66168: ICE due to incorrect invariant register info

2015-05-19 Thread Thomas Preud'homme
Hi, r223113 made it possible for invariant to actually be moved rather than moving the source to a new pseudoregister. However, when doing so the inv-reg is not set up properly: in case of a subreg destination it holds the inner register rather than the subreg expression. This patch fixes that.

[gomp4] Add OpenACC vector-single/vector-partitioned tests

2015-05-19 Thread Julian Brown
Hi, This patch adds several tests of vector-single/vector-partitioned mode, as part of work implementing the OpenACC execution model. Pre-approved for gomp4 branch. I will apply there shortly. Thanks, Julian ChangeLog libgomp/ *

Re: [PATCH] Handle multiple vector sizes in BB vectorization

2015-05-19 Thread Richard Biener
On Tue, 19 May 2015, Rainer Orth wrote: Richard Biener rguent...@suse.de writes: Well, not really - but at least don't fail vectorization because of that but allow it to proceed the build up from scalar pieces path. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.

Commit: MSP430: Enhance the zero_extendhisi2 pattern

2015-05-19 Thread Nick Clifton
Hi Guys, I am applying the patch below to enhance the zero_extendhisi2 pattern in the MSP430 backend so that it can cope with separate source and destination registers. This makes zero extending into another register more efficient and it also helps to work around a reload bug reported

Re: [dwarf, RFC] Emitting per-function dwarf info

2015-05-19 Thread Senthil Kumar Selvaraj
Ping! Regards Senthil On Fri, Apr 10, 2015 at 12:19:36PM +0530, Senthil Kumar Selvaraj wrote: Hi, This (rather big) patch is an attempt to generate per function DWARF information for functions that go into their own sections (through -ffunction-section or otherwise). This is so that

[PATCH] fixup hash table descriptor in winnt.c

2015-05-19 Thread tbsaunde+gcc
From: Trevor Saunders tbsaunde+...@tbsaunde.org Hi, This is a straight forward fixup of the hash table descriptor in winnt.c causing the PR. Tested a cross to i686-cygwin now builds, and committing to trunk. Trev gcc/ChangeLog: 2015-05-19 Trevor Saunders tbsaunde+...@tbsaunde.org

[PATCH] Fix PR target/65730

2015-05-19 Thread Max Filippov
2015-05-20 Max Filippov jcmvb...@gmail.com gcc/ * config/xtensa/xtensa.c (init_alignment_context): Replace MULT by BITS_PER_UNIT with ASHIFT by exact_log2 (BITS_PER_UNIT). --- gcc/config/xtensa/xtensa.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

Re: [patch, gcc 5 regression] re-enable biarch for powerpc-linux-gnu

2015-05-19 Thread Alan Modra
On Tue, May 19, 2015 at 02:18:23PM -0400, David Edelsohn wrote: This seems reasonable to me. Alan, any thoughts from you? Looks good. -- Alan Modra Australia Development Lab, IBM

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread H.J. Lu
On Tue, May 19, 2015 at 1:54 PM, Rich Felker dal...@libc.org wrote: On Tue, May 19, 2015 at 01:27:06PM -0700, H.J. Lu wrote: On Tue, May 19, 2015 at 1:15 PM, Rich Felker dal...@libc.org wrote: On Tue, May 19, 2015 at 12:17:18PM -0700, H.J. Lu wrote: On Tue, May 19, 2015 at 12:11 PM, Richard

Re: [PATCH 3/4] split-stack for powerpc64

2015-05-19 Thread Alan Modra
On Tue, May 19, 2015 at 07:40:15AM -0500, Lynn A. Boger wrote: Questions on the use of the options for split stack: - The way this is implemented, split stack is generated if the target platform supports split stack, on ppc64/ppc64le as well as on x86, and the use of -fno-split-stack doesn't

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Rich Felker
On Tue, May 19, 2015 at 05:10:11PM -0700, H.J. Lu wrote: On Tue, May 19, 2015 at 1:54 PM, Rich Felker dal...@libc.org wrote: On Tue, May 19, 2015 at 01:27:06PM -0700, H.J. Lu wrote: On Tue, May 19, 2015 at 1:15 PM, Rich Felker dal...@libc.org wrote: On Tue, May 19, 2015 at 12:17:18PM

fix class enum lookup

2015-05-19 Thread Nathan Sidwell
I've committed this obvious patch for 65954. We failed to issue a diagnostic for a failed class enum lookup, and returned NULL, rather than error_mark_node, leading to a seg fault. booted tested on x86_64-linux. nathan 2015-05-19 Nathan sidwell nat...@acm.org cp/ PR c++/65954 *

Fix minor memory leak in jump threader

2015-05-19 Thread Jeff Law
Honza pointed out we were leaking some memory in the jump threading code. There's at least two leaks, this fixes the first. Bootstrapped and regression tested on x86_64-unknown-linux-gnu. Installed on the trunk. commit f4517a9d2f17b0890d9fe1159a3981985bed5672 Author: Jeff Law

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

2015-05-19 Thread Kugan
On 07/05/15 17:24, James Greenhalgh wrote: On Wed, May 06, 2015 at 03:12:33AM +0100, Kugan wrote: gcc/ChangeLog: 2015-04-24 Kugan Vivekanandarajah kug...@linaro.org Jim Wilson jim.wil...@linaro.org * config/arm/aarch-common-protos.h (struct mem_cost_table): Added new

Re: [patch, gcc 5 regression] re-enable biarch for powerpc-linux-gnu

2015-05-19 Thread Sandra Loosemore
On 05/19/2015 05:34 PM, Alan Modra wrote: On Tue, May 19, 2015 at 02:18:23PM -0400, David Edelsohn wrote: This seems reasonable to me. Alan, any thoughts from you? Looks good. Thanks. I've checked the patch into mainline (r223418). I assume this is also a candidate for backporting to

[PING] [C++17] Implement N3928 - Extending static_assert

2015-05-19 Thread Ed Smith-Rowland
On 05/02/2015 04:16 PM, Ed Smith-Rowland wrote: This extends' static assert to not require a message string. I elected to make this work also for C++11 and C++14 and warn only with -pedantic. I think many people just write static_assert(thing, ); . I took the path of building an empty

Cleanup and improve canonical type construction in LTO

2015-05-19 Thread Jan Hubicka
Richard, this is my attempt to make sense of TYPE_CANONICAL at LTO. My undrestanding is that gimple_canonical_types_compatible_p needs to return true for all pairs of types that are considered compatible across compilation unit for any of languages we support (and in a sane way for cross

Re: [PATCH 1/4] rs6000_stack_info changes for -fsplit-stack

2015-05-19 Thread Alan Modra
On Mon, May 18, 2015 at 02:05:59PM -0400, David Edelsohn wrote: On Sun, May 17, 2015 at 10:54 PM, Alan Modra amo...@gmail.com wrote: This patch changes rs6000_stack_info to keep save areas offsets even when not used. I need lr_save_offset valid for split-stack, and it seemed reasonable to

Re: [PATCH 4/4] Split-stack arg pointer init refinement

2015-05-19 Thread Alan Modra
Thanks for reviewing! Somehow I missed seeing your OK in yesterday's email. I have far too many emails in my inbox.. Patch series committed as revisions 223424, 223425, 223426, and 223427. I made a change to the comment for rs6000_supports_split_stack, instead of /* -fsplit-stack uses a field

Re: [patch, gcc 5 regression] re-enable biarch for powerpc-linux-gnu

2015-05-19 Thread David Edelsohn
On Tue, May 19, 2015 at 8:03 PM, Sandra Loosemore san...@codesourcery.com wrote: On 05/19/2015 05:34 PM, Alan Modra wrote: On Tue, May 19, 2015 at 02:18:23PM -0400, David Edelsohn wrote: This seems reasonable to me. Alan, any thoughts from you? Looks good. Thanks. I've checked the

Re: [PATCH] plugin event for C/C++ function definitions

2015-05-19 Thread Andres Tiraboschi
2015-05-18 16:51 GMT-03:00 andres.tirabos...@tallertechnologies.com: Hi, this patch adds two new plugin events PLUGIN_START_PARSE_FUNCTION and PLUGIN_FINISH_PARSE_FUNCTION. These events are invoked at start_function and finish_function in gcc/c/c-decl.c and gcc/cp/decl.c respectively in the

Re: [Patch ARM-AArch64/testsuite 00/13] Neon intrinsics executable tests

2015-05-19 Thread James Greenhalgh
On Tue, May 12, 2015 at 09:30:48PM +0100, Christophe Lyon wrote: This patch series is a follow-up to the tests I already contributed, converted from my original testsuite. This series consists in 13 new files, which can be committed independently. Another series (hopefully final) will

Re: [patch] libstdc++/66055 add missing constructors to unordered containers

2015-05-19 Thread Jonathan Wakely
On 17/05/15 22:21 +0200, François Dumont wrote: Ok, I just commit fixing some other lines length except those having a long hyperlink, I didn't want to break those. Yep, thanks. I think we should backport Nathan's patch and your one to the gcc-5-branch too. I'll make a note to do that before

Re: [Patch, fortran, 64674, v2] [OOP] ICE in ASSOCIATE with class array

2015-05-19 Thread Andre Vehreschild
Hi, so here is the update on pr 64674. Besides adapting to current trunk nothing has changed from the previous version. The links for getting the patches this one depends on are: PR65548 v5: https://gcc.gnu.org/ml/fortran/2015-05/msg00123.html and PR44672 v6:

[patch, avr] Restore base register if not marked dead/unused

2015-05-19 Thread Sivanupandi, Pitchumani
Test gcc.c-torture/execute/memcpy-bi.c (-O2) failed for attiny40 device. Cause seems to be in load from memory as it is not restoring base register after load instructions generated. Function avr_out_load_psi_reg_no_disp_tiny in avr.c: It returns just after emitting instructions to load from

[Patch AArch64] Add cpu_defines.h for AArch64.

2015-05-19 Thread Ramana Radhakrishnan
Hi, Like the ARM port, the AArch64 ports needs to set glibc_integral_traps to false as integer divide instructions do not trap. Bootstrapped and regression tested on aarch64-none-linux-gnu Ok to apply ? regards Ramana 2015-05-17 Ramana Radhakrishnan ramana.radhakrish...@arm.com

[PATCH] Fix duplicated warning with __attribute__((format)) (PR c/64223)

2015-05-19 Thread Marek Polacek
This PR points out that we output same -Wformat warning twice when using __attribute__ ((format)). The problem was that attribute_value_equal (called when processing merge_attributes) got two lists: format printf, 1, 2 and __format__ __printf__, 1, 2, these should be equal. But since

[patch] Optimize std::list when using new ABI

2015-05-19 Thread Jonathan Wakely
This fixes some missed optimizations I should have made when adding the new std::__cxx11::list, making use of the O(1) list::size() when it saves work. In the equality comparisons two lists can't be equal if their sizes differ. When resizing a list we don't need to walk the list to find whether

[Patch ARM] Add cpu_defines.h for ARM

2015-05-19 Thread Ramana Radhakrishnan
Hardware Integer divide instructions do not trap. Define this to be so for the ARM port. Applied to trunk after a build and test across architecture ranges and a bootstrap and regression run on a Cortex-A15 - a v7ve core that has hardware divide instructions. A patch for AArch64 follows.

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

2015-05-19 Thread H.J. Lu
On Tue, May 19, 2015 at 8:21 AM, Joseph Myers jos...@codesourcery.com wrote: On Mon, 18 May 2015, H.J. Lu wrote: Have updates for all affected specs for all targets been posted? I just saw a small and apparently arbitrary subset of targets with patches, and no explanation of how those

Re: [PATCH, alpha]: Some cleanups

2015-05-19 Thread Uros Bizjak
On Tue, May 19, 2015 at 5:10 PM, Uros Bizjak ubiz...@gmail.com wrote: No functional changes. 2015-05-18 Uros Bizjak ubiz...@gmail.com * config/alpha/alpha.c (alpha_legitimize_reload_address) (alpha_preferred_reload_class, alpha_legitimate_constant_p): Use CONST_INT_P,

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

2015-05-19 Thread Paul_Koning
From: gcc-patches-ow...@gcc.gnu.org [gcc-patches-ow...@gcc.gnu.org] on behalf of H.J. Lu [hjl.to...@gmail.com] Sent: Tuesday, May 19, 2015 11:27 AM To: Joseph Myers Cc: Magnus Granberg; GCC Patches Subject: Re: PING^3: [PATCH]: New configure options that

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

2015-05-19 Thread Joseph Myers
On Tue, 19 May 2015, H.J. Lu wrote: I think the whole thing should be posted as one patch, with both the target-independent changes and the target-specific changes for all targets. That is what makes me concerned. I have some simple target-specified patches which weren't reviewed for

Re: [PATCH 2/4] prologue and epilogue tidy and -mno-vrsave bug fix

2015-05-19 Thread David Edelsohn
On Sun, May 17, 2015 at 10:54 PM, Alan Modra amo...@gmail.com wrote: This patch tidies the prologue and epilogue altivec code a little. A number of places using info-altivec_size unnecessarily also test TARGET_ALTIVEC_ABI, when rs6000_stack_info() guarantees that info-altivec_size is zero if

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Jeff Law
On 05/19/2015 08:43 AM, Michael Matz wrote: Hi, On Fri, 15 May 2015, Rich Felker wrote: Forget lazy binding. It's dead anyway because serious distros want PIE+relro+bindnow+... You keep saying this, but I can't help the feeling it's mostly because musl doesn't support it ;-) FWIW, Red Hat

Re: [PATCH 4/4] Split-stack arg pointer init refinement

2015-05-19 Thread David Edelsohn
This small refinement to the -fsplit-stack prologue arg pointer initialization improves code generation. Compare the -O2 gcc/testsuite/gcc.dg/split-3.c code for down() below. beforeafter mflr 0mflr 0 std 31,-8(1)std 31,-8(1) std 0,16(1)mr 12,1

Re: [PATCH 02/13] optabs: Fix vec_perm - V16QI middle end lowering.

2015-05-19 Thread Richard Henderson
On 05/19/2015 01:41 AM, Andreas Krebbel wrote: On 05/18/2015 07:35 PM, Richard Henderson wrote: On 05/11/2015 06:23 AM, Andreas Krebbel wrote: @@ -6784,14 +6784,18 @@ expand_vec_perm (machine_mode mode, rtx v0, rtx v1, rtx sel, rtx target) { /* Multiply each element by its byte

breakage with series [0/9] Record number of hard registers in a REG

2015-05-19 Thread Hans-Peter Nilsson
From: Richard Sandiford richard.sandif...@arm.com Date: Mon, 18 May 2015 20:09:19 +0200 While looking at a profile of gcc, I noticed one thing fairly high up the list was a loop iterating over all the registers in a REG, apparently due to the delay in computing the index for hard_regno_nregs

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

2015-05-19 Thread H.J. Lu
On Tue, May 19, 2015 at 8:33 AM, paul_kon...@dell.com wrote: From: gcc-patches-ow...@gcc.gnu.org [gcc-patches-ow...@gcc.gnu.org] on behalf of H.J. Lu [hjl.to...@gmail.com] Sent: Tuesday, May 19, 2015 11:27 AM To: Joseph Myers Cc: Magnus Granberg;

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Michael Matz
Hi, On Fri, 15 May 2015, Rich Felker wrote: Forget lazy binding. It's dead anyway because serious distros want PIE+relro+bindnow+... You keep saying this, but I can't help the feeling it's mostly because musl doesn't support it ;-) No, you don't have to use bindnow to get the effects of

[PATCH, alpha]: Some cleanups

2015-05-19 Thread Uros Bizjak
No functional changes. 2015-05-18 Uros Bizjak ubiz...@gmail.com * config/alpha/alpha.c (alpha_legitimize_reload_address) (alpha_preferred_reload_class, alpha_legitimate_constant_p): Use CONST_INT_P, CONST_SCALAR_INT_P and CONST_DOUBLE_P predicates. (alpha_split_reload_pair)

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

2015-05-19 Thread Joseph Myers
On Mon, 18 May 2015, H.J. Lu wrote: Have updates for all affected specs for all targets been posted? I just saw a small and apparently arbitrary subset of targets with patches, and no explanation of how those targets were identified or why the other targets with specs mentioning the

RE: Refactor gimple_expr_type

2015-05-19 Thread Aditya K
Date: Tue, 19 May 2015 11:33:16 +0200 Subject: Re: Refactor gimple_expr_type From: richard.guent...@gmail.com To: hiradi...@msn.com CC: tbsau...@tbsaunde.org; gcc-patches@gcc.gnu.org On Tue, May 19, 2015 at 12:04 AM, Aditya K hiradi...@msn.com

[ping**2] Handle MULTILIB_REUSE in auto-generated SYSROOT_SUFFIX_SPEC macro

2015-05-19 Thread Sandra Loosemore
Re-pinging a patch from last year that never got reviewed: https://gcc.gnu.org/ml/gcc-patches/2014-06/msg00511.html This problem still exists in GCC 5.1 and the above patch still fixes it. I haven't tried mainline head yet, but it doesn't look like anything else has touched this since we

[committed] linear/lastprivate clause fixes

2015-05-19 Thread Jakub Jelinek
Hi! When working on taskloop, I've noticed various issues in the OpenMP 4.0 handling of the linear/lastprivate (explicit as well as implicit) clauses. Bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk, plan to backport to 5/4.9 after a while. 2015-05-19 Jakub Jelinek

Re: [RFC] COMDAT Safe Module Level Multi versioning

2015-05-19 Thread Sriraman Tallam
On Tue, May 19, 2015 at 2:39 AM, Richard Biener richard.guent...@gmail.com wrote: On Tue, May 19, 2015 at 8:16 AM, Sriraman Tallam tmsri...@google.com wrote: We have the following problem with selectively compiling modules with -misa options and I have provided a solution to solve this. I

RE: [PATCH, MIPS]: Fix internal compiler error: in check_bool_attrs, at recog.c:2218 for micromips attribute

2015-05-19 Thread Robert Suchanek
Hi, The original patch had a missing declaration of micromips_globals in mips.h that appears to be the cause of segmentation faults when building mips-mti-linux-gnu. I didn't get any failures just before the submission neither on mips-img-linux-gnu nor mips64el-linux-gnu and the test case is

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Richard Henderson
On 05/19/2015 11:06 AM, Rich Felker wrote: I'm still mildly worried that concerns for supporting relaxation might lead to decisions not to optimize code in ways that would be difficult to relax (e.g. certain types of address load reordering or hoisting) but I don't understand GCC internals

Re: [Patch] [AArch64] PR target 66049: fix add/extend gcc test suite failures

2015-05-19 Thread Kyrill Tkachov
Hi Venkat, On 19/05/15 16:37, Kumar, Venkataramanan wrote: Hi Maintainers, Please find the attached patch, that fixes add/extend gcc test suite failures in Aarch64 target. Ref: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66049 These tests started to fail after we prevented combiner from

[Patch] [AArch64] PR target 66049: fix add/extend gcc test suite failures

2015-05-19 Thread Kumar, Venkataramanan
Hi Maintainers, Please find the attached patch, that fixes add/extend gcc test suite failures in Aarch64 target. Ref: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66049 These tests started to fail after we prevented combiner from converting shift RTX to mult RTX, when the RTX is not inside

Re: [RFC] COMDAT Safe Module Level Multi versioning

2015-05-19 Thread Xinliang David Li
Hm. But which options are unsafe? Also wouldn't it be better to simply _not_ have unsafe options produce comdats but always make local clones for them (thus emit the comdat with unsafe flags dropped)? Always localize comdat functions may lead to text size increase. It does not work if the

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

2015-05-19 Thread Andre Vehreschild
Hi, attached is the most recent version of the patch for 58586. It adapts to recent trunk and addresses the caveats so far, i.e. the testcases in the comments now compile and run again w/o errors. Bootstraps and regtests fine on x86_64-linux-gnu/f21. Comments? - Andre On Fri, 8 May 2015

Re: [Patch, fortran, pr65548, 2nd take, v5] [5/6 Regression] gfc_conv_procedure_call

2015-05-19 Thread Mikael Morin
Le 19/05/2015 10:50, Andre Vehreschild a écrit : Hi all, find attached latest version to fix 65548. Bootstraps and regtests ok on x86_64-linux-gnu/f21. OK. Thanks. Mikael

Re: [RFC] COMDAT Safe Module Level Multi versioning

2015-05-19 Thread Yury Gribov
On 05/19/2015 09:16 AM, Sriraman Tallam wrote: We have the following problem with selectively compiling modules with -misa options and I have provided a solution to solve this. I would like to hear what you think. Multi versioning at module granularity is done by compiling a subset of modules

Re: [RFC]: Remove Mem/address type assumption in combiner

2015-05-19 Thread Jeff Law
On 05/16/2015 07:55 PM, Hans-Peter Nilsson wrote: On Sat, 16 May 2015, Segher Boessenkool wrote: On Sat, May 16, 2015 at 12:36:38PM -0400, Hans-Peter Nilsson wrote: On Sat, 16 May 2015, Segher Boessenkool wrote: On Fri, May 15, 2015 at 10:40:48PM -0400, Hans-Peter Nilsson wrote: I confess

Re: [PATCH] Add SPECIAL_FLOAT_MODE to enable adding IEEE 128-bit floating point to PowerPC

2015-05-19 Thread Michael Meissner
On Fri, May 08, 2015 at 01:05:59PM -0600, Jeff Law wrote: On 05/06/2015 11:29 AM, Michael Meissner wrote: On Wed, May 06, 2015 at 04:03:00PM +0100, Richard Sandiford wrote: Jeff Law l...@redhat.com writes: So my worry here is that folks writing these loops to iterate over modes are going to

ODR merging and implicit typedefs

2015-05-19 Thread Jan Hubicka
Jason, I just noticed that there are bogus ODR violation warnings during LTO-bootstrap (that breaks -Werror builds). It was caused by my work-around for type_in_anonymous_namespace for the issue discussed in: https://gcc.gnu.org/ml/gcc-patches/2015-05/msg01245.html (i.e. the TYPE_STUB_DECL

Re: ODR merging and implicit typedefs

2015-05-19 Thread Jason Merrill
On 05/19/2015 01:33 PM, Jan Hubicka wrote: I tracked down that those are implicit typedef created by create_implicit_typedef. My patch made them no longer anonymous that in turn triggers the bogus diagnostics. I do not think it is fully correct though - those types are not anonymous. Hmm?

Re: ODR merging and implicit typedefs

2015-05-19 Thread Jan Hubicka
On 05/19/2015 01:33 PM, Jan Hubicka wrote: I tracked down that those are implicit typedef created by create_implicit_typedef. My patch made them no longer anonymous that in turn triggers the bogus diagnostics. I do not think it is fully correct though - those types are not anonymous.

Re: Fwd: Re: [PATCH, RFC]: Next stage1, refactoring: propagating rtx subclasses

2015-05-19 Thread Jeff Law
On 05/18/2015 11:07 PM, Mikhail Maltsev wrote: It seems rather weird to me: I have definitely sent this message, but now I realized that it did not get to the mailing list archive (and it was not bounced either). Thus, resending. I got the original, not sure why it didn't show up in the

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Richard Henderson
On 05/19/2015 12:35 PM, Rich Felker wrote: Why would you recompute it (this requires a fairly expensive call that reads or pops its own return address) rather than simply spilling the already-computed value and reloading it from the stack? The only example I can think of where it might make

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread H.J. Lu
On Tue, May 19, 2015 at 1:15 PM, Rich Felker dal...@libc.org wrote: On Tue, May 19, 2015 at 12:17:18PM -0700, H.J. Lu wrote: On Tue, May 19, 2015 at 12:11 PM, Richard Henderson r...@redhat.com wrote: On 05/19/2015 12:06 PM, H.J. Lu wrote: On Tue, May 19, 2015 at 11:59 AM, Richard Henderson

  1   2   >