RE: [patch, avr] extend part-clobbered check to AVR_TINY architecture

Ping! > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Sivanupandi, Pitchumani > Sent: Tuesday, April 21, 2015 8:21 PM > To: Georg-Johann Lay; Denis Chertykov > Cc: GCC Patches > Subject: [patch, avr] extend part-clobbered ch

C++ PATCH for c++/59012 (alignas pack expansion)

When we added support for C++11 attributes, we mentioned pack expansions in the comments but didn't do anything to support them in the code. I'm not sure how that happened. Anyway, this patch adds that support. Tested x86_64-pc-linux-gnu, applying to trunk. commit 9aa0f9dc3731adcd64b9a1670c17

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

This is a separate issue which really shouldn't have anything to do with this, but is there a specific reason why: void good1(int x, int y) { _Bool pf; asm("cmpl %2,%1" : "=@ccp" (pf) : "r" (x), "g" (y)); if (pf) beta(); } ... ends up generating a jump to a jump?

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

On 05/07/2015 02:38 PM, Richard Henderson wrote: > Here's a prototype for i386 only, which stands up to light testing. > I'd rather post this tonight rather than wait until tomorrow when I > can write more proper dejagnu tests. > > I've tested the intermedate patches via config-list.mk, so despite

[patch 10/10] debug-early merge: compiler proper

As seen on TV. gcc/ * dwarf2out.c: Remove deferred_locations*. (dwarf2_debug_hooks): Add early_finish hook. Remove global_decl hook. Add early_global_decl and late_global_decl hook. Add dumped_early bit to die_struct. New global early_dwarf_dumping. (struct limbo_die_struct): Document crea

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

gcc/testsuite/ * g++.dg/cpp0x/pr57101.C: Add warning. * g++.dg/debug/dwarf2/dwarf4-typedef.C: Same. * g++.dg/ext/visibility/anon7.C: Same. * g++.dg/ipa/pr63587-2.C: Same. * g++.dg/opt/dump1.C: Same. * g++.dg/opt/pr59622-3.C: Same. * g++.dg/opt/pr59622.C: Same. * g++.dg/torture/pr46383.C:

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

gcc/objc/ * objc-act.c (objc_write_global_declarations): Adjust comment. * objc-gnu-runtime-abi-01.c (build_selector_table_decl): Set TREE_USED bit. diff --git a/gcc/objc/objc-act.c b/gcc/objc/objc-act.c index 718fdf0..3528c89 100644 --- a/gcc/objc/objc-act.c +++ b/gcc/objc/objc-act.c @@ -439

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

gcc/lto/ * lto-lang.c (lto_write_globals): Remove. (LANG_HOOKS_WRITE_GLOBALS): Same. diff --git a/gcc/lto/lto-lang.c b/gcc/lto/lto-lang.c index 073bf17..094ed1f 100644 --- a/gcc/lto/lto-lang.c +++ b/gcc/lto/lto-lang.c @@ -1116,19 +1116,6 @@ lto_getdecls (void) return NULL_TREE; } -static

[patch 6/10] debug-early merge: Java front-end

gcc/java/ * class.c: Remove pending_static_fields. (add_field): Remove use of pending_static_fields. (java_write_globals): Remove. * decl.c (java_mark_class_local): Remove use of pending_static_fields. * java-tree.h: Remove pending_static_fields and java_write_globals. * jcf-parse.c (java

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

gcc/go/ * go-gcc.cc (write_global_definitions): Remove call to finalize_compilation_unit. Remove Go specific debug generation. * go-lang.c (go_langhook_parse_file): Call go_write_globals. (go_langhook_write_globals): Remove. Remove LANG_HOOKS_WRITE_GLOBALS everywhere. diff --git a/gcc/go/g

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

gcc/fortran/ * f95-lang.c (gfc_write_global_declarations): Remove. (LANG_HOOKS_WRITE_GLOBALS): Remove. (gfc_write_global_declarations): Move code from here to... (gfc_be_parse_file): ...here. Call global_decl_processing. * trans-decl.c (gfc_emit_parameter_debug_info): Rename global_decl to

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

gcc/cp/ * cp-objcp-common.c: Adjust comment for cxx_warn_unused_global_decl. * cp-objcp-common.h (LANG_HOOKS_WRITE_GLOBALS): Remove (LANG_HOOKS_POST_COMPILATION_PARSING_CLEANUPS): New. * cp-tree.h (note_mangling_alias): Protoize. (cp_write_global_declarations): Remove. (cxx_post_compilatio

[patch 2/10] debug-early merge: C front-end (include c-family/)

gcc/c-family/ * c-common.h (c_parse_final_cleanups): New prototype. * c-opts.c (c_common_parse_file): Call c_parse_final_cleanups. gcc/c/ * c-decl.c (finish_struct): Save C_TYPE_INCOMPLETE_VARS and immediately clobber it. (c_write_global_declarations_1): Remove call to check_global_declar

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

gcc/ada/ * gcc-interface/gigi.h (note_types_used_by_globals): New prototype. * gcc-interface/misc.c (gnat_parse_file): Call note_types_used_by_globals. * gcc-interface/utils.c: Remove global_decls. (gnat_pushdecl): Remove use of global_decls. (gnat_write_global_declarations): Rename to no

[patch 0/10] debug-early merge

Hi folks. I have divided the patches into 10 pieces. The patches are interdependent and cannot be applied independently. I am merely dividing them up to aid the relevant reviewers. As I've mentioned elsewhere, the patchset as posted has been bootstrapped and GCC tested on: x86_64

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

I noticed this while reading the match-and-simplify docs. The gimple_simplify API list has two built-in function cases with three tree args. The last one is supposed to have four tree args for a ternary function (3 inputs and one output). Similarly, in the gimple_build API list, the built-in ter

Re: update docs for --enable-languages

ping https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01690.html Jim On Mon, Apr 27, 2015 at 5:09 PM, Jim Wilson wrote: > I noticed this while working on my mostlyclean patch. The list of > languages in the docs for --enable-languages is incomplete. It is > missing jit and lto. I also noticed t

[PATCH, ARM, doc] add missing -mtune options

I noticed that the list of -mtune options in the arm-cores.def file didn't match the list in the doc/invoke.texi file. There are 3 cores missing: generic-armv7-a, cortex-a17, and cortex-a17.cortex-a7. This patch adds the missing cores to the docs. Jim 2015-05-07 Jim Wilson * doc/invoke.texi

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

On 05/06/15 01:58, Michael Haubenwallner wrote: Trivial patch for fixincludes. A) sufficiently trivial that explicit permission ought not be required B) it is now officially blessed that we can coalesce year lists. Let's do so, okay? Am 2015-05-05 um 18:03 schrieb Michael Haubenwallner: H

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

On Mon, 20 Apr 2015, Pavel Kopyl wrote: > 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

Re: [PATCH, stage1] Better error recovery for merge-conflict markers

On Fri, 17 Apr 2015, David Malcolm wrote: > gcc/c-family/ChangeLog: > * c-common.h (conflict_marker_get_final_tok_kind): New prototype. > * c-lex.c (conflict_marker_get_final_tok_kind): New function. > > gcc/c/ChangeLog: > * c-parser.c (struct c_parser): Expand array "tokens_buf

[PATCH 4/6] Convert to md_asm_adjust

Using proper vectors instead of lists of trees. --- gcc/cfgexpand.c | 614 --- gcc/config/cris/cris.c | 88 +++ gcc/config/i386/i386.c | 24 +- gcc/config/mn10300/mn10300.c | 20 +- gcc/config/rs6000/rs6000.c | 17 +- gcc/c

[PATCH 6/6] i386: Implement asm flag outputs

All j mnemonics implemented as =@cc to make it easy for someone reading the manual to figure out what condition is desired. --- gcc/config/i386/constraints.md | 5 ++ gcc/config/i386/i386.c | 132 ++--- 2 files changed, 130 insertions(+), 7 deletions(-

[PATCH 5/6] i386: Add CCPmode

For testing parity coming out of asm flags. --- gcc/config/i386/i386-modes.def | 2 ++ gcc/config/i386/i386.c | 19 +++ 2 files changed, 13 insertions(+), 8 deletions(-) diff --git a/gcc/config/i386/i386-modes.def b/gcc/config/i386/i386-modes.def index 0b6a1f1..714c138 10

[PATCH 3/6] Canonicalize asm volatility earlier

If gimple_asm_volatile_p is correct, no point re-checking. This is also done by the C and C++ front ends, but not Ada. So we can't yet trust ASM_VOLATILE_P from the front end. --- gcc/cfgexpand.c | 11 +++ gcc/gimplify.c | 2 +- 2 files changed, 4 insertions(+), 9 deletions(-) diff --gi

[PATCH 2/6] Merge expand_asm_operands into expand_asm_stmt

Prepatory to converting from tree chains to vectors. --- gcc/cfgexpand.c | 167 1 file changed, 72 insertions(+), 95 deletions(-) diff --git a/gcc/cfgexpand.c b/gcc/cfgexpand.c index c77a4ac..a5de512 100644 --- a/gcc/cfgexpand.c +++ b/gcc/c

[PATCH 1/6] Only resolve_asm_operand_names once

We do it in the front end already; no need to repeat. --- gcc/cfgexpand.c | 2 -- gcc/stmt.c | 7 --- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/gcc/cfgexpand.c b/gcc/cfgexpand.c index 5905ddb..c77a4ac 100644 --- a/gcc/cfgexpand.c +++ b/gcc/cfgexpand.c @@ -2554,8 +2554

[RFC 0/6] Flags outputs for asms

Here's a prototype for i386 only, which stands up to light testing. I'd rather post this tonight rather than wait until tomorrow when I can write more proper dejagnu tests. I've tested the intermedate patches via config-list.mk, so despite mucking around with vec.h vs target.h, all targets still c

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

On Fri, 6 Mar 2015, H.J. Lu wrote: > +# We don't want to compile the compiler with -fPIE, it make PCH fail. > +COMPILER += @NO_PIE_CFLAGS@ > + > +# Link with -no-pie since we compile the compiler with -fno-PIE. > +LINKER += @NO_PIE_FLAG@ As I understand it, what we don't want is the compiler to b

RE: Patch: Refactor number_of_iterations_exit

> Subject: Re: Patch: Refactor number_of_iterations_exit > From: richard.guent...@gmail.com > Date: Thu, 7 May 2015 19:47:37 +0200 > To: hiradi...@msn.com; gcc-patches@gcc.gnu.org > > On May 7, 2015 6:27:28 PM GMT+02:00, Aditya K wrote: >>Hi, >>I refactor

Re: C PATCH to add -Woverride-init-side-effects (PR c/64918)

On Fri, 24 Apr 2015, Marek Polacek wrote: > As discussed in the PR, the "initialized field with side-effects overwritten" > warning is sometimes not so useful, so it probably makes sense to provide an > option so that users are able to specifically enable/disable it. Since the > warning is enable

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

Richard, Agree that the code is handling a very special case but this special case is common enough and is limiting the vectorizer in a significant way. The special case is: loops with unsigned index, non-constant start value, and step 1. We have a code for a matrix multiply – loops blocked by han

Re: [PATCH diagnostics/fortran] Handle two locations for the same diagnostic. Convert all gfc_warning_1 and gfc_notify_std_1 calls

Hello Manuel, Sorry for my late reply, and thank you very much for working on this. I have looked at the patch and I like it! I guess I just have some few lateral nits to pick. > The Fortran FE allows diagnostics with two different locations. > Depending on whether these locations are on the sa

RE: Fix compiler warnings

Thank you very much Jakub. -Aditya > Date: Thu, 7 May 2015 19:58:11 +0200 > From: ja...@redhat.com > To: hiradi...@msn.com > CC: gcc-patches@gcc.gnu.org > Subject: Re: Fix compiler warnings > > On Wed, May 06, 2015 at 04:37:49PM +, Aditya K wrote: >>

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

On Thu, May 07, 2015 at 11:23:51AM -0600, Sandra Loosemore wrote: > How about making the new Enumerator Attributes node a subsection of the Type > Attributes section, instead of a section of its own at the same level? Sorry, I don't particularly like this idea. I think an enumerator is not close

Re: [rfc] gcc trunk - libgomp thread affinity for freebsd

On 7 May 2015 at 00:57, Jakub Jelinek wrote: > On Wed, May 06, 2015 at 04:29:02PM -0700, Adrian Chadd wrote: >> This patch implements basic top level thread affinity support for >> freebsd. It doesn't yet implement thread affinity support for >> core/socket grouping yet; I'm working on a library t

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

On Thu, May 07, 2015 at 12:00:20PM -0600, Jeff Law wrote: > OK. Please install if you haven't already. I have not, so will do momentarily. Thanks, Marek

Re: [PATCH] Expand PIC calls without PLT with -fno-plt

On Thu, May 7, 2015 at 11:22 AM, Jeff Law wrote: > On 05/06/2015 09:24 AM, Alexander Monakov wrote: >> >> On Mon, 4 May 2015, Jeff Law wrote: >>> >>> On 05/04/2015 11:39 AM, Jakub Jelinek wrote: On Mon, May 04, 2015 at 11:34:05AM -0600, Jeff Law wrote: > > On 05/04/2015 10:37 AM,

Re: acc_on_device for device_type_host_nonshm (was: libgomp nvptx plugin: rework initialisation and support the proposed load/unload hooks) (PR65742)

On Fri, 17 Apr 2015 15:16:19 +0200 Jakub Jelinek wrote: > On Tue, Apr 14, 2015 at 05:43:26PM +0200, Thomas Schwinge wrote: > > On Tue, 14 Apr 2015 15:15:02 +0100, Julian Brown > > wrote: > > > On Wed, 8 Apr 2015 17:58:56 +0300 > > > Ilya Verbin wrote: > > > > I see several regressions: > > > >

Re: [PATCH] Expand PIC calls without PLT with -fno-plt

On 05/06/2015 09:24 AM, Alexander Monakov wrote: On Mon, 4 May 2015, Jeff Law wrote: On 05/04/2015 11:39 AM, Jakub Jelinek wrote: On Mon, May 04, 2015 at 11:34:05AM -0600, Jeff Law wrote: On 05/04/2015 10:37 AM, Alexander Monakov wrote: This patch introduces option -fno-plt that allows to exp

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

On 05/06/2015 05:36 AM, Szabolcs Nagy wrote: On 30/04/15 00:18, Joseph Myers wrote: On Wed, 29 Apr 2015, Szabolcs Nagy wrote: only affects [u]int_fastN_t types (on 64bit systems for N=16,32 musl uses int but glibc uses long) i can fix glibc-stdint.h, but it's yet another way in which the comp

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

On May 7, 2015, at 4:39 AM, Jakub Jelinek wrote: > On Thu, May 07, 2015 at 01:26:57PM +0200, Rainer Orth wrote: >> As reported in the PR, with the addition of all those OpenACC tests, >> libgomp make check times have skyrocketed since the testsuite is still >> run sequentially. >> >> Even on a re

Re: [PATCH] PR debug/61352 back port from mainline

> > The attached patch is a back port of the change from > > https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=211067 for > > gcc-4_9-branch. Bootstrap and regression tested on > > x86_64-apple-darwin14 with Xcode 6.3. Okay for gcc-4_9-branch? > > Jack > > > > Ok. > > Committed

Re: [PATCH 2/13] musl libc config

On 05/06/2015 05:24 AM, Szabolcs Nagy wrote: On 29/04/15 00:30, Joseph Myers wrote: On Mon, 20 Apr 2015, Szabolcs Nagy wrote: * config/linux.opt (mmusl): New option. New -m options need documenting in invoke.texi. Patch v3. Now with documentation in invoke.texi. Based on previo

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

On 05/06/2015 05:37 AM, Marek Polacek wrote: On Wed, Apr 29, 2015 at 10:54:58PM +, Joseph Myers wrote: On Mon, 27 Apr 2015, Marek Polacek wrote: trigger by default. One change is that we reject programs that use shift with undefined behavior in a context where a constant expression is req

Re: Fix compiler warnings

On Wed, May 06, 2015 at 04:37:49PM +, Aditya K wrote: > Thanks! Updated patch. > > > 2015-05-06  Aditya Kumar  > >     * gcov-tool.c (do_merge): Refactored to remove int ret. >     * ipa-icf.c (sem_item::hash_referenced_symbol_properties): Changed > (!type == FUNC) to (type != FUNC

RFC: Patch to align spills beyond what the stack supports

I would like to get some feedback on an idea of how to spill registers that require (or perhaps only prefer for performance reasons) an alignment greater than that supported by the stack. If you look at how GCC supports local variables with alignment requirements greater than the stack supports, t

Re: Patch: Refactor number_of_iterations_exit

On May 7, 2015 6:27:28 PM GMT+02:00, Aditya K wrote: >Hi, >I refactored number_of_iterations_exit a little bit. I hope it is >helpful. >The idea is to move the call to function dominated_by_p after some >sanity checks so as to avoid call to it. The call is very cheap though. Do you have data tha

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

On 05/07/2015 10:22 AM, Marek Polacek wrote: [snip] * doc/extend.texi (Enumerator Attributes): New section. Document syntax of enumerator attributes. How about making the new Enumerator Attributes node a subsection of the Type Attributes section, instead of a section of its

Re: Vectorize stores with unknown stride

NP, and sorry for the spurious comments, hadn't spotted you were using nunits. I like the testcase, thanks :). A. Michael Matz wrote: On Thu, 7 May 2015, Alan Lawrence wrote: Also update comment? (5 identical cases) Also update comment? Obviously a good idea, thanks :) (s/loads/accesses/

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

Hi Bernd, On 05/06/2015 03:01 PM, Bernd Edlinger wrote: > On Tue, 5 May 2015 18:03:15, Michael Haubenwallner wrote: >> >> Now that gcc-5 is out, what about an automake-1.11.6 update for gcc-6? >> >> BTW, the actual commands I use to re-run automake for everything (I found) >> is: >> $ export AUTO

Re: C++ PATCH for c++/51747 (list-initialization from same type)

We also need to adjust digest_init_r. Tested x86_64-pc-linux-gnu, applying to trunk and 5. commit 23cbda06982fbaf061719dff124ae90c3b033d7e Author: Jason Merrill Date: Wed May 6 17:11:54 2015 -0500 DR 1467 PR c++/51747 * typeck2.c (digest_init_r): Fix single element list. diff

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

Le 07/05/2015 11:52, Andre Vehreschild a écrit : > Hi all, > > my work on pr60322 caused a regression on trunk. This patch fixes it. The > regression had two causes: > 1. Not taking the correct attribute for BT_CLASS objects with allocatable >components into account (chunk 1), and > 2. taking

Patch: Refactor number_of_iterations_exit

Hi, I refactored number_of_iterations_exit a little bit. I hope it is helpful. The idea is to move the call to function dominated_by_p after some sanity checks so as to avoid call to it. Thanks, -Aditya 2015-05-07  Aditya Kumar      * tree-ssa-loop-niter.c (number_of_iterations_exit): Mo

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

On Thu, May 07, 2015 at 10:21:28AM -0400, Ed Smith-Rowland wrote: > Instead of NULL_TREE in pt.c I grabbed the attrs. > > /* Actually build the enumerator itself. */ > build_enumerator > (DECL_NAME (decl), value, newtag, DECL_ATTRIBUTES (decl), > DECL_SOURCE_LOCATION (decl));

Re: Vectorize stores with unknown stride

On Thu, 7 May 2015, Alan Lawrence wrote: > Also update comment? (5 identical cases) > > Also update comment? Obviously a good idea, thanks :) (s/loads/accesses/ for the commit) > > @@ -5013,7 +5025,7 @@ vectorizable_store (gimple stmt, gimple_stmt_iterator > > *gsi, gimple *vec_stmt, > >tr

Re: OpenACC: initialization with unsupported acc_device_t (was: [PR testsuite/65205, libgomp/65993] Fix dg-shouldfail usage in OpenACC libgomp tests)

On Tue, 5 May 2015 16:09:18 +0200 Thomas Schwinge wrote: > Hi! > > On Tue, 5 May 2015 08:43:48 -0400, John David Anglin > wrote: > > On 2015-05-05 5:43 AM, Thomas Schwinge wrote: > > >> FAIL: libgomp.oacc-c/../libgomp.oacc-c-c++-common/lib-62.c > > >> >-DACC_DEVICE_TYPE_hos > > >> >t=1 -DACC_ME

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

On 05/06/2015 04:09 PM, Jeff Law wrote: > On 05/05/2015 10:03 AM, Michael Haubenwallner wrote: >> Hello build machinery maintainers, >> >> following up >> http://thread.gmane.org/gmane.comp.gcc.patches/331902/focus=334462 >> http://thread.gmane.org/gmane.comp.gcc.patches/332160 >> > [ ... snip ...

Re: [PATCH] Make mergeable read-only sections per-function, if requested.

On Thu, May 07, 2015 at 07:31:33AM -0600, Jeff Law wrote: > If you could cobble together a little test (ppc specific is fine with > me) it'd be appreciated. > > With that, approved for the trunk. I did one for *-*-linux*. Tested on powerpc64-linux and x86_64-linux. Committed. Segher 2015-05

[gomp4.1] Taskloop construct lowering and expansion

Hi! This patch gimplifies OMP_TASKLOOP as 3 different gimple statements - two GIMPLE_OMP_FOR with GIMPLE_OMP_TASK sandwiched in between them, so that we can represent properly the effect of what happens in the GOMP_taskloop caller (needs to compute number of iterations, prepare counts of collapsed

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

On Thu, 7 May 2015, Bernd Edlinger wrote: > But that is not the case for other tool scripts.  I think these should > be in-sync with the automake version that creates the configure scripts > that make use of them. At least some of these scripts are also usable other than from automake-generate

Re: [debug-early] call debug_hooks->late_global_decl for optimized away symbols

And now for my next trick, I will revert all changes I had made to the non-dwarf backends. No special kludges will be needed for stabs et al, with my previous patch in place. Tested on: x86_64-unknown-linux-gnu powerpc-ibm-aix7.1.2.0 powerpc64-unknown-linux-gnu aarch64-unknow

Re: Remove mode argument from gen_rtx_SET

On 05/07/2015 07:59 AM, Richard Sandiford wrote: Jeff Law writes: On 05/07/2015 05:37 AM, Richard Sandiford wrote: One problem with the automatically-generated gen_rtx_FOO () macros is that they always have a mode parameter, even for codes like SET where the mode should always be VOIDmode. Th

Re: [AArch64/GCC] PR64304, miscompilation with -mgeneral-regs-only

On 14/01/15 15:31, Jiong Wang wrote: > as discussed here https://gcc.gnu.org/ml/gcc-patches/2015-01/msg00563.html > > the problem is aarch64 hardware only support left shift truncation for SI/DI, > while SHIFT_COUNT_TRUNCATED is enabled for all mode including QI/HI, which is > inconsistent with ha

[PATCH] Fix 65697. Add memory model support for stronger __sync operations.

There has been some debate over the strength requirement of barriers for __sync operations... This is documented within the PR. Originally __sync was suppose to be synonymous with SEQ_CST, but there has been a slight slackening of the barrier-ness of SEQ_CST from the language lawyers. Under so

Re: Fix ubsan tests on ARM (PR testsuite/66046)

On Thu, May 07, 2015 at 05:35:30PM +0300, Maxim Ostapenko wrote: > This works fine for me on target ARM board, thanks for fixing this! > > -Maxim > On 07/05/15 13:41, Marek Polacek wrote: > >Trailing \[^\n\r]*(\n|\r\n|\r) in the last dg-output aroused dismay on ARM. > > > >Here's a patch to clean

Re: Fix ubsan tests on ARM (PR testsuite/66046)

This works fine for me on target ARM board, thanks for fixing this! -Maxim On 07/05/15 13:41, Marek Polacek wrote: Trailing \[^\n\r]*(\n|\r\n|\r) in the last dg-output aroused dismay on ARM. Here's a patch to clean this up; can you please test this on ARM? make check-g{cc,++} RUNTESTFLAGS='--t

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

The patches to gotools and libbacktrace are fine if the rest of the patches are approved. I will handle the patch to libgo, which is copied from a different repository. Please do not commit it yourself. Thanks. Ian Michael Haubenwallner writes: > Patch for gotools. > > Am 2015-05-05 um 18:0

Re: Vectorize stores with unknown stride

(Below are all minor/style points only, no reason for patch not to go in.) Michael Matz wrote: diff --git a/gcc/tree-vect-data-refs.c b/gcc/tree-vect-data-refs.c index 96afc7a..6d8f17e 100644 --- a/gcc/tree-vect-data-refs.c +++ b/gcc/tree-vect-data-refs.c @@ -665,7 +665,7 @@ vect_compute_data_re

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

On 05/07/2015 09:59 AM, Marek Polacek wrote: On Wed, May 06, 2015 at 11:17:20PM -0400, Ed Smith-Rowland wrote: In addition to a PR this is 1/2 of a C=+17 feature. (The other half - really a separate thing - is attributes on namespaces). Ah, nice, I wasn't aware. For the record, this is

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

On Wed, May 06, 2015 at 11:17:20PM -0400, Ed Smith-Rowland wrote: > In addition to a PR this is 1/2 of a C=+17 feature. (The other half - really > a separate thing - is attributes on namespaces). Ah, nice, I wasn't aware. For the record, this is

Re: Remove mode argument from gen_rtx_SET

Jeff Law writes: > On 05/07/2015 05:37 AM, Richard Sandiford wrote: >> One problem with the automatically-generated gen_rtx_FOO () macros >> is that they always have a mode parameter, even for codes like SET >> where the mode should always be VOIDmode. This inevitably leads to >> cases where a ca

Re: Vectorize stores with unknown stride

On Wed, May 6, 2015 at 5:37 PM, Michael Matz wrote: > Hi, > > I'm sitting on this since quite some time already and always missed stage > 1. This implements support for vectorizing strided stores with unknown > but loop invariant stride, like: > > sumit (float * __restrict dest, >float *

Re: Remove mode argument from gen_rtx_SET

On 05/07/2015 05:37 AM, Richard Sandiford wrote: One problem with the automatically-generated gen_rtx_FOO () macros is that they always have a mode parameter, even for codes like SET where the mode should always be VOIDmode. This inevitably leads to cases where a caller accidentally passes somet

Re: [PATCH] Make mergeable read-only sections per-function, if requested.

On 05/07/2015 06:30 AM, Segher Boessenkool wrote: On Thu, May 07, 2015 at 10:15:08AM +0200, Marek Polacek wrote: +const char * +function_mergeable_rodata_prefix (void) This function needs a comment. And much more importantly, I forgot to make it static. Fixed. +{ + section *s = targetm.a

Re: [PATCH] Make mergeable read-only sections per-function, if requested.

On 05/07/2015 02:15 AM, Marek Polacek wrote: On Wed, May 06, 2015 at 03:59:33PM -0700, Segher Boessenkool wrote: Currently GCC does not put mergeable read-only data in a per-function section, so the -gc-sections linker option does not do much for such data. Fix that. Dunno if it is a good ide

Minor operand shortening patch reapplied

I found the i686 bootstrap comparison failure reported by HJ after the minor improvement to match.pd was installed. Basically we were violating the ODR via inline functions in *-match-head.c. In an unoptimized compile, those routines would not be inlined and had implicit external linkage w

Re: [PATCH] Make mergeable read-only sections per-function, if requested.

On Thu, May 07, 2015 at 10:15:08AM +0200, Marek Polacek wrote: > > +const char * > > +function_mergeable_rodata_prefix (void) > > This function needs a comment. And much more importantly, I forgot to make it static. Fixed. > > +{ > > + section *s = targetm.asm_out.function_rodata_section > >

Re: C PATCH to add -Woverride-init-side-effects (PR c/64918)

Ping. (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64918#c6 is not addressed in this patch, it's something to be looked at before closing the PR.) On Fri, Apr 24, 2015 at 12:36:25AM +0200, Marek Polacek wrote: > As discussed in the PR, the "initialized field with side-effects overwritten" > warn

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

On Thu, May 07, 2015 at 01:26:57PM +0200, Rainer Orth wrote: > As reported in the PR, with the addition of all those OpenACC tests, > libgomp make check times have skyrocketed since the testsuite is still > run sequentially. > > Even on a reasonably fast x86 machine (4 x 2.0 Ghz Xeon E7450) the ru

Remove mode argument from gen_rtx_SET

One problem with the automatically-generated gen_rtx_FOO () macros is that they always have a mode parameter, even for codes like SET where the mode should always be VOIDmode. This inevitably leads to cases where a caller accidentally passes something other than VOIDmode. E.g. when expanding an SI

Re: [PATCH] Fix eipa_sra AAPCS issue (PR target/65956)

On Thu, May 07, 2015 at 12:16:32PM +0100, Alan Lawrence wrote: > So for my two cents, or perhaps three: > > (1) It'd be great to have something in the documentation for > TARGET_FUNCTION_ARG that explains what the contract for the type information > provided is. Even/especially if some of this is

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

As reported in the PR, with the addition of all those OpenACC tests, libgomp make check times have skyrocketed since the testsuite is still run sequentially. Even on a reasonably fast x86 machine (4 x 2.0 Ghz Xeon E7450) the run takes 4286 seconds. On slower sparc boxes (1.2 GHz UltraSPARC-T2) we

Re: [PATCH] Fix eipa_sra AAPCS issue (PR target/65956)

Richard Biener wrote: On May 5, 2015 4:33:58 PM GMT+02:00, Richard Earnshaw wrote: On 05/05/15 15:33, Richard Earnshaw wrote: On 05/05/15 15:29, Jakub Jelinek wrote: On Tue, May 05, 2015 at 02:20:43PM +0100, Richard Earnshaw wrote: On 05/05/15 14:06, Jakub Jelinek wrote: For the middle-end

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

[ CC'ing Sandra: since you were recently cleaning up the attributes docs (thanks), the doc/extend.texi bits in this patch might be of interest to you. ] On Wed, May 06, 2015 at 08:44:10PM +0200, Marek Polacek wrote: > 2015-05-06 Marek Polacek > > PR c/47043 > * c-common.c (hand

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

Hi Segher, Thank you I committed as r222874. Ref: https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=222874 Regards, Venkat. -Original Message- From: Segher Boessenkool [mailto:seg...@kernel.crashing.org] Sent: Tuesday, May 05, 2015 10:46 PM To: Kumar, Venkataramanan Cc: Jeff Law

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

On 05/07/2015 10:49 AM, Ramana Radhakrishnan wrote: > > > On 06/05/15 15:20, Christian Bruel wrote: >> In preparation of the pragma target >> >> reorganize ÂTARGET_CPU_CPP_BUILTINSÂ to redefine mode dependent macros >> based on current thumb_p. > > I'm not entirely happy with this patch as it

Fix ubsan tests on ARM (PR testsuite/66046)

Trailing \[^\n\r]*(\n|\r\n|\r) in the last dg-output aroused dismay on ARM. Here's a patch to clean this up; can you please test this on ARM? make check-g{cc,++} RUNTESTFLAGS='--target_board=unix\{-m32,-m64\} ubsan.exp' passes on x86_64-linux. 2015-05-07 Marek Polacek PR testsuite/66

Re: [PATCH] Fix OpenACC shutdown and PTX image unloading (PR65904)

On Wed, 6 May 2015 10:32:56 +0200 Thomas Schwinge wrote: > Hi! > > On Fri, 1 May 2015 10:47:19 +0100, Julian Brown > wrote: > > The patch also fixes a thinko that was revealed in image unloading > > in the NVPTX backend. Tested for libgomp with PTX offloading. > > Confirming that both nvptx (P

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

Hello, Le 05/05/2015 11:00, Andre Vehreschild a écrit : > diff --git a/gcc/fortran/trans-decl.c b/gcc/fortran/trans-decl.c > index 4c18920..0b63175 100644 > --- a/gcc/fortran/trans-decl.c > +++ b/gcc/fortran/trans-decl.c > @@ -2158,6 +2158,8 @@ build_function_decl (gfc_symbol * sym, bool global) >

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

Hi, On Wed, 6 May 2015 15:55:57, Joseph S. Myers wrote: > > On Wed, 6 May 2015, Michael Haubenwallner wrote: > >> Am 2015-05-05 um 18:03 schrieb Michael Haubenwallner: >>> Now that gcc-5 is out, what about an automake-1.11.6 update for gcc-6? >> >> Patch for top-level files. > > I don't think this

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

Hi all, my work on pr60322 caused a regression on trunk. This patch fixes it. The regression had two causes: 1. Not taking the correct attribute for BT_CLASS objects with allocatable components into account (chunk 1), and 2. taking the address of an address (chunk 2). When a class or derived ty

Re: [PATCH] Fix(?) PR66002

On Mon, 4 May 2015, Richard Biener wrote: > > This fixes a missed vectorization of a function in paq8p. Without > merged PHI nodes phiopt doesn't recognize adjacent MIN/MAX_EXPRs. > Certainly no other pass I schedule mergephi over cares for merged > PHIs (DCE might even be confused here). > > B

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

On 05/06/2015 03:01 PM, Bernd Edlinger wrote: > Hi, > > On Tue, 5 May 2015 18:03:15, Michael Haubenwallner wrote: >> >> Now that gcc-5 is out, what about an automake-1.11.6 update for gcc-6? >> >> BTW, the actual commands I use to re-run automake for everything (I found) >> is: >> $ export AUT

Re: [nvptx] Re: Mostly rewrite genrecog

On Thu, May 07, 2015 at 10:59:01AM +0200, Thomas Schwinge wrote: > build/genrecog [...]/source-gcc/gcc/common.md > [...]/source-gcc/gcc/config/nvptx/nvptx.md \ >insn-conditions.md > tmp-recog.c > -[...]/source-gcc/gcc/config/nvptx/nvptx.md:1206: warning: operand 0 > missi

[nvptx] Re: Mostly rewrite genrecog

Hi! On Mon, 27 Apr 2015 11:20:30 +0100, Richard Sandiford wrote: > I think it's been the case for a while that parallel builds of GCC tend > to serialise around the compilation of insn-recog.c, especially with > higher --enable-checking settings. This patch tries to speed that > up by replacing

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

On 06/05/15 15:22, Christian Bruel wrote: Re-implement ARM_DECLARE_FUNCTION_NAME as a function. That will make changed related to unified/divided and mode directives easier to insert. Patch could be smaller as below. Thanks Christian 2014-09-23 Christian Bruel * config/arm/ar

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

On 06/05/15 15:20, Christian Bruel wrote: In preparation of the pragma target reorganize ÂTARGET_CPU_CPP_BUILTINSÂ to redefine mode dependent macros based on current thumb_p. I'm not entirely happy with this patch as it appears to be too tied to just the "thumbness" of the attributes. Addit

Re: [Cec-weeklies] ST40 tools #1517,#1518,#1519

please ignore On 05/07/2015 10:43 AM, Christian Bruel wrote: > > Highlights > --- > - Beta Breakpad delived > - Upstreaming (almost accepted) > > Issues > --- > - Current FSF trunk boostrap broken (delay validations) > > Completed > -- > - Fixed FSF attribute optimize in

[Cec-weeklies] ST40 tools #1517,#1518,#1519

Highlights --- - Beta Breakpad delived - Upstreaming (almost accepted) Issues --- - Current FSF trunk boostrap broken (delay validations) Completed -- - Fixed FSF attribute optimize inlining - Fixed FSF bad handling of ipa-cp (inlining of functions with constant propagat

  1   2   >