Re: PR65217.c, improve canonicalization of implied copy from equality comparison

On Mon, Apr 27, 2015 at 10:07:46PM -0600, Jeff Law wrote: Thanks for the nice summary in the first part of the mail. Two typos: > + /* Most of the time tree_swap_operands_p does what we want. But there's Shouldn't that be "there are"? > + cases where we we know one operand is better for

[PATCH] Fix PR65851

This fixes PR65851 and a few missed optimizations in CCP that can cause similar issues. This also fixes set_lattice_value to pass new_val by reference so that in case we drop to VARYING the caller can see that. It also uses operand_equal_p instead of simple_cst_equal because the latter doesn't ev

Re: Improve LTO type checking during symtab merging

On Tue, 28 Apr 2015, Jan Hubicka wrote: > Hi, > actually I bootstrapped/regtested without fortran (as I frogot the setting > from > LTO bootstrap). There are several new warnings in the fortran's testsuite. > As far as I can tell, they represent real mismatches. I wonder, do we want > to fix th

Re: Bare bones of type verifier

On Tue, 28 Apr 2015, Jan Hubicka wrote: > Hi, > this patch adds bare bones of type checker. You can call verify_type on any > type in the IL and see compiler bomb if some of invariants are broken. So far > it only verify tests I already tested in last stage1 with my reverted variant > streaming

RE: [PATCH, Aarch64] Add FMA steering pass for Cortex-A57

> From: Marcus Shawcroft [mailto:marcus.shawcr...@gmail.com] > Sent: Thursday, February 05, 2015 5:17 PM > > > > *** gcc/ChangeLog *** > > > > 2015-01-26 Thomas Preud'homme thomas.preudho...@arm.com > > > > * config.gcc: Add cortex-a57-fma-steering.o to extra_objs for > > aarch64-*-*. > >

[PATCH] Fix PR62283

The following fixes a missed optimization in basic-block vectorization. Currently we require the SLP chain to end up in a sequence of loads we support. But of course we can in theory end the SLP chain at any point and simply construct the vector operand of the uses by pieces. This is what the pa

[gomp4] Merge trunk r222446 (2015-04-26) into gomp-4_0-branch

Hi! Committed to gomp-4_0-branch in r222513: commit be14a100eee0a16881bf59c4dc1b869e483fd6cb Merge: 9973ee1 8047deb Author: tschwinge Date: Tue Apr 28 08:22:41 2015 + svn merge -r 221877:222446 svn+ssh://gcc.gnu.org/svn/gcc/trunk git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc

Re: [PATCH, ARM] Alternatives and type attributes fixes.

Hi, On 27 April 2015 at 15:58, Yvan Roux wrote: > On 27 April 2015 at 15:20, Kyrill Tkachov wrote: >> Hi Yvan, >> >> >> On 27/04/15 13:25, Yvan Roux wrote: >>> >>> Hi, >>> >>> This is a follow-up of PR64208 where an LRA loop was due to redundancy >>> in insn's alternatives. I've checked all the

Re: [PATCH] Fix PR65549, avoid force_decl_die in late compilation

On Fri, 17 Apr 2015, Jakub Jelinek wrote: > On Fri, Apr 17, 2015 at 12:32:03PM +0200, Richard Biener wrote: > > So Jakub says that using comp_unit_die () for the context of the stub > > DIE is wrong and he is of course right. The following adjusted patch > > uses the correct context, but only if

Change -fdump-ada-spec output for static member functions

Back in February I changed the way the Ada compiler recognizes imported C++ methods on Windows: https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00478.html Unfortunately this change uncovered a deeper issue with these C++ methods so we have decided to change the -fdump-ada-spec output accordingly.

[committed] Fix example in header comment in tree-call-cdce.c

Hi, Consider log.c: ... #include void f (double d) { log (d); } ... When compiling with -O2, at optimized dump the code looks like this, thanks to pass_call_cdce: ... f (double d) { : if (d_2(D) <= 0.0) goto ; else goto ; : log (d_2(D)); [tail call] : return; } ..

Re: Bare bones of type verifier

Hi, I will fix the free count issue - my internet access in China was bit sporadic and I hoped you will know why the count seem to disagree between PPC64 and Linux builds. In meantime PPC64 stopped to build for me. > > (early) dwarf2out is a good place, likewise verifying right after > free-lang-

Re: [PATCH][PR65823] Fix va_arg ap_copy nop detection

On 27-04-15 09:45, Tom de Vries wrote: On 22-04-15 15:50, Richard Biener wrote: On Wed, Apr 22, 2015 at 3:38 PM, Tom de Vries wrote: On 22-04-15 10:06, Richard Biener wrote: On Wed, Apr 22, 2015 at 9:41 AM, Tom de Vries wrote: Hi, this patch fixes PR65823. The patches fixes the pr

Re: [PATCH, FRV, libgcc]: Move CRT_GET_RFIB_DATA to libgcc

On Wed, Apr 22, 2015 at 1:11 PM, Uros Bizjak wrote: > Hello! > > Attached patch moves CRT_GET_RFIB_DATA from gcc to libgcc in the same > way as was recently done for x86 targets [1]. > > The second patch is needed to successfully compile frv-elf libgcc, > otherwise libgcc build fails in defaults.h

Re: [PATCH][PR65823] Fix va_arg ap_copy nop detection

On Tue, Apr 28, 2015 at 11:10 AM, Tom de Vries wrote: > On 27-04-15 09:45, Tom de Vries wrote: >> >> On 22-04-15 15:50, Richard Biener wrote: >>> >>> On Wed, Apr 22, 2015 at 3:38 PM, Tom de Vries >>> wrote: On 22-04-15 10:06, Richard Biener wrote: > > > On Wed, Apr 22, 2015

RE: [RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value to output_constant_pool_2

Ping. -Original Message- From: Dharmakan Rohit-B30502 Sent: Friday, March 27, 2015 7:57 PM To: gcc-patches@gcc.gnu.org; rguent...@suse.de; Jakub Jelinek Cc: Alan Modra; David Edelsohn; Wienskoski Edmar-RA8797; Dharmakan Rohit-B30502 Subject: RE: [RFC: Patch, PR 60158] gcc/varasm.c : Pass

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

On 27/04/15 21:13, Jeff Law wrote: On 04/21/2015 11:33 AM, Kyrill Tkachov wrote: On 21/04/15 15:09, Jeff Law wrote: On 04/21/2015 02:30 AM, Kyrill Tkachov wrote: From reading config/stormy16/stormy-abi it seems to me that we don't pass arguments partially in stormy16, so this code would ne

Re: C PATCH for c/65345 (file-scope _Atomic expansion, this time with floats)

Ping. On Tue, Apr 14, 2015 at 06:08:45PM +0200, Marek Polacek wrote: > You were right: my earlier fix for c/65345 only handled non-float types. > This patch thus handles even the float types. The fix is analogical to the > previous: use create_tmp_var_raw and TARGET_EXPR to avoid ICE. > > This o

Re: [patch, c, ping] Fix PR c/48956: diagnostics for conversions involving complex types (reviewed)

On 20.04.2015 21:35, Jeff Law wrote: > On 04/17/2015 11:37 AM, Mikhail Maltsev wrote: >> On 04/17/2015 08:10 PM, Jeff Law wrote: >>> Have you received confirmation from the FSF WRT your copyright >>> assignment was accepted? >>> >>> jeff >>> >> Yes, it's ID is [gnu.org #972407]. Should I forward th

[PATCH] Fix PR65911

During the removal of "#if ARGS_GROW_DOWNWARD" in r222508, Trevor forgot to add parentheses around a ternary operator. Tested on powerpc64-unknown-linux-gnu. Preapproved by Jakub and Richard on bugzilla. Commited to trunk. 2015-04-28 Markus Trippelsdorf PR other/65911 * functi

Re: [PATCH][PR65818][bootstrap,hppa] Return side-effect free result in gimplify_va_arg_internal

On Mon, Apr 27, 2015 at 5:04 PM, Tom de Vries wrote: > On 27-04-15 15:40, Richard Biener wrote: >> >> On Mon, Apr 27, 2015 at 3:06 PM, Tom de Vries >> wrote: >>> >>> On 27-04-15 10:17, Richard Biener wrote: > > > This patch fixes that by gimplifying the address expression of the >

Re: [Patch, Fortran] Prevent segfault with dump-*-original for implicit class expressions.

Hi Mikael, hi all, Mikael, thanks for the comments. I have worked in your suggestions and the patch now looks like the attached one. To get the testsuite accepting a dump-fortran-original, I had to insert a dg-prune-output. Without the prune output, dejagnu was reporting excess errors. The prune

c-family PATCH to improve -Wbool-compare (PR c/64610)

This improves -Wbool-compare a bit: boolean >= 0 is always true, on the other hand boolean < 0 is always false, and similar. One feature/bug is that in C++ we don't warn for e.g. bool >= false, since the warning cares about CMP or CMP . I can fix that in a follow-up if anyone wants me to. Boo

[C++ RFC/Patch] PR 39813

Hi, I'm having another look at this old bug, and I think it should be easy to fix, but there are a few details which I do not understand. Essentially the issue is about: struct B { template void fn() { cout << __PRETTY_FUNCTION__ << endl; } }; int main() { B().fn();

Re: [Patch, Fortran] Prevent segfault with dump-*-original for implicit class expressions.

Hello, Le 28/04/2015 12:43, Andre Vehreschild a écrit : > To get the testsuite accepting a dump-fortran-original, I had to insert a > dg-prune-output. Without the prune output, dejagnu was reporting excess > errors. > The prune removes all output after the first Namespace: that starts the > fortr

Re: C++ delayed folding branch review

2015-04-24 20:25 GMT+02:00 Jason Merrill : > On 04/24/2015 09:46 AM, Kai Tietz wrote: >> >> Sure, we can use here instead *_fully_fold, but for what costs? In >> general we need to deal here a simple one-level fold for simplifying >> constant-values, and/or removing useless type-conversions. > > >

Re: [PATCH] PR 62173, re-shuffle insns for RTL loop invariant hoisting

Hi Matthew, 2015-04-21 15:24 GMT+01:00 Jiong Wang : > > 2015-04-21 Jiong Wang > > gcc/ > * loop-invariant.c (find_defs): Enable DF_DU_CHAIN build. > (vfp_const_iv): New hash table. > (expensive_addr_check_p): New boolean. > (init_inv_motion_data): Initialize new variables.> > (free_i

[patch] Add std::void_t

Define std::void_t for -std=c++17 and -std=gnu++11 (like Ville's uncaught_exceptions() patch I think this is useful to define as an extension pre-C++17, and the feature-test macro makes it easy for users to test for the presence/absence of it in different modes). Tested ppc64le-linux, committed t

Re: [libstdc++ PATCH] Add support for std::uncaught_exceptions

On 27/04/15 21:53 +0100, Jonathan Wakely wrote: On 27/04/15 21:40 +0100, Jonathan Wakely wrote: Tested x86_64-linux and powerpc64le-linux. Committed to trunk. The baseline_symbols changes aren't needed now and I tweaked the gnu.ver file slightly. Ville noticed a typo in a comment I added, fix

Re: [PATCH] [libstdc++/65839] whatis support for xmethods

On 27/04/15 15:44 -0700, Doug Evans wrote: PR libstdc++/65839 * python/libstdcxx/v6/xmethods.py (get_bool_type): New function. Replace all lookups of "bool" with this. (get_std_size_type): New function. Replace all lookups of std::size_t with this.

[patch] libstdc++/65631 make std::seed_seq noncopyable

Tested ppc64le-linux, committed to trunk. commit 8009da43b7be1a10335240b7b2f4e27250d5376f Author: Jonathan Wakely Date: Mon Mar 30 19:22:52 2015 +0100 PR libstdc++/65631 * include/bits/random.h (seed_seq) Define copy constructor and copy assignment as deleted. * testsuite/

Re: [PATCH] missing return in debug/vector _Safe_vector::operator=

[CCing gcc-patches] On 28/04/15 14:32 +0200, Stephan Bergmann wrote: broken on GCC 5 and trunk Ouch. OK for trunk and the branch, do you have commit privs? Index: libstdc++-v3/ChangeLog === --- libstdc++-v3/ChangeLog (rev

Re: [PATCH] missing return in debug/vector _Safe_vector::operator=

On 04/28/2015 02:38 PM, Jonathan Wakely wrote: [CCing gcc-patches] On 28/04/15 14:32 +0200, Stephan Bergmann wrote: broken on GCC 5 and trunk Ouch. OK for trunk and the branch, do you have commit privs? No, I don't; would be great if you can push that.

Re: [C++ RFC/Patch] PR 39813

On 04/28/2015 06:59 AM, Paolo Carlini wrote: && !DECL_FRIEND_PSEUDO_TEMPLATE_INSTANTIATION (t) the name of the macro seems weird to me: it mentions friends (and friends are mentioned in its comment too) but isn't used only for DECL_FRIEND_P true and indeed it is true for the non-friend 'fn

PR libstdc++/65883 missing unsigned in numeric_limits

Hello, this patch fixes an obvious typo. Regtested without problem, and I manually checked that the new return value of max() makes more sense. 2015-04-28 Marc Glisse PR libstdc++/65883 * include/std/limits (numeric_limits): Add missing unsigned. -- Marc GlisseIndex: inclu

Re: [PATCH] missing return in debug/vector _Safe_vector::operator=

On 28/04/15 14:43 +0200, Stephan Bergmann wrote: On 04/28/2015 02:38 PM, Jonathan Wakely wrote: [CCing gcc-patches] On 28/04/15 14:32 +0200, Stephan Bergmann wrote: broken on GCC 5 and trunk Ouch. OK for trunk and the branch, do you have commit privs? No, I don't; would be great if you ca

Re: [C++ RFC/Patch] PR 39813

Hi, On 04/28/2015 02:45 PM, Jason Merrill wrote: On 04/28/2015 06:59 AM, Paolo Carlini wrote: && !DECL_FRIEND_PSEUDO_TEMPLATE_INSTANTIATION (t) the name of the macro seems weird to me: it mentions friends (and friends are mentioned in its comment too) but isn't used only for DECL_FRIEND_

Re: [C++ RFC/Patch] PR 39813

On 04/28/2015 08:54 AM, Paolo Carlini wrote: Hi, On 04/28/2015 02:45 PM, Jason Merrill wrote: On 04/28/2015 06:59 AM, Paolo Carlini wrote: && !DECL_FRIEND_PSEUDO_TEMPLATE_INSTANTIATION (t) the name of the macro seems weird to me: it mentions friends (and friends are mentioned in its com

[patch] libstdc++/61645 add noexcept to forward_list::splice_after

This isn't required, but we might as well add it for consistency with std::list:splice and for extra noexceptiness. Tested ppc64le-linux, committed to trunk. commit 24308d6f00c2235fcc87ce0ab05e219696714699 Author: Jonathan Wakely Date: Wed Apr 8 17:35:13 2015 +0100 PR libstdc++/61645

Re: PR libstdc++/65883 missing unsigned in numeric_limits

On 28/04/15 14:47 +0200, Marc Glisse wrote: Hello, this patch fixes an obvious typo. Regtested without problem, and I manually checked that the new return value of max() makes more sense. OK, thanks.

Re: [PATCH 6/n] OpenMP 4.0 offloading infrastructure: option handling

On 04/27/2015 06:08 PM, Thomas Schwinge wrote: OK to do the following instead? (Coding style/code copied from gcc/config/i386/intelmic-mkoffload.c for consistency.) Err, was this a question for me? I'm fine with that too. Bernd

[patch] libstdc++/60333 fix make_signed and make_unsigned for long long enumerations

We were assuming that no enumeration would ever be bigger than long and were also failing to preserve cv qualifiers on enums. I fixed make_unsigned and then changed make_signed to reuse that instead of duplicating all the logic. Tested x86_64-linux (with -m32) and ppc64le-linux. Committed to tr

[PATCH] Support -rdynamic on darwin12 and later

The attached patch adds support for the -rdynamic compiler flag on darwin12 and later. The darwin linker, starting with Xcode 5 on darwin12, added support for the associated new -export_dynamic linker flag. -export_dynamic Preserves all global symbols in main executables

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

Mikhail Maltsev writes: > I'm sending an updated patch (rebased to recent trunk, bootstrapped and > regtested on x86_64-unknown-linux-gnu). > > On 04/25/2015 02:49 PM, Richard Sandiford wrote: >> FWIW I think the split between label_rtx and live_label_rtx is good, >> but I think we should give the

Use wi:: in canon_condition

Fix a simple case where we can generalise the CONST_INT handling to all integer constants (CONST_DOUBLE or CONST_WIDE_INT). Tested on x86_64-linux-gnu. OK to install? Thanks, Richard gcc/ * loop-iv.c (canon_condition): Generalize to all types of integer constant. Index: gcc/lo

Re: C++ delayed folding branch review

On 04/28/2015 08:06 AM, Kai Tietz wrote: 2015-04-24 20:25 GMT+02:00 Jason Merrill : So for warning folding, I think the caching approach we were talking about in the call today is the way to go. Ok. Just a question about where to hook up the hash-variable. What would be a good place to creat

[PATCH] Remove some restrictions on loop shape in tree-if-conv.c

Tree if-conversion currently bails out for loops that (a) contain nested loops; (b) have more than one exit; (c) where the exit block (source of the exit edge) does not dominate the loop latch; (d) where the exit block is the loop header, or there are statements after the exit. This patch remo

RE: [PATCH] PR 62173, re-shuffle insns for RTL loop invariant hoisting

> Hi Matthew, > > 2015-04-21 15:24 GMT+01:00 Jiong Wang : > > > > > 2015-04-21 Jiong Wang > > > > gcc/ > > * loop-invariant.c (find_defs): Enable DF_DU_CHAIN build. > > (vfp_const_iv): New hash table. > > (expensive_addr_check_p): New boolean. > > (init_inv_motion_data): Initialize new

Re: [PATCH] Remove some restrictions on loop shape in tree-if-conv.c

Alan Lawrence wrote: Tree if-conversion currently bails out for loops that (a) contain nested loops; (b) have more than one exit; (c) where the exit block (source of the exit edge) does not dominate the loop latch; (d) where the exit block is the loop header, or there are statements after the e

Re: [PATCH] PR 62173, re-shuffle insns for RTL loop invariant hoisting

2015-04-28 14:56 GMT+01:00 Matthew Fortune : >> Hi Matthew, >> >> 2015-04-21 15:24 GMT+01:00 Jiong Wang : >> >> > >> > 2015-04-21 Jiong Wang >> > >> > gcc/ >> > * loop-invariant.c (find_defs): Enable DF_DU_CHAIN build. >> > (vfp_const_iv): New hash table. >> > (expensive_addr_check_p): New

Re: Improve LTO type checking during symtab merging

> > Only emitting the warnings with -Wodr looks good to me. I can't see > how we can decide what cases lead to wrong code surprises and what not OK, I will go with -Wodr for all the warnings then, that seems fine to me. > (other than using types_compatible_p ...). Wrong-code can only(?) happen

Re: [PATCH, ARM] Alternatives and type attributes fixes.

On 28/04/15 09:32, Yvan Roux wrote: Hi, On 27 April 2015 at 15:58, Yvan Roux wrote: On 27 April 2015 at 15:20, Kyrill Tkachov wrote: Hi Yvan, On 27/04/15 13:25, Yvan Roux wrote: Hi, This is a follow-up of PR64208 where an LRA loop was due to redundancy in insn's alternatives. I've chec

Re: C++ PATCH for c++/50800 (ICE with may_alias and templates)

On 04/23/2015 01:45 PM, H.J. Lu wrote: On Thu, Apr 23, 2015 at 8:52 AM, Jason Merrill wrote: We try to strip attributes that aren't reflected in mangling from template arguments, but were failing to do that in this case. Fixed by making strip_typedefs strip such attributes as well. Tested x86

C++ PATCH for c++/65656 (constexpr and __builtin_constant_p)

We have two desires for interaction of __builtin_constant_p with constexpr: 1) it should be a constant-expression even if its operands are not, and 2) we shouldn't fold it to false prematurely when parsing a constexpr function (c++/54021). We were having trouble with both of these, and this p

Re: [PATCH] Improve PR44563

Hi, the g++.dg/tree-ssa/pr61034.C has turned out to be sensitive to PUSH_ARGS settings so fixing it on PPC64 caused x86_64 to regress. This patch introduce temporary so the code is consistent between PPC64 and x86_64. Comitted after ICR dicussion with Richard. Honza Index: ChangeLog

[PATCH,PING][ARM]Remove vec_shr and vec_shr optabs

No new code here ;). There is a slight change of execution path, i.e. some VEC_PERM_EXPRs (e.g. those for reductions via shifts) will be expanded using arm_expand_vec_perm_const rather than the vec_shr pattern. This generates EXT instructions equivalent to the original, but using the mode of the s

[PING 2][PATCH] libgcc: Add CFI directives to the soft floating point support code for ARM

This patch adds CFI directives to the soft floating point support code for ARM. Previously, if we tried to do a backtrace from that code in a debug session we'd get something like this: (gdb) bt #0 __nedf2 () at ../../../../../../gcc-4.9.2/libgcc/config/arm/ieee754-df.S:1082 #1 0x0db6 in _

[PATCH][AARCH64]Use shl for vec_shr_ rtx pattern.

Hi all, unsigned shift left dosen't support immediate shift. Previouly, gcc will generate asm instruction like this: "ushl d1, d0, 32", which is not a legal insn and will be rejected by assembler. This patch change the use of ushl in vec_shr_ into shl. A test case is added, and it passes on

[PING][PATCH][AARCH64]Add ACLE 2.0 predefined macros: __ARM_ALIGN_MAX_PWR and __ARM_ALIGN_MAX_STACK_PWR

Ping~ Regards, Renlin Li On 16/12/14 10:19, Renlin Li wrote: Hi all, This is a simple patch to add another two ACLE 2.0 predefined macros into aarch64 backend. They are __ARM_ALIGN_MAX_PWR and __ARM_ALIGN_MAX_STACK_PWR. Currently, those two values are hard-wired to 16. The following clauses f

Re: Bare bones of type verifier

On April 28, 2015 11:05:44 AM GMT+02:00, Jan Hubicka wrote: >Hi, >I will fix the free count issue - my internet access in China was bit >sporadic >and I hoped you will know why the count seem to disagree between PPC64 >and Linux >builds. In meantime PPC64 stopped to build for me. >> >> (early) dw

Re: Re: [PING][PATCH][4.9]Backport "Fix register corruption bug in ree"

Ping~ Regards, Renlin Li On 16/04/15 10:32, Renlin Li wrote: Ping~ Regards, Renlin Li On 16/04/15 10:09, wrote: Ping~ Anybody has time to review it? Regards, Renlin Li On 06/02/15 17:48, Renlin Li wrote: Hi all, This is a backport patch for branch 4.9. You can find the original patch he

[PATCH, PR65915] Fix float conversion split.

> > I've renamed EXT_SSE_REG_P into EXT_REX_SSE_REG_P for consistency. > > Ok for stage1? > Patch is OK for stage1. > > -- > Thanks, K > > > > On 19 Mar 12:24, Ilya Tocar wrote: > > > Hi, > > > > > > There were some discussion about "x" constraints being too conservative > > > for some patterns

Re: [PING 2][PATCH] libgcc: Add CFI directives to the soft floating point support code for ARM

On Tue, Apr 28, 2015 at 4:19 PM, Martin Galvan wrote: > This patch adds CFI directives to the soft floating point support code for > ARM. > > Previously, if we tried to do a backtrace from that code in a debug session > we'd > get something like this: > > (gdb) bt > #0 __nedf2 () at > ../../..

Re: [PATCH,PING][ARM]Remove vec_shr and vec_shr optabs

On Tue, Apr 28, 2015 at 4:15 PM, Alan Lawrence wrote: >> No new code here ;). There is a slight change of execution path, i.e. some >> VEC_PERM_EXPRs (e.g. those for reductions via shifts) will be expanded >> using >> arm_expand_vec_perm_const rather than the vec_shr pattern. This generates >> EXT

Re: [PATCH, PR65915] Fix float conversion split.

On Tue, Apr 28, 2015 at 9:20 AM, Ilya Tocar wrote: >> > I've renamed EXT_SSE_REG_P into EXT_REX_SSE_REG_P for consistency. >> > Ok for stage1? >> Patch is OK for stage1. >> >> -- >> Thanks, K >> >> >> > On 19 Mar 12:24, Ilya Tocar wrote: >> > > Hi, >> > > >> > > There were some discussion about "x

Re: [PATCH][AARCH64]Add ACLE 2.0 predefined macros: __ARM_ALIGN_MAX_PWR and __ARM_ALIGN_MAX_STACK_PWR

On 16 December 2014 at 10:19, Renlin Li wrote: > 2014-12-16 Renlin Li > > * config/aarch64/aarch64.h(TARGET_CPU_CPP_BUILTINS): Define > __ARM_ALIGN_MAX_PWR > and __ARM_ALIGN_MAX_STACK_PWR. OK /Marcus

gcc/c-family/c-pragma.h:pragma_omp_clause (was: Pragma parsing)

Hi Jakub! We now have to revisit this discussion about gcc/c-family/c-pragma.h:pragma_omp_clause, : On Thu, 18 Dec 2014 14:48:32 +0100, I wrote: > > > -/* All clauses defined by OpenMP 2.5, 3.0, 3.1 and 4.0. > >

Re: Use wi:: in canon_condition

On 04/28/2015 07:49 AM, Richard Sandiford wrote: Fix a simple case where we can generalise the CONST_INT handling to all integer constants (CONST_DOUBLE or CONST_WIDE_INT). Tested on x86_64-linux-gnu. OK to install? Thanks, Richard gcc/ * loop-iv.c (canon_condition): Generalize to al

Re: gcc make clean fixes

On 04/27/2015 05:03 PM, Jim Wilson wrote: On Mon, Apr 27, 2015 at 12:30 PM, Jeff Law wrote: Looks good to me. Please install if you haven't already done so. Thanks, I checked in the patch. I'm not maintainer of anything currently, so I'm assuming all of my patches need to be approved before

Re: gcc make clean fixes

On Apr 27, 2015, at 4:03 PM, Jim Wilson wrote: > On Mon, Apr 27, 2015 at 12:30 PM, Jeff Law wrote: >> Looks good to me. Please install if you haven't already done so. > > Thanks, I checked in the patch. I'm not maintainer of anything > currently, Actually, you are still listed: ia64 port

Re: [PING 2][PATCH] libgcc: Add CFI directives to the soft floating point support code for ARM

Thanks a lot. I don't have write access to the repository, could you commit this for me? On Tue, Apr 28, 2015 at 1:21 PM, Ramana Radhakrishnan wrote: > On Tue, Apr 28, 2015 at 4:19 PM, Martin Galvan > wrote: >> This patch adds CFI directives to the soft floating point support code for >> ARM. >

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

On 04/28/2015 07:46 AM, Richard Sandiford wrote: I still think we should rename label_rtx too, because I think it's confusing for label_rtx to return something other than a label. That's probably a separate, follow-up patch though. Seems fine as a follow-up. I added the member functions. The

Re: [PATCH] Fix size & type for cold partition names (hot-cold function partitioning)

Caroline, Your patch has broken bootstrap on AIX and probably other platforms. /nasfarm/edelsohn/src/src/gcc/varasm.c: In function 'void assemble_end_function(tree, const char*)': /nasfarm/edelsohn/src/src/gcc/varasm.c:1870:12: error: 'ASM_DECLARE_FUNCTION_SIZE' was not declared in this scope

Re: [PATCH] Fix size & type for cold partition names (hot-cold function partitioning)

Yes, this is already mentioned in PR 65910 (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65910). There is a patch mentioned in that PR that both appears to fix the problem and appears to have been approved. I thought the author of the patch would commit it, but that does not appear to have happen

Re: [PATCH] Fix size & type for cold partition names (hot-cold function partitioning)

On 04/28/2015 11:12 AM, Caroline Tice wrote: Yes, this is already mentioned in PR 65910 (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65910). There is a patch mentioned in that PR that both appears to fix the problem and appears to have been approved. I thought the author of the patch would com

Re: [PATCH] Fix size & type for cold partition names (hot-cold function partitioning)

On Tue, 28 Apr 2015, Caroline Tice wrote: > Yes, this is already mentioned in PR 65910 > (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65910). > > There is a patch mentioned in that PR that both appears to fix the > problem and appears to have been approved. I thought the author of > the patch wo

Re: c-family PATCH to improve -Wbool-compare (PR c/64610)

On 04/28/2015 04:49 AM, Marek Polacek wrote: This improves -Wbool-compare a bit: boolean >= 0 is always true, on the other hand boolean < 0 is always false, and similar. One feature/bug is that in C++ we don't warn for e.g. bool >= false, since the warning cares about CMP or CMP . I can fix

Re: gcc/c-family/c-pragma.h:pragma_omp_clause (was: Pragma parsing)

On Tue, Apr 28, 2015 at 06:59:12PM +0200, Thomas Schwinge wrote: > We're getting ready to submit patches extending the C/C++ front ends for > the remaining OpenACC clauses, and given the current layout of > gcc/c-family/c-pragma.h:pragma_omp_clause, we're then at 69 unique > clauses, which is more

Re: [PATCH] Fix size & type for cold partition names (hot-cold function partitioning)

I just committed the patch for Dominique. - David On Tue, Apr 28, 2015 at 1:12 PM, Caroline Tice wrote: > Yes, this is already mentioned in PR 65910 > (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65910). > > There is a patch mentioned in that PR that both appears to fix the > problem and appea

Re: [PATCH] Fix size & type for cold partition names (hot-cold function partitioning)

Thank you! -- Caroline Tice cmt...@google.com On Tue, Apr 28, 2015 at 10:16 AM, David Edelsohn wrote: > I just committed the patch for Dominique. > > - David > > > On Tue, Apr 28, 2015 at 1:12 PM, Caroline Tice wrote: >> Yes, this is already mentioned in PR 65910 >> (https://gcc.gnu.org/bugzil

[committed] Remove stray line in c_parser_binary_expression

Bootstrapped/regtested on x86_64-linux, applying to trunk. 2015-04-28 Marek Polacek * c-parser.c (c_parser_binary_expression): Remove duplicate line. diff --git gcc/c/c-parser.c gcc/c/c-parser.c index f5e2ac2c..cc8a4e3 100644 --- gcc/c/c-parser.c +++ gcc/c/c-parser.c @@ -6370,11 +6370

[Patch, MIPS] Change mips4 default processor to r10K

This patch changes the default processor for mips4 from the r8000 to the r1. There are several reasons for this change, the main one being the difference in the r8000 madd instruction and the rest of the mips4 family. The r8000 has a fused madd instruction (no truncation between the multiply

Re: [C++ RFC/Patch] PR 39813

Hi, On 04/28/2015 02:59 PM, Jason Merrill wrote: On 04/28/2015 08:54 AM, Paolo Carlini wrote: Hi, On 04/28/2015 02:45 PM, Jason Merrill wrote: On 04/28/2015 06:59 AM, Paolo Carlini wrote: && !DECL_FRIEND_PSEUDO_TEMPLATE_INSTANTIATION (t) the name of the macro seems weird to me: it men

Re: [PATCH] missing return in debug/vector _Safe_vector::operator=

On 28/04/15 13:47 +0100, Jonathan Wakely wrote: On 28/04/15 14:43 +0200, Stephan Bergmann wrote: On 04/28/2015 02:38 PM, Jonathan Wakely wrote: [CCing gcc-patches] On 28/04/15 14:32 +0200, Stephan Bergmann wrote: broken on GCC 5 and trunk Ouch. OK for trunk and the branch, do you have comm

Re: [RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value to output_constant_pool_2

On 04/28/2015 03:44 AM, rohitarul...@freescale.com wrote: Ping. -Original Message- From: Dharmakan Rohit-B30502 Sent: Friday, March 27, 2015 7:57 PM To: gcc-patches@gcc.gnu.org; rguent...@suse.de; Jakub Jelinek Cc: Alan Modra; David Edelsohn; Wienskoski Edmar-RA8797; Dharmakan Rohit-B305

[PATCH, testsuite, committed] Remove spurious comment line from gcc.dg/vect/vect-33.c

Hi, As pointed out by Bin Cheng in https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01575.html, an extra comment line slipped into one of the test cases when I was updating tests for the POWER8 unaligned memory access patch. This comment had the unfortunate side effect of disabling the test for AArc

RE: [Patch, MIPS] Change mips4 default processor to r10K

Steve Ellcey writes: > This patch changes the default processor for mips4 from the r8000 to > the > r1. There are several reasons for this change, the main one > being the difference in the r8000 madd instruction and the rest of the > mips4 > family. The r8000 has a fused madd instruction (

RE: [RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value to output_constant_pool_2

>-Original Message- >From: Jeff Law [mailto:l...@redhat.com] >Sent: Tuesday, April 28, 2015 11:48 PM >To: Dharmakan Rohit-B30502; gcc-patches@gcc.gnu.org; rguent...@suse.de; Jakub >Jelinek >Cc: Alan Modra; David Edelsohn; Wienskoski Edmar-RA8797 >Subject: Re: [RFC: Patch, PR 60158] gcc/

Re: Bare bones of type verifier

On April 28, 2015 6:01:24 PM GMT+02:00, Bernhard Reutner-Fischer wrote: >On April 28, 2015 11:05:44 AM GMT+02:00, Jan Hubicka >wrote: >>Hi, >>I will fix the free count issue - my internet access in China was bit >>sporadic >>and I hoped you will know why the count seem to disagree between PPC64

Fix OpenMP's target update directive in templated code

Hi Jakub! When adding support for C++ templates usage with OpenACC directives (gcc/cp/pt.c:tsubst_expr), we found: On Mon, 27 Apr 2015 10:57:14 -0700, Cesar Philippidis wrote: > On 04/27/2015 10:40 AM, Thomas Schwinge wrote: > > Cesar wrote: > >> case OMP_TARGET_UPDATE: > >>tmp = t

Re: [commited, Patch, Fortran] Prevent segfault with dump-*-original for implicit class expressions.

Hi Mikael, hi all, Mikael, thanks for the review. Commited as r222539 including the improved regexp. Regards, Andre On Tue, 28 Apr 2015 14:05:57 +0200 Mikael Morin wrote: > Hello, > > Le 28/04/2015 12:43, Andre Vehreschild a écrit : > > To get the testsuite accepting a dump-fortran-or

Re: [PATCH] Support -rdynamic on darwin12 and later

On Apr 28, 2015, at 6:39 AM, Jack Howarth wrote: > The attached patch adds support for the -rdynamic compiler flag > on darwin12 and later. > Okay for gcc trunk? Jack was able to confirm that this doesn’t change darwin11. Ok.

Re: [patch] Only configure libstdc++-v3/python dir for hosted builds

On 27/08/14 18:50 +0100, Jonathan Wakely wrote: Currently a freestanding build installs the Python GDB hooks as ${libdir}/libstdc*-gdb.py (with a literal * character in the filename) because there is no libstdc++.so library file and the wildcard doesn't get expanded (see the install-data-local ta

C++ PATCH for c++/65896 (constexpr and empty bases)

In this testcase we were getting confused when trying to initialize an empty base with a CONSTRUCTOR: the initialization target was *(base*)this, which constexpr evaluation resolves to the object argument, which isn't actually of the base type. So we end up storing a base constructor as the va

Re: [PATCH][PR65818][bootstrap,hppa] Return side-effect free result in gimplify_va_arg_internal

On 28-04-15 12:34, Richard Biener wrote: On Mon, Apr 27, 2015 at 5:04 PM, Tom de Vries wrote: On 27-04-15 15:40, Richard Biener wrote: On Mon, Apr 27, 2015 at 3:06 PM, Tom de Vries wrote: On 27-04-15 10:17, Richard Biener wrote: This patch fixes that by gimplifying the address expressio

Re: C PATCH for c/65345 (file-scope _Atomic expansion, this time with floats)

On Tue, 14 Apr 2015, Marek Polacek wrote: > You were right: my earlier fix for c/65345 only handled non-float types. > This patch thus handles even the float types. The fix is analogical to the > previous: use create_tmp_var_raw and TARGET_EXPR to avoid ICE. > > This only fixes x86 though, other

[PATCH/libiberty] fix build of gdb/binutils with clang.

I believe this is the same problem as https://gcc.gnu.org/ml/gcc-patches/2008-07/msg00292.html The asprinf declaration is messed up when using clang to build gdb. diff --git a/include/libiberty.h b/include/libiberty.h index b33dd65..a294903 100644 --- a/include/libiberty.h +++ b/include/libibert

[C++ Patch] PR 59729

Hi, apparently we aren't implementing the bits of DR 1732 [C++14] about range-based for declarations. Tested x86_64-linux. Thanks, Paolo. /// /cp 2015-04-28 Paolo Carlini PR c++/59729, DR 1732 * parser.c (cp_parser_for_init_statement): Do not allow type

Re: [RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value to output_constant_pool_2

On 04/28/2015 12:38 PM, rohitarul...@freescale.com wrote: -Original Message- From: Jeff Law [mailto:l...@redhat.com] Sent: Tuesday, April 28, 2015 11:48 PM To: Dharmakan Rohit-B30502; gcc-patches@gcc.gnu.org; rguent...@suse.de; Jakub Jelinek Cc: Alan Modra; David Edelsohn; Wienskoski

Re: [RFC stage 1] Proposed new warning: -Wmisleading-indentation

On Fri, 17 Apr 2015, Manuel López-Ibáñez wrote: > On 17/04/15 16:42, Tom Tromey wrote: > > > > > > > "Dave" == David Malcolm writes: > > > > Dave> However within libcpp and gcc, in linemap's expanded_location and in > > Dave> diagnostic messages, the "column" numbers are actually 1-based counts

  1   2   >