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

2014-05-06 Thread Ray Donnelly
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 | 46 +- 1 file changed, 37 insertions(+), 9 deletions(-) diff --git a/libiberty/

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

2014-05-06 Thread Ray Donnelly
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 unnecessarily Ray Donnelly (1): Windows libibery: Don't quote args unnec

Re: [RS6000] PR60737, expand_block_clear uses word stores

2014-05-06 Thread Alan Modra
On Tue, May 06, 2014 at 06:00:14PM +0930, Alan Modra wrote: > __asm__ __volatile__ ("ld %0,0(%1)" > : "=r" (scratch[0]) : "r" (p)); Ick, I can't write ppc asm. That should be "b" (p). -- Alan Modra Australia Development Lab, IBM

fallout on x86-64 from the wide-int merge.

2014-05-06 Thread Kenneth Zadeck
here is a comparison. The two areas were built using configure with no options at all on x86-64. The comparison is between revision 210112 and 210113.Tsan is very unhappy but everything else looks ok.I know that this worked a couple of days before the merge. I know that there was some

Re: debug container patch

2014-05-06 Thread Paolo Carlini
-- Francois, remember to regenerate and commit the Makefile.in changes. Thanks, Paolo.

Re: we are starting the wide int merge

2014-05-06 Thread Mike Stump
On May 6, 2014, at 3:54 PM, Mike Stump wrote: > >> My build robot experiences errors like this: >> >> 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-attribu

Re: we are starting the wide int merge

2014-05-06 Thread Mike Stump
On May 6, 2014, at 3:54 PM, Mike Stump wrote: > > I’m pretty sure this: > will fix it. If someone can confirm that would be great. I tried that on my p7 and on my x86_64 machines, they both build fine.

Re: we are starting the wide int merge

2014-05-06 Thread Jan-Benedict Glaw
On Tue, 2014-05-06 15:54:05 -0700, Mike Stump wrote: > On May 6, 2014, at 2:17 PM, Jan-Benedict Glaw wrote: > > On Tue, 2014-05-06 12:20:54 -0700, Mike Stump wrote: > > > On May 6, 2014, at 8:19 AM, Kenneth Zadeck > > > wrote: > > > > please hold off on committing patches for the next couple o

Re: we are starting the wide int merge

2014-05-06 Thread Christophe Lyon
I have noticed build failures too (arm, aarch64). It also looks like the git-svn-id property is now wrong/incomplete. For instance, commit 9a5942c1d4d9116ab74b0741cfe3894a89fd17fb has: git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/branches/wide-int@201706 138bc75d-0d04-0410-961f-82ee72b054a4 How

Re: we are starting the wide int merge

2014-05-06 Thread Mike Stump
On May 6, 2014, at 2:17 PM, Jan-Benedict Glaw wrote: > On Tue, 2014-05-06 12:20:54 -0700, Mike Stump wrote: >> On May 6, 2014, at 8:19 AM, Kenneth Zadeck wrote: >>> please hold off on committing patches for the next couple of hours as we >>> have a very large merge to do. >>> thanks. >> >> All

Re: Fix PR ipa/60965 (placement new wrt ipa-devirt)

2014-05-06 Thread Paolo Carlini
Hi Honza, On 05/05/2014 09:44 PM, Jan Hubicka wrote: Hi, this patch fixes unfortunate thinko in get_class_context that invalidates transitions from non-POD type to a polymorphic type that may happen by virtue of placement new and apparently breaks openJDK and Qt. I really tried to keep placement

Re: we are starting the wide int merge

2014-05-06 Thread Jan-Benedict Glaw
On Tue, 2014-05-06 12:20:54 -0700, Mike Stump wrote: > On May 6, 2014, at 8:19 AM, Kenneth Zadeck wrote: > > please hold off on committing patches for the next couple of hours as we > > have a very large merge to do. > > thanks. > > All done… It is in. My build robot experiences errors like t

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

2014-05-06 Thread Mike Stump
On May 6, 2014, at 8:07 AM, Nicholas Clifton wrote: > + tmp = size_int ((5 * MAX (POINTER_SIZE, BITS_PER_WORD)) / > BITS_PER_WORD - 1); This is not right. The denominator should either be GET_MODE_SIZE (Pmode) or POINTER_SIZE. See get_nl_goto_field for additional fun. Also, the save are

Re: [RFC][PATCH][MIPS] Patch to enable LRA for MIPS backend

2014-05-06 Thread Richard Sandiford
Kyrill Tkachov writes: > On 03/05/14 20:21, Richard Sandiford wrote: >> Vladimir Makarov writes: > Not sure how the constraint would/should exclude $sp-based address in > LRA. In this particular case, a spilled pseudo is changed to memory > giving the following RTL form: > >

Re: [PATCH 0/3] Compile-time gimple checking, without typedefs

2014-05-06 Thread Richard Biener
On May 6, 2014 9:15:09 PM CEST, Jeff Law wrote: >On 05/06/14 12:58, David Malcolm wrote: >>> But the prefix can be as short as e.g. "g" (for gimple), so gtry, >ggoto, >>> gassign, gcall. >> >> FWIW, I like this: I think it's clear and concise, and I think it >avoids >> the need for namespaces. (I'

Re: we are starting the wide int merge

2014-05-06 Thread Mike Stump
On May 6, 2014, at 8:19 AM, Kenneth Zadeck wrote: > please hold off on committing patches for the next couple of hours as we have > a very large merge to do. > thanks. All done… It is in.

Re: [PATCH 0/3] Compile-time gimple checking, without typedefs

2014-05-06 Thread Jeff Law
On 05/06/14 12:58, David Malcolm wrote: But the prefix can be as short as e.g. "g" (for gimple), so gtry, ggoto, gassign, gcall. FWIW, I like this: I think it's clear and concise, and I think it avoids the need for namespaces. (I'd prefer not to have a namespace, due to gengtype). Then let's go

Re: [PATCH 0/3] Compile-time gimple checking, without typedefs

2014-05-06 Thread David Malcolm
On Mon, 2014-05-05 at 21:53 +0200, Jakub Jelinek wrote: > On Mon, May 05, 2014 at 01:44:06PM -0600, Jeff Law wrote: > > On 05/05/14 11:37, Richard Biener wrote: > > > > > >Well, I hope that Andrew doesn't do without a namespace (and I still > > >don't believe in what he tries to achieve without lay

Re: Hurd port for gcc go PATCH 7-9 (9)

2014-05-06 Thread Samuel Thibault
Svante Signell, le Tue 06 May 2014 14:13:54 +0200, a écrit : > > > +# Special treatment of EWOULDBLOCK for GNU/Hurd > > > +# /usr/include/bits/errno.h: #define EWOULDBLOCK EAGAIN > > > +egrep '^const EWOULDBLOCK = Errno(_EWOULDBLOCK)' ${OUT} | \ > > > +sed -i.bak -e 's/_EWOULDBLOCK/_EAGAIN/' ${

Re: [C++ Patch] PR 60999

2014-05-06 Thread Jason Merrill
On 05/06/2014 04:23 AM, Paolo Carlini wrote: On further thought (yesterday was a little tired, sorry), I think that nsdmi-template7.C is telling us that, in fact, we *do* want something true when we are handling an NSDMI inside a non-template class of a template class! Right, sorry, I was think

Re: [PATCH] [ARM] [RFC] Fix longstanding push_minipool_fix ICE (PR49423, lp1296601)

2014-05-06 Thread Julian Brown
On Tue, 6 May 2014 17:05:16 +0100 Charles Baylis wrote: > Ping? > > At this stage looking for general feedback on whether the define_split > approach in this patch is appropriate. If it is, I'll do a clean patch > for full review. > > Archive link: http://gcc.gnu.org/ml/gcc-patches/2014-04/msg0

Re: [PATCH] [ARM] [RFC] Fix longstanding push_minipool_fix ICE (PR49423, lp1296601)

2014-05-06 Thread Charles Baylis
Ping? At this stage looking for general feedback on whether the define_split approach in this patch is appropriate. If it is, I'll do a clean patch for full review. Archive link: http://gcc.gnu.org/ml/gcc-patches/2014-04/msg00078.html On 2 April 2014 14:29, Charles Baylis wrote: > Hi > > This p

Re: [gomp4] Add tables generation

2014-05-06 Thread Ilya Verbin
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 this. > > Bernd This approach does not w

Re: [AArch64][4.8 backport] Fix default CPU architecture flags

2014-05-06 Thread Marcus Shawcroft
On 6 May 2014 15:25, Kyrill Tkachov wrote: > 2014-05-06 Kyrylo Tkachov > > * config.gcc (aarch64*-*-*): Use ISA flags from aarch64-arches.def. > Do not define target_cpu_default2 to generic. OK /Marcus

we are starting the wide int merge

2014-05-06 Thread Kenneth Zadeck
please hold off on committing patches for the next couple of hours as we have a very large merge to do. thanks. kenny

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

2014-05-06 Thread Jeff Law
On 05/06/14 07:31, Richard Biener wrote: On Tue, May 6, 2014 at 2:10 PM, Ilya Enkovich wrote: 2014-04-16 15:00 GMT+04:00 Ilya Enkovich : Hi, This patch restarts the series for introducing Pointer Bounds Checker instrumentation and supporting Intel Memory Protection Extension (MPX) technolog

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

2014-05-06 Thread Nicholas Clifton
Hi Jakub, But what will this do on targets where POINTER_SIZE is smaller than BITS_PER_WORD? E.g. I think some options on s390 or ppc. If you want it to be always 5 pointers, then you want tmp = size_int (4); and not something else, otherwise it really depends on how exactly it is used, perh

Re: [Patch,Microblaze]: Added Break Handler Support

2014-05-06 Thread Michael Eager
On 05/06/14 06:10, Ajit Kumar Agarwal wrote: Hello All: Could you please review this patch. [PATCH] Break handler is used to support a normal hardware break. This is similar to interrupt_handler except that RTBD should be used instead of RTID. Changelog 2014-05-06 Nagaraju * gcc/config/mic

Re: [PATCH, PR52252] Vectorization for load/store groups of size 3.

2014-05-06 Thread Richard Biener
On Tue, 6 May 2014, Evgeny Stupachenko wrote: > Patch with fixes attached. Ok if bootstrap/regtest passes. Thanks, Richard. > Currently if-structure is as following: > + if (count == 3) > ... > + else > + { > + /* If length is not equal to 3 then only power of 2 is suppo

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

2014-05-06 Thread Jakub Jelinek
On Tue, May 06, 2014 at 03:34:37PM +0100, Nicholas Clifton wrote: > --- gcc/except.c (revision 210096) > +++ gcc/except.c (working copy) > @@ -286,8 +286,8 @@ >tmp = size_int (FIRST_PSEUDO_REGISTER + 2 - 1); > #endif > #else > - /* builtin_setjmp takes a pointer to 5 words.

[PATCH2/2, PR52252] Vectorization for load/store groups of size 3.

2014-05-06 Thread Evgeny Stupachenko
2nd part of patch is on stores group. Bootstrap and make check passed on x86. Is it ok? 2014-05-06 Evgeny Stupachenko * tree-vect-data-refs.c (vect_grouped_store_supported): New check for storess group of length 3. (vect_permute_store_chain): New permutations for store

Re: [PATCH, PR52252] Vectorization for load/store groups of size 3.

2014-05-06 Thread Evgeny Stupachenko
Patch with fixes attached. Currently if-structure is as following: + if (count == 3) ... + else + { + /* If length is not equal to 3 then only power of 2 is supported. */ + gcc_assert (exact_log2 (count) != -1); For stores group I've created another mail thread. T

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

2014-05-06 Thread Richard Biener
On Tue, May 6, 2014 at 4:34 PM, Nicholas Clifton wrote: > Hi Jakub, > > >>> /* builtin_setjmp takes a pointer to 5 words. */ >>> - tmp = size_int (5 * BITS_PER_WORD / POINTER_SIZE - 1); >>> + tmp = size_int (5 * POINTER_SIZE / BITS_PER_WORD - 1); > > >> That doesn't look correct

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

2014-05-06 Thread Nicholas Clifton
Hi Jakub, /* builtin_setjmp takes a pointer to 5 words. */ - tmp = size_int (5 * BITS_PER_WORD / POINTER_SIZE - 1); + tmp = size_int (5 * POINTER_SIZE / BITS_PER_WORD - 1); That doesn't look correct to me. If the code wants to create 5 words long array, but with pointer el

[AArch64][4.8 backport] Fix default CPU architecture flags

2014-05-06 Thread Kyrill Tkachov
Hi all, A latent issue was recently exposed in the 4.8 branch. Due to the recently fixed TARGET_FLOAT macro on that branch (r210004), we now hit the bug that the configure-time default architecture flags do not include FP and SIMD even though the default cpu target mandates them in aarch64-arc

Re: [AArch64/ARM 3/3] Add execution tests of ARM ZIP Intrinsics

2014-05-06 Thread Alan Lawrence
Right. Tests appear correct and problem is in arm_neon.h. I'll work on a fix. Cheers, Alan Christophe Lyon wrote: This caused: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=61062 On 30 April 2014 11:18, Ramana Radhakrishnan wrote: On Thu, Mar 27, 2014 at 10:53 AM, Alan Lawrence wrote: Fina

Re: [RFC][PATCH][MIPS] Patch to enable LRA for MIPS backend

2014-05-06 Thread Kyrill Tkachov
On 03/05/14 20:21, Richard Sandiford wrote: Vladimir Makarov writes: Not sure how the constraint would/should exclude $sp-based address in LRA. In this particular case, a spilled pseudo is changed to memory giving the following RTL form: (insn 30 29 31 4 (set (reg:SI 4 $4) (and:SI (

Re: debug container patch

2014-05-06 Thread Jonathan Wakely
On 28/04/14 23:07 +0200, François Dumont wrote: On 27/04/2014 15:39, Jonathan Wakely wrote: On 17/04/14 22:43 +0200, François Dumont wrote: Hi Here is a patch to globally enhance debug containers implementation. François, sorry for the delay, this is a large patch and I wanted to give it t

[PATCH][buildrobot] AVR: Fix warning for unused argument

2014-05-06 Thread Jan-Benedict Glaw
Hi! This patch fixes a warning / leftover (from Feb 22) which I oversaw too long in buildrobot's output. Committed as obvious. g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual -Wmiss

Go patch committed: Store flags for division checks in Gogo object

2014-05-06 Thread Ian Lance Taylor
This patch by Chris Manghane stores the flags for division checks in the Gogo object, rather than referring directly to the GCC global variables. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian 2014-05-06 Chris Manghane * go-c.h (go_create_go

[PATCH][ARM] Remove unused vtrn, vzip, vuzp NEON builtins

2014-05-06 Thread Kyrill Tkachov
Hi all, This patch removes the NEON builtin functions for vtrn, vzip, vuzp and their associated wiring and machine descriptions. The builtins were initially used to implement the corresponding intrinsics in arm_neon.h but those have since been reimplemented using the more generic __builtin_shu

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

2014-05-06 Thread Richard Biener
On Tue, May 6, 2014 at 2:10 PM, Ilya Enkovich wrote: > 2014-04-16 15:00 GMT+04:00 Ilya Enkovich : >> Hi, >> >> This patch restarts the series for introducing Pointer Bounds Checker >> instrumentation and supporting Intel Memory Protection Extension (MPX) >> technology. Detailed description is o

Re: GCC's -fsplit-stack disturbing Mach's vm_allocate

2014-05-06 Thread Samuel Thibault
Svante Signell, le Tue 06 May 2014 15:25:38 +0200, a écrit : > On Tue, 2014-05-06 at 15:07 +0200, Samuel Thibault wrote: > > Svante Signell, le Tue 06 May 2014 15:05:20 +0200, a écrit : > > > On Tue, 2014-05-06 at 14:51 +0200, Samuel Thibault wrote: > > > > Just to explicitly ask for it: > > > > >

Re: GCC's -fsplit-stack disturbing Mach's vm_allocate

2014-05-06 Thread Svante Signell
On Tue, 2014-05-06 at 15:07 +0200, Samuel Thibault wrote: > Svante Signell, le Tue 06 May 2014 15:05:20 +0200, a écrit : > > On Tue, 2014-05-06 at 14:51 +0200, Samuel Thibault wrote: > > > Just to explicitly ask for it: > > > > > > Svante Signell, le Tue 06 May 2014 10:06:49 +0200, a écrit : > > >

Re: [C PATCH] Warn about variadic main (PR c/60156)

2014-05-06 Thread Marek Polacek
On Thu, May 01, 2014 at 11:37:58PM +, Joseph S. Myers wrote: > As a matter of QoI we should also diagnose use of _Atomic in the return > type or argument types of main (something I deferred doing in the initial > _Atomic support). Ok, I opened PR61077 and I'm taking it. But I wonder if I sh

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

2014-05-06 Thread Jakub Jelinek
On Tue, May 06, 2014 at 01:55:18PM +0100, Nick Clifton wrote: > 2014-05-06 Nick Clifton > > * except.c (init_eh): Fix computation of builtin setjmp buffer > size. > > Index: gcc/except.c > === > --- gcc/except.c (

Re: GCC's -fsplit-stack disturbing Mach's vm_allocate

2014-05-06 Thread Samuel Thibault
Svante Signell, le Tue 06 May 2014 15:05:20 +0200, a écrit : > On Tue, 2014-05-06 at 14:51 +0200, Samuel Thibault wrote: > > Just to explicitly ask for it: > > > > Svante Signell, le Tue 06 May 2014 10:06:49 +0200, a écrit : > > > For some (yet) unknown reason all libgo tests fails with a segfault

Re: GCC's -fsplit-stack disturbing Mach's vm_allocate

2014-05-06 Thread Svante Signell
On Tue, 2014-05-06 at 14:51 +0200, Samuel Thibault wrote: > Just to explicitly ask for it: > > Svante Signell, le Tue 06 May 2014 10:06:49 +0200, a écrit : > > For some (yet) unknown reason all libgo tests fails with a segfault when > > run in the build tree: make, sh or something else, the test c

Re: GCC's -fsplit-stack disturbing Mach's vm_allocate

2014-05-06 Thread Ian Lance Taylor
On Tue, May 6, 2014 at 4:06 AM, Svante Signell wrote: > On Fri, 2014-05-02 at 12:52 +0200, Samuel Thibault wrote: >> Svante Signell, le Fri 02 May 2014 12:45:56 +0200, a écrit : >> > On Fri, 2014-05-02 at 12:00 +0200, Samuel Thibault wrote: >> > > Samuel Thibault, le Fri 02 May 2014 11:57:53 +0200

Go patch committed: Fix bug in embedded method handling

2014-05-06 Thread Ian Lance Taylor
This patch to the Go frontend fixes a bug in the handling of methods of embedded fields. If the same type was embedded in a struct at different levels, only the first embedded instance was considered when building the set of all methods. If the second instance was at a higher level, its methods w

RFA: Fix calculation of size of builtin setjmp buffer

2014-05-06 Thread Nick Clifton
Hi Guys, There is a small bug in the computation for the size of the builtin setjmp buffer. The size is based upon BITS_PER_WORD / POINTER_SIZE which for most targets equates to 1. But for targets where pointers are larger than a word, it equates to zero. This leads to stack corruptio

Re: GCC's -fsplit-stack disturbing Mach's vm_allocate

2014-05-06 Thread Samuel Thibault
Just to explicitly ask for it: Svante Signell, le Tue 06 May 2014 10:06:49 +0200, a écrit : > For some (yet) unknown reason all libgo tests fails with a segfault when > run in the build tree: make, sh or something else, the test commands are > rather hard to track. Doesn't that dump a core? Do y

Re: [PATCH, Pointer Bounds Checker 9/x] Cgraph extension

2014-05-06 Thread Ilya Enkovich
Ping 2014-04-16 18:03 GMT+04:00 Ilya Enkovich : > Hi, > > This patch introduces changes in call graph for Pointer Bounds Checker. > > New fields instrumented_version, instrumentation_clone and orig_decl are > added for cgraph_node: > - instrumentation_clone field is 1 for nodes created for instr

Re: [PATCH, Pointer Bounds Checker 7/x] Call/ret ifaces

2014-05-06 Thread Ilya Enkovich
2014-04-16 17:11 GMT+04:00 Ilya Enkovich : > Hi, > > This patch adds flags and ifaces to mark instrumented calls, extends return > stms with additional operand and introduces some basic bounds predicates. > These changes were previously reverted from 4.9 and I'll assume patch is OK > for trunk

Re: Hurd port for gcc go PATCH 7-9 (9)

2014-05-06 Thread Svante Signell
On Tue, 2014-05-06 at 11:07 +0200, Samuel Thibault wrote: > Svante Signell, le Tue 06 May 2014 10:58:38 +0200, a écrit : > > The patch for st_dev by Thomas Schwinge was not liked by Samuel > > Uh? > > I said “These should be fine, however.” and “a sed rule can't hurt even > if there is no occurre

Re: [PATCH, Pointer Bounds Checker 8/x] Add varpool node field

2014-05-06 Thread Ilya Enkovich
2014-04-16 17:18 GMT+04:00 Ilya Enkovich : > Hi, > > This patch add new field for varpool_node to mark vars requiring bounds > initalization. These changes were previously reverted from 4.9 and I'll > assume patch is OK for trunk if no objections arise. > > Patch was bootstrapped and tested for

Re: [PATCH, Pointer Bounds Checker 6/x] New static constructor types

2014-05-06 Thread Ilya Enkovich
2014-04-16 16:33 GMT+04:00 Ilya Enkovich : > Hi, > > This patch add new static constructor types used by Pointer Bounds Checker. > It was approved earlier for 4.9 and I'll assume patch is OK for trunk if no > objections arise. > > Patch was bootstrapped and tested for linux-x86_64. > > Thanks, >

Re: [PATCH, Pointer Bounds Checker 5/x] Attributes

2014-05-06 Thread Ilya Enkovich
Ping 2014-04-16 16:26 GMT+04:00 Ilya Enkovich : > Hi, > > This patch introduces attributes used by Pointer Bounds Checker. Comparing > to what was approved for 4.9, this one has additional attribute > 'bnd_instrument' to be used for selective instrumentation. > > Bootstrapped and tested on linu

Re: [PATCH, Pointer Bounds Checker 4/x] Built-in functions

2014-05-06 Thread Ilya Enkovich
PING 2014-04-16 16:19 GMT+04:00 Ilya Enkovich : > Hi, > > This patch introduces built-in functions used by Pointer Bounds Checker. It > is mostly similar to what was reverted from 4.9, I just added types and > attributes to builtins. This patch also introduces pointer_bounds_type_node > to be

Re: [PATCH, Pointer Bounds Checker 3/x] Target hooks for Pointer Bounds Checker

2014-05-06 Thread Ilya Enkovich
PING 2014-04-16 15:52 GMT+04:00 Ilya Enkovich : > Hi, > > This patch introduces target hooks to be used by Pointer Bounds Checker. > Hooks set is different from what was approved for 4.9 (and later reverted). > I added hooks to work with returned bounds and to prepare incoming bounds for > va

Re: [PATCH, i386, Pointer Bounds Checker 2/x] Intel Memory Protection Extensions (MPX) instructions support

2014-05-06 Thread Ilya Enkovich
2014-04-16 15:35 GMT+04:00 Ilya Enkovich : > Hi, > > This patch introduces Intel MPX bound registers and instructions. It was > approved earlier for 4.9 and had no significant changes since then. I'll > assume patch is OK if no objections arise. > > Patch was bootstrapped and tested for linux-x

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

2014-05-06 Thread Ilya Enkovich
2014-04-16 15:00 GMT+04:00 Ilya Enkovich : > Hi, > > This patch restarts the series for introducing Pointer Bounds Checker > instrumentation and supporting Intel Memory Protection Extension (MPX) > technology. Detailed description is on GCC Wiki page: > http://gcc.gnu.org/wiki/Intel%20MPX%20sup

Re: [PATCH, PR52252] Vectorization for load/store groups of size 3.

2014-05-06 Thread Richard Biener
On Tue, 6 May 2014, Evgeny Stupachenko wrote: > The patch on cost model was successfully committed. > I've separated the rest part of the patch on loads/stores group into > 2: on loads group and on stores group. > Below is first part on loads group. > > Bootstrap and make check passed on x86. >

Re: [PATCH] Fix PR39246: -Wuninitialized for partially initialized complex

2014-05-06 Thread Richard Biener
On Tue, May 6, 2014 at 11:56 AM, Thomas Preud'homme wrote: > Sorry, I forgot to include the new ChangeLogs: > > > *** gcc/ChangeLog *** > > 2014-05-06 Thomas Preud'homme > > PR middle-end/39246 > * tree-complex.c (expand_complex_move): Keep line info when expanding > com

Re: [PATCH, PR52252] Vectorization for load/store groups of size 3.

2014-05-06 Thread Evgeny Stupachenko
The patch on cost model was successfully committed. I've separated the rest part of the patch on loads/stores group into 2: on loads group and on stores group. Below is first part on loads group. Bootstrap and make check passed on x86. Is it ok? ChangeLog: 2014-05-06 Evgeny Stupachenko

RE: RE: [PATCH, MIPS] Alter default number of single-precision registers

2014-05-06 Thread Matthew Fortune
Hi Qiuji, I'll run the tests to double check. I haven't run spec with this patch nor do I have current results for spec to hand. Can I check that spec2000 runs OK for you without the patch for the same base version of GCC? Regards, Matthew From: qiuji-loongson [mailto:qi...@loongson.cn] Sen

Re: [PATCH][C++] Fix PR61004

2014-05-06 Thread Richard Biener
On Wed, 30 Apr 2014, Richard Biener wrote: > > With no longer recording alias subsets using BINFOs we now emit > bogus alias warnings for accessing empty bases. The following > avoids this, accessing those with an incompatible alias-set is > harmless. > > Bootstrapped and tested on x86_64-unkno

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

2014-05-06 Thread Richard Biener
On Tue, May 6, 2014 at 10:39 AM, Bin.Cheng wrote: > On Fri, Dec 6, 2013 at 6:19 PM, Richard Biener > wrote: >> On Mon, Nov 25, 2013 at 7:41 PM, Jeff Law wrote: >>> On 11/25/13 02:22, bin.cheng wrote: Hi, I previously committed two patches lowering complex address expression for >>

Re: [C PATCH] proposal to add new warning -Wsizeof-array-argument

2014-05-06 Thread Marek Polacek
On Sat, May 03, 2014 at 04:51:53PM +0530, Prathamesh Kulkarni wrote: > * c-tree.h (C_ARRAY_PARM): New macro, alias for DECL_LANG_FLAG_2. "New macro." would be enough. > --- gcc/c/c-decl.c(revision 210004) > +++ gcc/c/c-decl.c(working copy) > @@ -4650,6 +4650,8 @@ push_parm_decl (const s

RE: [PATCH] Fix PR39246: -Wuninitialized for partially initialized complex

2014-05-06 Thread Thomas Preud'homme
Sorry, I forgot to include the new ChangeLogs: *** gcc/ChangeLog *** 2014-05-06 Thomas Preud'homme PR middle-end/39246 * tree-complex.c (expand_complex_move): Keep line info when expanding complex move. * tree-ssa-uninit.c (uninit_undefined_value_p): For SSA_N

RE: [PATCH] Fix PR39246: -Wuninitialized for partially initialized complex

2014-05-06 Thread Thomas Preud'homme
Hi Richard, > From: Richard Biener [mailto:richard.guent...@gmail.com] > > > > The ChangeLog are as follows: > > > > *** gcc/ChangeLog *** > > > > 2014-05-04 Thomas Preud'homme > > > > PR middle-end/39246 > > * tree-complex.c (expand_complex_move): Keep line info when > expandin

Re: [PATCH 0/3] Compile-time gimple checking, without typedefs

2014-05-06 Thread Martin Jambor
On Mon, May 05, 2014 at 09:53:49PM +0200, Jakub Jelinek wrote: > On Mon, May 05, 2014 at 01:44:06PM -0600, Jeff Law wrote: > > On 05/05/14 11:37, Richard Biener wrote: > > > > > >Well, I hope that Andrew doesn't do without a namespace (and I still > > >don't believe in what he tries to achieve with

Re: [PATCH] Disable -ftree-loop-distribute-patterns for -ffreestanding, PR56888

2014-05-06 Thread Jakub Jelinek
On Tue, May 06, 2014 at 11:19:58AM +0200, Richard Biener wrote: > > While I'd still like to detect the obvious wrong cases like outlined > in comment #27 of the PR the following makes the seemingly obvious > workarounds work (use -fno-builtin, -ffreestanding or -fno-hosted). > > Ok for trunk and

[PATCH] Disable -ftree-loop-distribute-patterns for -ffreestanding, PR56888

2014-05-06 Thread Richard Biener
While I'd still like to detect the obvious wrong cases like outlined in comment #27 of the PR the following makes the seemingly obvious workarounds work (use -fno-builtin, -ffreestanding or -fno-hosted). Ok for trunk and branches? Thanks, Richard. 2014-05-06 Richard Biener c-family/

Re: Hurd port for gcc go PATCH 7-9 (9)

2014-05-06 Thread Samuel Thibault
Svante Signell, le Tue 06 May 2014 10:58:38 +0200, a écrit : > The patch for st_dev by Thomas Schwinge was not liked by Samuel Uh? I said “These should be fine, however.” and “a sed rule can't hurt even if there is no occurrence...” So just keep that precise part back as it was, no need for bein

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

2014-05-06 Thread Marek Polacek
On Tue, May 06, 2014 at 10:14:42AM +0200, Jakub Jelinek wrote: > > Hmm, but isn't TYPE_ALIGN_UNIT wrong then? > > No, after all, you don't want to change __alignof__ (long long), that is > pretty essential part of ABI. Yeah, as I understand things, for x86_64 __alignof__ (T) returns *preferred* a

Hurd port for gcc go PATCH 7-9 (9)

2014-05-06 Thread Svante Signell
(continued) patch7.diff: src/libgo/go/syscall/wait.c Set WCONTINUED to zero if not defined (same fix as for lto in gcc-4.9) patch8.diff: src/libgo/mksysinfo.sh Add special treatment of EWOULDBLOCK, SYS_FCNTL and st_dev since they are either not defined or defined differently for the script to ca

Hurd port for gcc go PATCH 4-6 (9)

2014-05-06 Thread Svante Signell
(continued) patch4.diff: src/libgo/go/syscall/libcall_posix-1.go: New file, a copy of libcall_posix.go with the mount, mlockall and munlockall calls removed. mount/umount functionality exists but is currently part of Hurd utilities, the outhe two functions are not yet implemented. patch5.diff: s

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

2014-05-06 Thread Bin.Cheng
On Fri, Dec 6, 2013 at 6:19 PM, Richard Biener wrote: > On Mon, Nov 25, 2013 at 7:41 PM, Jeff Law wrote: >> On 11/25/13 02:22, bin.cheng wrote: >>> >>> Hi, >>> I previously committed two patches lowering complex address expression for >>> IVOPT at http://gcc.gnu.org/ml/gcc-patches/2013-11/msg0054

Hurd port for gcc go PATCH 0-3 (9)

2014-05-06 Thread Svante Signell
Hi, Attached are patches to enable gccgo to build properly on Debian GNU/Hurd on gcc-4.9 (4.9-4.9.0-1). With split stack enabled 95 libgo tests PASS and 27 FAIL. Many of the failed tests do fail only in a few sub-tests. patch0.diff is debian specific, while the rest are for upstream. patch0.diff

Re: [RS6000] PR60737, expand_block_clear uses word stores

2014-05-06 Thread Alan Modra
On Fri, May 02, 2014 at 11:28:27AM -0400, David Edelsohn wrote: > On Fri, May 2, 2014 at 6:20 AM, Alan Modra wrote: > > In cases where the compiler has no alignment info, powerpc64le-linux > > gcc generates byte at a time copies for -mstrict-align (which is on > > for little-endian power7). That'

[PATCH] Fix PR61070

2014-05-06 Thread Richard Biener
Committed as obvious. Richard. 2014-05-06 Richard Biener PR middle-end/61070 * bitmap.c (debug_bitmap): Dump to stderr, not stdout. * tree-ssa-structalias.c (dump_solution_for_var): Likewise. Index: gcc/bitmap.c ===

Re: [C++ Patch] PR 60999

2014-05-06 Thread Paolo Carlini
Hi again, On 05/05/2014 10:48 PM, Jason Merrill wrote: On 05/05/2014 12:25 PM, Paolo Carlini wrote: Good, but is it Ok to use uses_template_parms for that? A few days ago I struggled to find something simpler ready to use, to no avail (well, this is probably well known to you, but there are sur

Re: [wide-int] Handle zero-precision INTEGER_CSTs again

2014-05-06 Thread Richard Biener
On Tue, May 6, 2014 at 10:11 AM, Richard Sandiford wrote: > Richard Biener writes: >> On Mon, May 5, 2014 at 10:58 PM, Richard Sandiford >> wrote: >>> Richard Biener writes: On Mon, May 5, 2014 at 4:51 PM, Richard Sandiford wrote: > Richard Biener writes: >> On Mon, May 5, 2

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

2014-05-06 Thread Jakub Jelinek
On Tue, May 06, 2014 at 10:04:33AM +0200, Richard Biener wrote: > On Mon, May 5, 2014 at 10:27 PM, Marek Polacek wrote: > > In this PR the issue is that we reject (valid) code such as > > _Alignas (long long) long long foo; > > with -m32, because we trip this condition: > > > >alignas_align =

Re: [wide-int] Handle zero-precision INTEGER_CSTs again

2014-05-06 Thread Richard Sandiford
Richard Biener writes: > On Mon, May 5, 2014 at 10:58 PM, Richard Sandiford > wrote: >> Richard Biener writes: >>> On Mon, May 5, 2014 at 4:51 PM, Richard Sandiford >>> wrote: Richard Biener writes: > On Mon, May 5, 2014 at 12:54 PM, Richard Sandiford > wrote: >> Richard Bien

Re: [wide-int] Handle zero-precision INTEGER_CSTs again

2014-05-06 Thread Richard Biener
On Mon, May 5, 2014 at 10:58 PM, Richard Sandiford wrote: > Richard Biener writes: >> On Mon, May 5, 2014 at 4:51 PM, Richard Sandiford >> wrote: >>> Richard Biener writes: On Mon, May 5, 2014 at 12:54 PM, Richard Sandiford wrote: > Richard Biener writes: >> On Fri, May 2, 2

Re: GCC's -fsplit-stack disturbing Mach's vm_allocate

2014-05-06 Thread Svante Signell
On Fri, 2014-05-02 at 12:52 +0200, Samuel Thibault wrote: > Svante Signell, le Fri 02 May 2014 12:45:56 +0200, a écrit : > > On Fri, 2014-05-02 at 12:00 +0200, Samuel Thibault wrote: > > > Samuel Thibault, le Fri 02 May 2014 11:57:53 +0200, a écrit : > > > > So we just need to fix guardsize in our

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

2014-05-06 Thread Richard Biener
On Mon, May 5, 2014 at 10:27 PM, Marek Polacek wrote: > In this PR the issue is that we reject (valid) code such as > _Alignas (long long) long long foo; > with -m32, because we trip this condition: > >alignas_align = 1U << declspecs->align_log; >if (alignas_align < TYPE_ALIGN_UNIT (type))

Re: [DOC PATCH] Rewrite docs for inline asm

2014-05-06 Thread Andrew Haley
On 05/05/2014 09:23 PM, Gerald Pfeifer wrote: > Understood. Let's see that we can get an update committed soon. > We can always improve on it further later on, which then will be > a lot easier to do, review, and get pushed. Yes. We already know that this is better than the current docs. Let's c

Re: Ping2: [PATCH] PR debug/16063. Add DW_AT_type to DW_TAG_enumeration.

2014-05-06 Thread Mark Wielaard
On Mon, 2014-04-28 at 14:37 +0200, Mark Wielaard wrote: > On Mon, 2014-04-28 at 14:23 +0200, Jakub Jelinek wrote: > > On Mon, Apr 28, 2014 at 01:17:32PM +0200, Mark Wielaard wrote: > > > Ping2. Please let me know if I should ping/cc other people to get this > > > reviewed. > > > > Do you want to a