Re: [PATCH 0/2] add unique_ptr class

2017-08-13 Thread Ville Voutilainen
>Good point. "gnutools" might be more accurate, but people might object >to adding 10 extra characters for "gnutools::". >Naming is important, especially for a whole namespace (not just a >single type) so I do think it's worth spending time getting it right. >But I could live with gtl as long as

[Updated, PATCH] i386: Avoid stack realignment if possible

2017-08-13 Thread H.J. Lu
On Mon, Aug 07, 2017 at 08:58:49AM -0700, H.J. Lu wrote: > On Tue, Jul 25, 2017 at 7:54 AM, Uros Bizjak wrote: > > On Tue, Jul 25, 2017 at 3:52 PM, H.J. Lu wrote: > >> On Fri, Jul 14, 2017 at 4:46 AM, H.J. Lu wrote: > >>> On Fri, Jul 7, 2017 at 5:56 PM, H.J. Lu wrote: > On Fri, Jul 07, 201

Re: [PATCH, rs6000] More diagnostic cleanup, addressing PR79845

2017-08-13 Thread Martin Sebor
--- gcc/config/rs6000/rs6000-c.c(revision 250965) +++ gcc/config/rs6000/rs6000-c.c(working copy) @@ -5812,7 +5812,7 @@ altivec_resolve_overloaded_builtin (location_t loc types. */ if (nargs != 2) { - error ("vec_mul only accepts 2 arguments"); +

Re: [PATCH] Add libgcc support for cache clearing on ARM VxWorks

2017-08-13 Thread Olivier Hainque
Hi Richard, > I'm not going to object to this patch - it's ok at a GCC level; but I'm > not sure that architecturally this is going to work. The implementation > of cache clearing is very specific to the implementation and I don't > think it is possible to write a portable single implementation.

Re: [PATCH] i386: Replace frame pointer with stack pointer in debug insns

2017-08-13 Thread Uros Bizjak
On Sun, Aug 13, 2017 at 7:35 PM, H.J. Lu wrote: > On Sun, Aug 13, 2017 at 9:15 AM, Uros Bizjak wrote: >> On Sat, Aug 12, 2017 at 3:32 AM, H.J. Lu wrote: >>> When we eliminate frame pointer, we should also replace frame pointer >>> with stack pointer - UNITS_PER_WORD in debug insns. This patch f

*ping* [patch, fortran] Fix PR 81116, missing dependency handling for allocatable character

2017-08-13 Thread Thomas Koenig
the attached patch fixes the PR by adding a dependency check for the case of concatenation operators. Regression-tested. OK for trunk? Ping? Regards Thomas

Re: [PATCH] i386: Replace frame pointer with stack pointer in debug insns

2017-08-13 Thread H.J. Lu
On Sun, Aug 13, 2017 at 9:15 AM, Uros Bizjak wrote: > On Sat, Aug 12, 2017 at 3:32 AM, H.J. Lu wrote: >> When we eliminate frame pointer, we should also replace frame pointer >> with stack pointer - UNITS_PER_WORD in debug insns. This patch fixed: >> >> FAIL: gcc.dg/guality/pr58791-5.c -Os li

Re: [PATCH] i386: Replace frame pointer with stack pointer in debug insns

2017-08-13 Thread Uros Bizjak
On Sat, Aug 12, 2017 at 3:32 AM, H.J. Lu wrote: > When we eliminate frame pointer, we should also replace frame pointer > with stack pointer - UNITS_PER_WORD in debug insns. This patch fixed: > > FAIL: gcc.dg/guality/pr58791-5.c -Os line pr58791-5.c:20 b1 == 9 > FAIL: gcc.dg/guality/pr58791-5.

[PATCH, i386]: Convert *load_tp_ and *add_tp_ to address space

2017-08-13 Thread Uros Bizjak
Hello! Attached patch converts *load_tp_ and *add_tp_ to a load from address 0 in DEFAULT_TLS_SEG_REG address space. The conversion is done as a split after combine pass, so we still combine addresses, before they are converted to a non-default address space. 2017-08-13 Uros Bizjak * conf

[PATCH] Write dependency information (-M*) even if there are errors

2017-08-13 Thread Boris Kolpackov
Hi, I've been instructed to resend this patch from the gcc mailing list: Currently GCC does not write extracted header dependency information if there are errors. However, this can be useful when dealing with outdated generated headers that trigger errors which would have been resolved if we coul