Re: RFC: Faster for_each_rtx-like iterators

2014-05-08 Thread Richard Sandiford
Trevor Saunders tsaund...@mozilla.com writes: On Wed, May 07, 2014 at 09:52:49PM +0100, Richard Sandiford wrote: I noticed for_each_rtx showing up in profiles and thought I'd have a go at using worklist-based iterators instead. So far I have three: FOR_EACH_SUBRTX: iterates over const_rtx

Re: RFC: Faster for_each_rtx-like iterators

2014-05-08 Thread Richard Sandiford
Richard Sandiford rdsandif...@googlemail.com writes: Trevor Saunders tsaund...@mozilla.com writes: On Wed, May 07, 2014 at 09:52:49PM +0100, Richard Sandiford wrote: I noticed for_each_rtx showing up in profiles and thought I'd have a go at using worklist-based iterators instead. So far I

Re: [PATCH] Windows libiberty: Don't quote args unnecessarily (v2)

2014-05-08 Thread Kai Tietz
2014-05-07 8:55 GMT+02:00 Ray Donnelly mingw.andr...@gmail.com: We only quote arguments that contain spaces, \t or characters to prevent wasting 2 characters per argument of the CreateProcess() 32,768 limit. libiberty/ * pex-win32.c (argv_to_cmdline): Don't quote args

Re: [PATCH] Windows libibery: Don't quote args unnecessarily

2014-05-08 Thread Kai Tietz
Ah, that was my mail-client. Patch is ok IMO. I add Ian CC for a second look at it. Cheers, Kai

Re: [committed] PR 61095: tsan fallout from wide-int merge

2014-05-08 Thread Richard Sandiford
Richard Sandiford rdsandif...@googlemail.com writes: This PR was due to code in which -(int) foo was suposed to be sign-extended, but was being ORed with an unsigned int and so ended up being zero-extended. Fixed by using the proper-width type. As Kostya rightly said in the PR, this should

Re: [committed] PR 61095: tsan fallout from wide-int merge

2014-05-08 Thread Richard Biener
On Thu, May 8, 2014 at 9:48 AM, Richard Sandiford rdsandif...@googlemail.com wrote: Richard Sandiford rdsandif...@googlemail.com writes: This PR was due to code in which -(int) foo was suposed to be sign-extended, but was being ORed with an unsigned int and so ended up being zero-extended.

[PATCH, 1/2] shrink wrap a function with a single loop: copy propagation

2014-05-08 Thread Zhenqiang Chen
Hi, Similar issue was discussed in thread http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01145.html. The patches are close to Jeff's suggestion: sink just the moves out of the incoming argument registers. The patch and following one try to shrink wrap a function with a single loop, which can not

[PATCH, 2/2] shrink wrap a function with a single loop: split live_edge

2014-05-08 Thread Zhenqiang Chen
Hi, The patch splits the live_edge for move_insn_for_shrink_wrap to sink the copy out of the entry block. Bootstrap and no make check regression on X86-64 and ARM. OK for trunk? Thanks! -Zhenqiang ChangeLog: 2014-05-08 Zhenqiang Chen zhenqiang.c...@linaro.org * function.c

[ping] [PATCH] config-list.mk: `show' target to show all considered targets

2014-05-08 Thread Jan-Benedict Glaw
On Mon, 2014-05-05 14:05:40 +0200, Jan-Benedict Glaw jbg...@lug-owl.de wrote: I'd like to install this patch, which would help me to run the build robot (http://toolchain.lug-owl.de/buildbot/): 2014-05-05 Jan-Benedict Glaw jbg...@lug-owl.de contrib/ * config-list.mk (show): New

Re: [PATCH, Pointer Bounds Checker 1/x] Pointer bounds type and mode

2014-05-08 Thread Ilya Enkovich
2014-05-06 19:09 GMT+04:00 Jeff Law l...@redhat.com: On 05/06/14 07:31, Richard Biener wrote: On Tue, May 6, 2014 at 2:10 PM, Ilya Enkovich enkovich@gmail.com wrote: 2014-04-16 15:00 GMT+04:00 Ilya Enkovich enkovich@gmail.com: Hi, This patch restarts the series for introducing

Re: [PATCH, 2/2] shrink wrap a function with a single loop: split live_edge

2014-05-08 Thread Steven Bosscher
On Thu, May 8, 2014 at 10:07 AM, Zhenqiang Chen wrote: The patch splits the live_edge for move_insn_for_shrink_wrap to sink the copy out of the entry block. Maybe also time to take the shrink-wrapping code out of function.c and put it in its own file? Ciao! Steven

Re: [PATCH GCC]Pick up more address lowering cases for ivopt and tree-affine.c

2014-05-08 Thread Bin.Cheng
On Tue, May 6, 2014 at 6:44 PM, Richard Biener richard.guent...@gmail.com wrote: On Tue, May 6, 2014 at 10:39 AM, Bin.Cheng amker.ch...@gmail.com wrote: On Fri, Dec 6, 2013 at 6:19 PM, Richard Biener richard.guent...@gmail.com wrote: Hi, I split the patch into two and updated the test case.

[gomp4] Mark __OPENMP_TARGET__ as addressable (was: Offloading patches (2/3): Add tables generation)

2014-05-08 Thread Thomas Schwinge
Hi! On Tue, 17 Dec 2013 15:39:57 +0400, Michael V. Zolotukhin michael.v.zolotuk...@gmail.com wrote: in this patch we start to pass '__OPENMP_TARGET__' symbol to GOMP_target calls. --- a/gcc/omp-low.c +++ b/gcc/omp-low.c @@ -8371,19 +8372,22 @@ expand_omp_target (struct omp_region *region)

Re: [gomp4] Add tables generation

2014-05-08 Thread Bernd Schmidt
On 05/06/2014 05:32 PM, Ilya Verbin wrote: On 05 Apr 17:22, Bernd Schmidt wrote: Things seemed to work over here, but now I'm not certain whether the __start_/__stop_ functionality is GNU ld specific? Maybe we should just go back to the previous version of this patch which didn't try to use

[PATCH] Remove old pointer conversion special-case

2014-05-08 Thread Richard Biener
This removes special-casing of pointer conversion handling in gimple_fold_stmt_to_constant_1 - this was from times where most pointer conversions were not useless (only conversions to function/method pointers from other kinds are considered not useless - which I'll change with a followup).

[PATCH] Fix oversight in call gimplification

2014-05-08 Thread Richard Biener
When gimplifying a call we now remember the original function type used and record it in gimple_call_fntype. But we fail to use exactly that type for looking at TYPE_ARG_TYPES. The following fixes that. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2014-05-08

[PATCH] Improve and simplify VN expression combining

2014-05-08 Thread Richard Biener
The following gets rid of SCCVNs valueize_expr which was used on GENERIC expressions built via vn_get_expr_for which is used for stmt combining via fold (yeah, I know ...). The odd way was that it first folded and built the expression and then valueized it (and not folding again), resulting in

[C++ Patch] PR 13981 (resend)

2014-05-08 Thread Paolo Carlini
Hi, some time ago I worked a bit on this very old diagnostic enhancement issue: http://gcc.gnu.org/ml/gcc-patches/2013-08/msg01588.html and yesterday we got a duplicate. Adding an inform to convert_for_assignment still makes sense to me, thus I resurrected the patch, tightened a bit the

Re: RFC: Faster for_each_rtx-like iterators

2014-05-08 Thread Trevor Saunders
On Thu, May 08, 2014 at 07:25:50AM +0100, Richard Sandiford wrote: Trevor Saunders tsaund...@mozilla.com writes: On Wed, May 07, 2014 at 09:52:49PM +0100, Richard Sandiford wrote: I noticed for_each_rtx showing up in profiles and thought I'd have a go at using worklist-based iterators

Re: [C++ Patch] PR 13981 (resend)

2014-05-08 Thread Paolo Carlini
... Manuel suggested to also use DECL_SOURCE_LOCATION, while we are at it. Thus I'm retesting the below. Thanks, Paolo. // Index: cp/typeck.c === --- cp/typeck.c (revision 210205) +++ cp/typeck.c (working copy) @@

[PATCH][1/n][RFC] Make FRE/PRE somewhat predicate aware

2014-05-08 Thread Richard Biener
Ok, not really predicate aware, but this makes value-numbering pessimistically handle non-executable edges. In the following patch groundwork is laid and PHI value-numbering is adjusted to take advantage of edges known to be not executable. SCCVN is not well-suited to be control aware, but we

Re: [Patch, Fortran] Add support for TS18508's CO_SUM/MAX/MIN (part 1/2)

2014-05-08 Thread Paul Richard Thomas
Dear Tobias, OK for trunk with one slight quibble: I would rather see + /* Generate the function call. */ + if (code-resolved_isym-id == GFC_ISYM_CO_MAX) +fndecl = gfor_fndecl_co_max; + else if (code-resolved_isym-id == GFC_ISYM_CO_MIN) +fndecl = gfor_fndecl_co_min; + else if

[libgcc, build] Don't build libgcc-unwind.map with --disable-shared (PR libgcc/61097)

2014-05-08 Thread Rainer Orth
As reported in the PR, libgcc fails to build on Solaris with --disable-shared: the creation of libgcc-unwind.map depends on libgcc-std.ver which isn't built in this case. Fixed as follows, tested by verifying that a --disable-shared i386-pc-solaris2.10 build gets into stage2 without trying to

Re: [wide-int] Add fast path for hosts with HWI widening multiplication

2014-05-08 Thread Ramana Radhakrishnan
On Wed, Dec 4, 2013 at 12:56 PM, Richard Sandiford rdsandif...@googlemail.com wrote: Richard Sandiford rdsandif...@googlemail.com writes: This patch handles multiplications using a single HWIxHWI-2HWI multiplication on hosts that have one. This removes all uses of the slow (half-HWI) path

Re: [Fortran, Patch] Some prep patches for coarray communication

2014-05-08 Thread Paul Richard Thomas
Dear Tobias, This one is fine for trunk. Thanks for the patch. Paul

[patch,avr] Fix PR61055: Wrong branch instruction after INC, DEC.

2014-05-08 Thread Georg-Johann Lay
Some instructions like INC, DEC, NEG currently set cc0 to set_zn which is not the whole story because they also set the V flag. This leads to a wrong branch instruction in the remainder as cc0 is used. Fix is to do same as clobber cc0. For the matter of clarity, I added a new cc0 alternative

Re: [wide-int] Add fast path for hosts with HWI widening multiplication

2014-05-08 Thread Richard Sandiford
Ramana Radhakrishnan ramana@googlemail.com writes: diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 354cdb9..8ef9a0f 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2014-05-08 Ramana Radhakrishnan ramana.radhakrish...@arm.com + + * wide-int.cc (UTItype): Define.

Re: Contributing new gcc targets: i386-*-dragonfly and x86-64-*-dragonfly

2014-05-08 Thread Jonathan Wakely
On 3 May 2014 08:11, John Marino wrote: On 5/2/2014 22:15, Joseph S. Myers wrote: On Fri, 2 May 2014, John Marino wrote: 1) I don't know which type definitions are missing (iow, the important ones from sys/type.h that are required to build gcc) The default presumption should be: *

Re: [RFA/dwarf v2] Add DW_AT_GNAT_use_descriptive_type flag for Ada units.

2014-05-08 Thread Joel Brobecker
Ping? I understand there was a freeze period for a while, but the patch has been out for a little over 3 months, now, and is fairly trivial. Can someone review it for me? Suggest also including it in 4.9. Thank you! On Tue, Feb 19, 2013 at 10:50:46PM -0500, Jason Merrill wrote: On 02/19/2013

Re: Contributing new gcc targets: i386-*-dragonfly and x86-64-*-dragonfly

2014-05-08 Thread Jeff Law
On 05/08/14 07:14, Jonathan Wakely wrote: Ian's approved the libiberty.h change, Joseph's approved the stddef.h change, I've approved the libstdc++ parts. IIUC it still needs explicit approval for the rest, e.g. trivial adjustments to configuration stuff in libitm and libcilkrts. Are there

Re: Contributing new gcc targets: i386-*-dragonfly and x86-64-*-dragonfly

2014-05-08 Thread John Marino
On 5/8/2014 15:32, Jeff Law wrote: On 05/08/14 07:14, Jonathan Wakely wrote: Anyone willing to give it an overall approval? I'll take a look at the rest. I mostly wanted someone else to deal with stddef.h :-) Thanks Jeff! I'm am very appreciative of that. John

Re: [RS6000] Fix PR61098, Poor code setting count register

2014-05-08 Thread David Edelsohn
On Wed, May 7, 2014 at 9:48 PM, Alan Modra amo...@gmail.com wrote: On powerpc64, to set a large loop count we have code like the following after split1: (insn 67 14 68 4 (set (reg:DI 160) (const_int 99942400 [0x5f5])) /home/amodra/unaligned_load.c:14 -1 (nil)) (insn 68 67

Re: [C++ Patch] PR 13981 (resend)

2014-05-08 Thread Jason Merrill
OK. Jason

RE: [Patch, PR 60158] Generate .fixup sections for .data.rel.ro.local entries.

2014-05-08 Thread rohitarul...@freescale.com
-Original Message- From: Alan Modra [mailto:amo...@gmail.com] Sent: Saturday, April 26, 2014 11:52 AM To: Dharmakan Rohit-B30502 Cc: gcc-patches@gcc.gnu.org; dje@gmail.com; Wienskoski Edmar-RA8797 Subject: Re: [Patch, PR 60158] Generate .fixup sections for .data.rel.ro.local

Re: RFA: Fix calculation of size of builtin setjmp buffer

2014-05-08 Thread Nicholas Clifton
Hi Mike, How about GET_MODE_SIZE (STACK_SAVEAREA_MODE (SAVE_NONLOCAL)) / GET_MODE_SIZE (Pmode) + 2 + /* slop for mips, see builtin_setjmp_setup */ 1 - 1. This retains the slop for mips, and fixes ports like ia64 and s390 (see STACK_SAVEAREA_MODE on those ports, it is larger one might

Re: emit __float128 typeinfo

2014-05-08 Thread Marc Glisse
Ping http://gcc.gnu.org/ml/gcc-patches/2014-04/msg01651.html On Fri, 25 Apr 2014, Marc Glisse wrote: On Fri, 25 Apr 2014, Marc Glisse wrote: the previous patch had to be reverted as it broke the strange handling of vectors in the ARM target. This new patch should be much more conservative

RE: [PATCH GCC]Add 'force-dwarf-lexical-blocks' command line option

2014-05-08 Thread Herman, Andrei
-Original Message- From: Joseph Myers [mailto:jos...@codesourcery.com] Sent: Wednesday, May 07, 2014 9:01 PM To: Herman, Andrei Cc: gcc-patches@gcc.gnu.org; herman_and...@mentor.com Subject: Re: [PATCH GCC]Add 'force-dwarf-lexical-blocks' command line option On Wed, 7 May 2014,

Fwd: [PATCH, alpha]: Fix PR61092, wide-int merge broke alpha bootstrap

2014-05-08 Thread Uros Bizjak
Hello! Wide-int merge triggered following ICE: In file included from ../../gcc-svn/trunk/gcc/wide-int.cc:37:0: ../../gcc-svn/trunk/gcc/wide-int.cc: In function ‘unsigned int wi::mul_internal(long int*, const long int*, unsigned int, const long int*, unsigned int, unsigned int, signop, bool*,

Re: Fix some tests for TARGET_SSE_PACKED_SINGLE_INSN_OPTIMAL

2014-05-08 Thread Uros Bizjak
On Thu, May 8, 2014 at 3:10 AM, Joseph S. Myers jos...@codesourcery.com wrote: Having fixed TARGET_SSE_PACKED_SINGLE_INSN_OPTIMAL to apply only to 128-bit vectors, some --with-arch=bdver3 --with-cpu=bdver3 scan-assembler failures relating to that tuning remain, because of different choices of

Re: [Patch, PR 60158] Generate .fixup sections for .data.rel.ro.local entries.

2014-05-08 Thread David Edelsohn
Rohit, The subject line and thread may confuse people that this is a PowerPC-specific issue. You need approval from a reviewer with authority over varasm.c. Thanks, David On Thu, May 8, 2014 at 9:54 AM, rohitarul...@freescale.com rohitarul...@freescale.com wrote: -Original Message-

Re: [PATCH, PR58066] preferred_stack_boundary update for tls expanded call

2014-05-08 Thread Uros Bizjak
On Thu, May 8, 2014 at 12:59 AM, Wei Mi w...@google.com wrote: The calls added in the templates of tls_local_dynamic_base_32 and tls_global_dynamic_32 in pr58066-3.patch are used to prevent sched2 from moving sp setting across implicit tls calls, but those calls make the combine of

Re: [C PATCH] Warn for _Alignas + main (PR c/61077)

2014-05-08 Thread Marek Polacek
Sorry, the subject should say _Atomic instead of _Alignas. Marek

Re: RFA: Fix calculation of size of builtin setjmp buffer

2014-05-08 Thread Mike Stump
On May 8, 2014, at 7:24 AM, Nicholas Clifton ni...@redhat.com wrote: What do you think of this version ? Now we just need a __builtin_setjmp style of maintainer to review…

[PING] [PATCH, wwwdocs, AArch64] Document issues with singleton vector types

2014-05-08 Thread Yufeng Zhang
Ping~ Originally posted here: http://gcc.gnu.org/ml/gcc-patches/2014-05/msg00019.html Thanks, Yufeng On 05/01/14 17:57, Yufeng Zhang wrote: Hi, This patch documents issues with singleton vector types in the 4.9 AArch64 backend. On AArch64, the singleton vector types int64x1_t, uint64x1_t

Re: [wide-int] Add fast path for hosts with HWI widening multiplication

2014-05-08 Thread Joseph S. Myers
On Thu, 8 May 2014, Ramana Radhakrishnan wrote: DATE Ramana Radhakrishnan ramana.radhakrish...@arm.com * wide-int.cc (UTItype): Define. (UDWtype): Define for appropriate W_TYPE_SIZE. This breaks builds for 32-bit hosts, where TImode isn't supported. You can only use TImode

Re: [PATCH, ARM] Fix PR60609 (Error: value of 256 too large for field of 1 bytes)

2014-05-08 Thread Charles Baylis
On 25 April 2014 16:31, Ramana Radhakrishnan ramana@googlemail.com wrote: On Fri, Apr 25, 2014 at 4:29 PM, Charles Baylis charles.bay...@linaro.org wrote: OK to backport to 4.8 and 4.7? Ok by me but give 24 working hours for an RM to object. Committed to 4.7 as r210227. Committed to 4.8

[patch] libstdc++/57394 - copy operations for basic_streambuf

2014-05-08 Thread Jonathan Wakely
This patch isn't very useful on its own, but fixes the PR and should be useful when adding move operations to the derived streambufs. The mem-initializer for basic_streambuf::_M_out_end was initialized from __sb._M_out_cur, which I assume was not intentional. Those initializers were only added

Re: [PATCH] Add support for MIPS r3 and r5

2014-05-08 Thread Joseph S. Myers
On Thu, 8 May 2014, Andrew Bennett wrote: Hi, This patch adds support for MIPS r3 and r5 to GCC. I have updated the msgid strings in the .po files for the error message I changed. Can I assume the actual msgstr entries will be updated later on? Never modify .po files in GCC; they should

RE: [PATCH GCC]Add 'force-dwarf-lexical-blocks' command line option

2014-05-08 Thread Joseph S. Myers
On Thu, 8 May 2014, Herman, Andrei wrote: The changes in gcc/c/c-decl.c are meant to deal with this problem. Declarations that would fall into the scope of a newly created label scope are moved into the enclosing normal (non label) scope, where they actually belong. Shouldn't you be

Re: [C PATCH] Warn for _Alignas + main (PR c/61077)

2014-05-08 Thread Joseph S. Myers
On Thu, 8 May 2014, Marek Polacek wrote: Joseph pointed out that we don't warn when the return type of main or the parameter type of main have the _Atomic qualifier. This patch adds such warning. Regtested/bootstrapped on x86_64-linux, ok for trunk? OK. -- Joseph S. Myers

[PATCH, AArch64] Use MOVN to generate 64-bit negative immediates where sensible

2014-05-08 Thread Ian Bolton
Hi, It currently takes 4 instructions to generate certain immediates on AArch64 (unless we put them in the constant pool). For example ... long long beefcafebabe () { return 0xBEEFCAFEBABEll; } leads to ... mov x0, 0x47806 mov x0, 0xcafe, lsl 16 mov x0, 0xbeef, lsl

[PATCH, AArch64] Fix macro in vdup_lane_2 test case

2014-05-08 Thread Ian Bolton
This patch fixes a defective macro definition, based on correct definition in similar testcases. The test currently passes through luck rather than correctness. OK for commit? Cheers, Ian 2014-05-08 Ian Bolton ian.bol...@arm.com gcc/testsuite * gcc.target/aarch64/vdup_lane_2.c

RE: [PATCH GCC]Add 'force-dwarf-lexical-blocks' command line option

2014-05-08 Thread Herman, Andrei
-Original Message- From: Joseph Myers [mailto:jos...@codesourcery.com] Sent: Thursday, May 08, 2014 8:27 PM To: Herman, Andrei Cc: gcc-patches@gcc.gnu.org Subject: RE: [PATCH GCC]Add 'force-dwarf-lexical-blocks' command line option On Thu, 8 May 2014, Herman, Andrei wrote:

Re: [C PATCH] Don't reject valid code with _Alignas (PR c/61053)

2014-05-08 Thread Marek Polacek
On Wed, May 07, 2014 at 11:31:38AM -0700, H.J. Lu wrote: OK, though I'm not sure if the lp64 conditions are right in the testcase It should be !ia32 instead of lp64. Ok, I changed lp64 to ! { ia32 } and committed the patch now. Marek

[patch] libstdc++/13860 enforce requirements on traits

2014-05-08 Thread Jonathan Wakely
Add static assertions to enforce requirements on the traits used with basic_filebuf, improving the diagnostics for invalid traits. Tested x86_64-linux, committed to trunk. commit 78df7e78e805c873883e63d9df4e6befa32ddac5 Author: Jonathan Wakely jwak...@redhat.com Date: Thu May 8 18:20:37 2014

Re: [wide-int] Add fast path for hosts with HWI widening multiplication

2014-05-08 Thread Richard Sandiford
Joseph S. Myers jos...@codesourcery.com writes: On Thu, 8 May 2014, Ramana Radhakrishnan wrote: DATE Ramana Radhakrishnan ramana.radhakrish...@arm.com * wide-int.cc (UTItype): Define. (UDWtype): Define for appropriate W_TYPE_SIZE. This breaks builds for 32-bit hosts, where

Re: [C PATCH] Make attributes accept enum values (PR c/50459)

2014-05-08 Thread Marek Polacek
Ping. On Fri, May 02, 2014 at 11:28:54AM +0200, Marek Polacek wrote: On Thu, May 01, 2014 at 11:20:25PM +, Joseph S. Myers wrote: On Wed, 23 Apr 2014, Marek Polacek wrote: diff --git gcc/testsuite/c-c++-common/attributes-1.c gcc/testsuite/c-c++-common/attributes-1.c index

Re: [wide-int] Add fast path for hosts with HWI widening multiplication

2014-05-08 Thread Kenneth Zadeck
everyone who has a private port will hate you forever. note that i have 2 of them. On 05/08/2014 02:31 PM, Richard Sandiford wrote: Joseph S. Myers jos...@codesourcery.com writes: On Thu, 8 May 2014, Ramana Radhakrishnan wrote: DATE Ramana Radhakrishnan ramana.radhakrish...@arm.com

[jit] Introduce params_c_finalize

2014-05-08 Thread David Malcolm
Attempting to repeatedly compile in-process led to assertion failures in a release build on the 2nd compile within a process: 156 void 157 set_default_param_value (compiler_param num, int value) 158 { 159 gcc_assert (!params_finished); 160 161 compiler_params[(int)

[PATCH, i386]: Fix PR59952, -march=core-avx2 should not enable RTM

2014-05-08 Thread Uros Bizjak
Hello! Apparently, not all Haswell processors have TSX. Attached patch removes PTA_RTM from default Haswell flags. PTA_HLX still makes sense for Haswell processors, since the prefix is ignored on non-TSX processors. 2014-05-08 Uros Bizjak ubiz...@gmail.com PR target/59952 *

Re: [wide-int] Add fast path for hosts with HWI widening multiplication

2014-05-08 Thread Richard Sandiford
Kenneth Zadeck zad...@naturalbridge.com writes: everyone who has a private port will hate you forever. note that i have 2 of them. Got any other ideas though? I suppose if we're prepared to break compatibility with whatever the upstream of longlong.h is, we could make more use of intN_t and

Re: [PATCH] Add support for MIPS r3 and r5

2014-05-08 Thread Richard Sandiford
Andrew Bennett andrew.benn...@imgtec.com writes: diff --git a/gcc/config/mips/mips-cpus.def b/gcc/config/mips/mips-cpus.def index 07fbf9c..f2e23c6 100644 --- a/gcc/config/mips/mips-cpus.def +++ b/gcc/config/mips/mips-cpus.def @@ -44,9 +44,13 @@ MIPS_CPU (mips4, PROCESSOR_R8000, 4, 0)

Re: [PATCH, Pointer Bounds Checker 1/x] Pointer bounds type and mode

2014-05-08 Thread Jeff Law
On 05/08/14 02:17, Ilya Enkovich wrote: Right. Richi explicitly wanted the entire set approved before staging in any of the bits. I thought it would be useful to have approved codes in the trunk to reveal some possible problems on earlier stages. It also requires significant effort to keep

Re: [wide-int] Add fast path for hosts with HWI widening multiplication

2014-05-08 Thread H.J. Lu
On Thu, May 8, 2014 at 12:18 PM, Richard Sandiford rdsandif...@googlemail.com wrote: Kenneth Zadeck zad...@naturalbridge.com writes: everyone who has a private port will hate you forever. note that i have 2 of them. Got any other ideas though? I suppose if we're prepared to break

[patch] fix some comments in libstdc++ files

2014-05-08 Thread Jonathan Wakely
Update an old URL and an old pathname. Tested x86_64-linux, committed to 4.8, 4.9 and trunk. commit 4692e5802722954c4dda17e8b7f4ed4b78bcc272 Author: Jonathan Wakely jwak...@redhat.com Date: Thu May 8 19:38:42 2014 +0100 * include/std/iostream: Fix URL in comment. *

Re: [wide-int] Add fast path for hosts with HWI widening multiplication

2014-05-08 Thread Jakub Jelinek
On Thu, May 08, 2014 at 12:34:28PM -0700, H.J. Lu wrote: On Thu, May 8, 2014 at 12:18 PM, Richard Sandiford rdsandif...@googlemail.com wrote: Kenneth Zadeck zad...@naturalbridge.com writes: everyone who has a private port will hate you forever. note that i have 2 of them. Got any

Re: [PATCH, Pointer Bounds Checker 1/x] Pointer bounds type and mode

2014-05-08 Thread H.J. Lu
On Thu, May 8, 2014 at 12:28 PM, Jeff Law l...@redhat.com wrote: On 05/08/14 02:17, Ilya Enkovich wrote: Right. Richi explicitly wanted the entire set approved before staging in any of the bits. I thought it would be useful to have approved codes in the trunk to reveal some possible

Re: [wide-int] Add fast path for hosts with HWI widening multiplication

2014-05-08 Thread H.J. Lu
On Thu, May 8, 2014 at 12:42 PM, Jakub Jelinek ja...@redhat.com wrote: On Thu, May 08, 2014 at 12:34:28PM -0700, H.J. Lu wrote: On Thu, May 8, 2014 at 12:18 PM, Richard Sandiford rdsandif...@googlemail.com wrote: Kenneth Zadeck zad...@naturalbridge.com writes: everyone who has a private

[patch] libstdc++/61117 FAQ should use free software

2014-05-08 Thread Jonathan Wakely
Committed to 4.8, 4.9 and trunk.

Re: [patch] libstdc++/61117 FAQ should use free software

2014-05-08 Thread Jonathan Wakely
On 08/05/14 20:52 +0100, Jonathan Wakely wrote: Committed to 4.8, 4.9 and trunk. And 4.7 since that branch is still open too.

Re: [wide-int] Add fast path for hosts with HWI widening multiplication

2014-05-08 Thread Richard Sandiford
H.J. Lu hjl.to...@gmail.com writes: On Thu, May 8, 2014 at 12:42 PM, Jakub Jelinek ja...@redhat.com wrote: On Thu, May 08, 2014 at 12:34:28PM -0700, H.J. Lu wrote: On Thu, May 8, 2014 at 12:18 PM, Richard Sandiford rdsandif...@googlemail.com wrote: Kenneth Zadeck zad...@naturalbridge.com

[google gcc-4_8] fix lipo ICE

2014-05-08 Thread Rong Xu
Hi, This patch fixed lipo ICE triggered by an out-of-bound access. This is google specific patch and tested with bootstrap and the program exposed the issue. Thanks, -Rong 2014-05-08 Rong Xu x...@google.com * tree-inline.c (add_local_variables): Check if the debug_expr is a

Re: [google gcc-4_8] fix lipo ICE

2014-05-08 Thread Xinliang David Li
ok. David On Thu, May 8, 2014 at 1:33 PM, Rong Xu x...@google.com wrote: Hi, This patch fixed lipo ICE triggered by an out-of-bound access. This is google specific patch and tested with bootstrap and the program exposed the issue. Thanks, -Rong

[Fortran-caf] Merge of the trunk into the branch

2014-05-08 Thread Tobias Burnus
Hi all, I have now merged the trunk into the branch. The main change is the new co_min/max/sum feature but also a patch which has reduced the differences between the trunk and the branch. - And of course, all the other trunk changes such as the wide-int patch. Committed as Rev. 210244.

Re: [wide-int] Add fast path for hosts with HWI widening multiplication

2014-05-08 Thread H.J. Lu
On Thu, May 8, 2014 at 1:12 PM, Richard Sandiford rdsandif...@googlemail.com wrote: H.J. Lu hjl.to...@gmail.com writes: On Thu, May 8, 2014 at 12:42 PM, Jakub Jelinek ja...@redhat.com wrote: On Thu, May 08, 2014 at 12:34:28PM -0700, H.J. Lu wrote: On Thu, May 8, 2014 at 12:18 PM, Richard

Re: [PATCH] Fix PR c++/60994 gcc does not recognize hidden/shadowed enumeration as valid nested-name-specifier

2014-05-08 Thread Momchil Velikov
Ping. previous post: http://gcc.gnu.org/ml/gcc-patches/2014-04/msg01938.html Again bootstrapped/regtested/diffed against xg++ (GCC) 4.10.0 20140508 (experimental) [master revision ed50168:49aa3a5:e79f58c7b12f37014efb7425399c93814cddb4c4] On 29.04.2014 12:58, Momchil Velikov wrote: Hello, gcc

RE: [PATCH GCC]Add 'force-dwarf-lexical-blocks' command line option

2014-05-08 Thread Joseph S. Myers
On Thu, 8 May 2014, Herman, Andrei wrote: Declarations that would fall into the scope of a newly created label scope are moved into the enclosing normal (non label) scope, where they actually belong. Shouldn't you be able to do something like that for the other cases as well, to

Re: [wide-int] Add fast path for hosts with HWI widening multiplication

2014-05-08 Thread Richard Sandiford
H.J. Lu hjl.to...@gmail.com writes: On Thu, May 8, 2014 at 1:12 PM, Richard Sandiford rdsandif...@googlemail.com wrote: H.J. Lu hjl.to...@gmail.com writes: On Thu, May 8, 2014 at 12:42 PM, Jakub Jelinek ja...@redhat.com wrote: On Thu, May 08, 2014 at 12:34:28PM -0700, H.J. Lu wrote: On Thu,

Re: [C PATCH] Make attributes accept enum values (PR c/50459)

2014-05-08 Thread Joseph S. Myers
On Fri, 2 May 2014, Marek Polacek wrote: Yeah, that was a weird thing to do. I yanked that particular test to a new testcase. Otherwise no changes. Tested again x86_64-linux, ok now? 2014-05-02 Marek Polacek pola...@redhat.com PR c/50459 c-family/ * c-common.c

[patch] fix impliedness of -Wunused-parameter depending on -Wexta option ordering

2014-05-08 Thread Matthias Klose
This fixes a regression introduced with 4.8, where the option ordering of -Wextra and -Wunused-parameter emits a warning, which is not emitted with 4.7. No regressions with the trunk, the 4.9 and 4.8 branches. Ok to check in for these? Matthias 2014-05-08 Manuel LC3B3pez-IbC3A1C3B1ez

Re: [patch] fix impliedness of -Wunused-parameter depending on -Wexta option ordering

2014-05-08 Thread Joseph S. Myers
On Thu, 8 May 2014, Matthias Klose wrote: This fixes a regression introduced with 4.8, where the option ordering of -Wextra and -Wunused-parameter emits a warning, which is not emitted with 4.7. No regressions with the trunk, the 4.9 and 4.8 branches. Ok to check in for these? OK. --

Re: [PATCH] Fix PR c++/60994 gcc does not recognize hidden/shadowed enumeration as valid nested-name-specifier

2014-05-08 Thread Paolo Carlini
Nit: normally, ChangeLog entries are not submitted as diffs, part of the patch proper, but separately (also because the ChangeLog files keep changing quite fast). Paolo.

iq2000-elf: wide-int fallout (was: we are starting the wide int merge)

2014-05-08 Thread Jan-Benedict Glaw
[...] Just found this for iq2000: g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual -Wmissing-format-attribute -Woverloaded-virtual -pedantic -Wno-long-long -Wno-variadic-macros

Re: iq2000-elf: wide-int fallout (was: we are starting the wide int merge)

2014-05-08 Thread Oleg Endo
On Fri, 2014-05-09 at 00:48 +0200, Jan-Benedict Glaw wrote: [...] Just found this for iq2000: g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual -Wmissing-format-attribute

UTItype fallout (was: wide-int fallout)

2014-05-08 Thread Jan-Benedict Glaw
On Fri, 2014-05-09 00:48:39 +0200, Jan-Benedict Glaw jbg...@lug-owl.de wrote: Just found this for iq2000: g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual

Re: [patch] change specific int128 - generic intN

2014-05-08 Thread DJ Delorie
The libstdc++v3 headers have __int128 hard-coded all over the place. Any suggestions on parameterizing those for the __intN types that are actually supported by the target?

Re: [patch] change specific int128 - generic intN

2014-05-08 Thread Marc Glisse
On Thu, 8 May 2014, DJ Delorie wrote: The libstdc++v3 headers have __int128 hard-coded all over the place. Any suggestions on parameterizing those for the __intN types that are actually supported by the target? (adding libstdc++@ in Cc:) The first idea that comes to mind (so possibly not

Re: [patch] change specific int128 - generic intN

2014-05-08 Thread DJ Delorie
Assuming that the formula sizeof(type)*char_bit==precision works for all It doesn't. THe MSP430 has __int20 for example. Would it be acceptable for the compiler to always define a set of macros for each of the intN types? I would have thought that would be discouraged, but it would be an

Re: [RS6000] Fix PR61098, Poor code setting count register

2014-05-08 Thread Alan Modra
On Thu, May 08, 2014 at 09:48:35AM -0400, David Edelsohn wrote: The history is 32 bit HWI. Right. The ChangeLog does not mention the changes to rs6000.md nor rs6000-protos.h. Oops, added. * config/rs6000/rs6000.md (movsi_internal1_single+1): Update call to

[PR tree-optimization/61009] Do not use a block as a joiner if it is too big for threading

2014-05-08 Thread Jeff Law
This was yet another problem issue with threading through a loop backedge and finding equivalences that should have been invalidated. In this instance, we were trying to thread through a large block. When we hit the statement threshold, thread_through_normal_block returned and thus

libbacktrace patch committed: Fixes for large binaries

2014-05-08 Thread Ian Lance Taylor
While testing on a large Google binary, I noticed that libbacktrace is allocating an inordinate amount of memory. The binary winds up with 377,944 entries in the unit_addrs vector. Each entry is 24 bytes, so this is 9,070,656 bytes, which is not too terrible. Unfortunately, for some reason I

Re: [PATCH] Windows libibery: Don't quote args unnecessarily

2014-05-08 Thread Ian Lance Taylor
On Tue, May 6, 2014 at 11:56 PM, Ray Donnelly mingw.andr...@gmail.com wrote: We only quote arguments that contain spaces, \t or characters to prevent wasting 2 characters per argument of the CreateProcess() 32,768 limit. This is OK. Thanks. Ian libiberty/pex-win32.c | 46

Re: Contributing new gcc targets: i386-*-dragonfly and x86-64-*-dragonfly

2014-05-08 Thread Jeff Law
On 05/03/14 01:11, John Marino wrote: revised patchset : http://leaf.dragonflybsd.org/~marino/gcc-df-target/patches/patch-dragonfly-target revised changelog : http://leaf.dragonflybsd.org/~marino/gcc-df-target/changelog_entries/gcc_ChangeLog_entry.txt revised commit msg:

Re: [ping] [PATCH] config-list.mk: `show' target to show all considered targets

2014-05-08 Thread Jeff Law
On 05/08/14 02:10, Jan-Benedict Glaw wrote: On Mon, 2014-05-05 14:05:40 +0200, Jan-Benedict Glaw jbg...@lug-owl.de wrote: I'd like to install this patch, which would help me to run the build robot (http://toolchain.lug-owl.de/buildbot/): 2014-05-05 Jan-Benedict Glaw jbg...@lug-owl.de

[gomp4] Merge trunk r210100 (2014-05-06) into gomp-4_0-branch

2014-05-08 Thread Thomas Schwinge
Hi! In r210257, I have committed a merge from trunk r210100 (2014-05-06) into gomp-4_0-branch. The LTO regression that appeared with the last merge, http://news.gmane.org/find-root.php?message_id=%3C87wqf483pl.fsf%40schwinge.name%3E, remains to be resolved: PASS: gcc.dg/lto/save-temps

Merge from GCC 4.9 branch to gccgo branch

2014-05-08 Thread Ian Lance Taylor
I merged GCC 4.9 branch revision 210256 to the gccgo branch. Ian