Re: [RFC PATCH, alpha]: ABI change: pass SFmode and SCmode variable arguments by reference

2016-09-04 Thread Uros Bizjak
On Fri, Sep 2, 2016 at 7:17 PM, Richard Henderson wrote: > On 09/02/2016 02:35 AM, Uros Bizjak wrote: >> --q-- >> /* Pass float and _Complex float variable arguments by reference. >> This avoids 64-bit store from a FP register to a pretend args save area >> and subsequent 32-bit load f

[PATCHv2] Add a warning for suspicious use of conditional expressions in boolean context

2016-09-04 Thread Bernd Edlinger
On 09/02/16 20:52, Bernd Edlinger wrote: > Hi! > > As reported in PR77434 and PR77421 there should be a warning for > suspicious uses of conditional expressions with non-boolean arguments. > > This warning triggers on conditional expressions in boolean context, > when both possible results are non-

Testing _Complex varargs passing [was: Alpha, ABI change: pass SFmode and SCmode varargs by reference]

2016-09-04 Thread Uros Bizjak
On Fri, Sep 2, 2016 at 2:11 PM, Jakub Jelinek wrote: > On Fri, Sep 02, 2016 at 12:09:30PM +, Joseph Myers wrote: >> On Fri, 2 Sep 2016, Uros Bizjak wrote: >> >> > argument. Passing _Complex float as a variable argument never >> > worked on alpha. Thus, we have no backward compatibi

Re: [PATCH, 2/4] Replace error_at with assert in build_va_arg

2016-09-04 Thread Tom de Vries
On 03/09/16 17:37, Andreas Schwab wrote: On Aug 24 2016, Tom de Vries wrote: > Replace error_at with assert in build_va_arg > > 2016-08-22 Tom de Vries > >* c-common.c (build_va_arg): Replace first argument type error >with assert. /daten/gcc/gcc-20160902/gcc/testsuite/c-c++-common

Re: [PATCH, c++, PR77427 ] Set TYPE_STRUCTURAL_EQUALITY for sysv_abi va_list

2016-09-04 Thread Tom de Vries
On 04/09/16 08:12, Richard Biener wrote: On September 3, 2016 5:23:35 PM GMT+02:00, Tom de Vries wrote: >Hi, > >this patch fixes a c++ ICE, a p1 6/7 regression. > > >Consider test.C: >... >void bar (__builtin_va_list &); > >struct c >{ > operator const __builtin_va_list &(); > operator __b

[PATCH, rtl-optimization]: Fix PR77452, ICE: in plus_constant, at explow.c

2016-09-04 Thread Uros Bizjak
Hello! As shown in the PR [1], combine is able to simplify lowpart CONST_VECTOR constant pool reference to its inner-mode reference. However, plus_constant was not able to extract the constant from narrowed access. Attached patch teaches plus_constant how to handle this situation. 2016-09-04 Ur

Re: [PATCH, c++, PR77427 ] Set TYPE_STRUCTURAL_EQUALITY for sysv_abi va_list

2016-09-04 Thread Richard Biener
On September 4, 2016 12:33:02 PM GMT+02:00, Tom de Vries wrote: >On 04/09/16 08:12, Richard Biener wrote: >> On September 3, 2016 5:23:35 PM GMT+02:00, Tom de Vries > wrote: >>> >Hi, >>> > >>> >this patch fixes a c++ ICE, a p1 6/7 regression. >>> > >>> > >>> >Consider test.C: >>> >... >>> >void b

Re: libgo patch committed: Change build procedure to use build tags

2016-09-04 Thread Matthias Klose
On 07.08.2016 17:18, Matthias Klose wrote: > On 07.08.2016 14:14, Andreas Schwab wrote: >> That breaks ia64: >> >> ../../../libgo/go/internal/syscall/unix/getrandom_linux.go:29:5: error: >> reference to undefined name 'randomTrap' >> if randomTrap == 0 { >> ^ >> ../../../libgo/go/internal/s

[committed] Update hpux_longjmp fix for hpux11.00

2016-09-04 Thread John David Anglin
The attached change updates the hpux_longjmp fix to handle the various longjmp declarations on hpux11.00. Tested with hppa2.0w-hp-hpux11.00 and hppa64-hp-hpux11.11 builds. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2016-09-04 John David Anglin * incl

Re: [PATCH 2/4][Ada,DJGPP] Ada support for DJGPP

2016-09-04 Thread Andris Pavenis
On 08/25/2016 12:17 PM, Arnaud Charlet wrote: -#if defined (__MINGW32__) +#if defined (__DJGPP__) + +/* FIXME: this is draft version only. Fix me if that is not correct */ +/*or not complete (AP) */ This FIXME needs to be addressed. DJGPP special i

Re: [PATCH 3/4][Ada,DJGPP] Ada support for DJGPP

2016-09-04 Thread Andris Pavenis
On 08/25/2016 12:46 PM, Arnaud Charlet wrote: Which exception handling mechanism is used by the DJGPP port? Is it (GCC) SJLJ? No. Updated to set ZCX_By_Default := true for DJGPP. New patch attached. Andris >From 45d1e01d7b29e88eea85a284733efdd7f38cc604 Mon Sep 17 00:00:00 2001 From: Andris Pa

Re: [PATCH 4/4][Ada,DJGPP] Ada support for DJGPP

2016-09-04 Thread Andris Pavenis
On 07/30/2016 08:47 AM, Andris Pavenis wrote: This last patch (4/4) contains DJGPP related changes to adaint.c (except one which belongs to patch 1/4). ChangeLog entry: 2016-07-30 Andris Pavenis * ada/adaint.c: Include process.h, signal.h, dir.h and utime.h for DJGPP. (DIR_SEPARATOR) defin

Re: [PATCH] Fix template-params-12f.C on darwin/vxworks (PR debug/77389)

2016-09-04 Thread Dominique d'Humières
The same should apply to g++.dg/debug/dwarf2/template-params-12g.C: --- ../_clean/gcc/testsuite/g++.dg/debug/dwarf2/template-params-12g.C 2016-08-12 09:59:34.0 +0200 +++ gcc/testsuite/g++.dg/debug/dwarf2/template-params-12g.C 2016-08-30 11:36:48.0 +0200 @@ -1,4 +1,4 @@ -//

[committed] libffi: Support go closures on 32-bit hppa

2016-09-04 Thread John David Anglin
The attached change adds libffi go closure support on 32-bit hppa. It also has some cleanups. We now use CFI directives for unwind support on linux. Go is a work in progress. Tested on hppa2.0w-hp-hpux11.11 and hppa-unknown-linux-gnu. Committed to gcc trunk. Dave -- John David Anglin d

Re: [PATCH 3/4][Ada,DJGPP] Ada support for DJGPP

2016-09-04 Thread Arnaud Charlet
> On 08/25/2016 12:46 PM, Arnaud Charlet wrote: > >Which exception handling mechanism is used by the DJGPP port? > >Is it (GCC) SJLJ? > No. Updated to set ZCX_By_Default := true for DJGPP. Well I'm confused now. How could this work before? In other words, has the target runtime been tested/used by

Re: [PATCH 4/4][Ada,DJGPP] Ada support for DJGPP

2016-09-04 Thread Arnaud Charlet
> >This last patch (4/4) contains DJGPP related changes to adaint.c > >(except one which belongs to patch 1/4). This patch is quite intrusive. Are all these changes really needed? > char > __gnat_get_default_identifier_character_set (void) > { > +#if defined (__DJGPP__) > + return 'p'; > +#el

Re: [PATCH 2/4][Ada,DJGPP] Ada support for DJGPP

2016-09-04 Thread Arnaud Charlet
This patch is OK > From 83fe70a17d811ebdec7ca70509e3c2521657d8f2 Mon Sep 17 00:00:00 2001 > From: Andris Pavenis > Date: Sun, 28 Aug 2016 08:02:11 +0300 > Subject: [PATCH 2/4] [DJGPP, Ada] DJGPP support > > * ada/ctrl_c.c: Do not use macro SA_RESTART for DJGPP. > > * ada/gsocket.h: Do not suppo

Re: *ping* Re: [PATCH, wwwdocs] Fix PR 50642

2016-09-04 Thread Gerald Pfeifer
On Wed, 22 May 2013, Shakthi Kannan wrote: > Is the following patch okay for trunk? Sorry for the enormous delay in getting to this. Not sure why that fell through the cracks. :-( The idea is a fine one, and would have worked back then. Since the end of last year or so, the web server at gcc.

Re: *ping* Re: [PATCH, wwwdocs] Fix PR 50642

2016-09-04 Thread Gerald Pfeifer
On Sun, 4 Sep 2016, Gerald Pfeifer wrote: > With the patch below I follow a slightly different approach, > referring to the global CSS we already have in place now instead. > That also allows tweaking this more dynamically. > > I'll be looking into the actual style change next. And here we go;

[patch v2] Get rid of stack trampolines for nested functions (0/4)

2016-09-04 Thread Eric Botcazou
Hi, this is the updated version of the patch initially posted at: https://gcc.gnu.org/ml/gcc-patches/2016-06/msg02016.html It takes into account Jeff's remarks, both on the code and the documentation. As discussed, I'm going to split it into 4 parts: common infrastructure, Ada front-end bits,

[patch v2] Get rid of stack trampolines for nested functions (1/4)

2016-09-04 Thread Eric Botcazou
This is the common infrastructure part. As explained in the initial message, nothing is activated unless both the language and the target opt in. 2016-07-04 Eric Botcazou PR ada/37139 PR ada/67205 * common.opt (-ftrampolines): New option. * doc/invoke.texi (C

[patch v2] Get rid of stack trampolines for nested functions (2/4)

2016-09-04 Thread Eric Botcazou
These are the Ada front-end bits. They define the langhook to true and set the flags on individual ADDR_EXPR and CALL_EXPR nodes with Ada convention; in other words, descriptors are not activated for subprograms imported from or exported to other languages. 2016-07-04 Eric Botcazou

[patch v2] Get rid of stack trampolines for nested functions (3/4)

2016-09-04 Thread Eric Botcazou
These are the individual back-end changes. Only the architectures for which native platforms are available are changed for now. The changes were tested at AdaCore over the years for every architecture and I'll retest them if they are accepted, except for those I cannot access any more (Alpha,

[patch v2] Get rid of stack trampolines for nested functions (4/4)

2016-09-04 Thread Eric Botcazou
This is the couple of testcases for the gnat.dg testsuite. They test for the presence of the GNU-stack executable marker in the assembly file on Linux. 2016-07-04 Eric Botcazou * gnat.dg/trampoline3.adb: New test. * gnat.dg/trampoline4.adb: Likewise. -- Eric BotcazouIndex:

Re: [PATCH, c++, PR77427 ] Set TYPE_STRUCTURAL_EQUALITY for sysv_abi va_list

2016-09-04 Thread Tom de Vries
On 04/09/16 16:08, Richard Biener wrote: On September 4, 2016 12:33:02 PM GMT+02:00, Tom de Vries wrote: On 04/09/16 08:12, Richard Biener wrote: On September 3, 2016 5:23:35 PM GMT+02:00, Tom de Vries wrote: Hi, this patch fixes a c++ ICE, a p1 6/7 regression. Consider test.C: ... void

Re: [PATCH 3/4][Ada,DJGPP] Ada support for DJGPP

2016-09-04 Thread Andris Pavenis
On 09/04/2016 09:50 PM, Arnaud Charlet wrote: On 08/25/2016 12:46 PM, Arnaud Charlet wrote: Which exception handling mechanism is used by the DJGPP port? Is it (GCC) SJLJ? No. Updated to set ZCX_By_Default := true for DJGPP. Well I'm confused now. How could this work before? In other words, ha

[RFC][SSA] Iterator to visit SSA

2016-09-04 Thread Kugan Vivekanandarajah
Hi All, While looking at gcc source, I noticed that we are iterating over SSA variable from 0 to num_ssa_names in some cases and 1 to num_ssa_names in others. It seems that variable 0 is always NULL TREE. But it can confuse people who are looking for the first time. Therefore It might be good to f

Re: [PATCH 3/4][Ada,DJGPP] Ada support for DJGPP

2016-09-04 Thread Arnaud Charlet
> GCC Ada compiler itself uses exceptions. Yes, but the bootstrap doesn't use system-dgjpp.ads, it uses the generic system.ads which uses front-end (GNAT) SJLJ, hence my question. Your answer isn't very reassuring. Arno

Re: [PATCH 3/4][Ada,DJGPP] Ada support for DJGPP

2016-09-04 Thread Eric Botcazou
> Attached output is from last test build (r239639 with DJGPP related patches > applied, last version of patches for Ada). Very strange error, line 28 of gtype-ada.h is supposed to have a guard for nodes containing the 'common' structure. Can you post an excerpt of the file? -- Eric Botcazou