[PATCH] Support cfi pseudo for debugging on UI_DWARF2 target

2017-07-28 Thread Tristan Gingold
Hi, this patch fixes a thinko in dwarf2out_do_cfi_asm: some targets (x86_64-windows) so support cfi psuedo for debug info, but not dwarf2 exception cfi. No regressions on x86_64-linux-gnu. Ok to commit ? 2017-07-28 gingold * dwarf2out.c

[PATCH] Add -nolibc option

2017-07-27 Thread Tristan Gingold
Hello, this patch adds a new option -nolibc to supress -lc in the link command. This refines -nostdlib/-nostartfiles/nodefaultlibs, so that it is possible to link with libgcc but without libc. Our main use case is for embedded targets when we use the GNAT compiler without an installed libc.

Re: Changing base compiler requirement for bootstrapping GNAT

2017-01-23 Thread Tristan Gingold
> On 19 Jan 2017, at 13:33, Jakub Jelinek <ja...@redhat.com> wrote: > > On Thu, Jan 19, 2017 at 01:26:32PM +0100, Andreas Schwab wrote: >> On Jan 19 2017, Tristan Gingold <ging...@adacore.com> wrote: >> >>> Is it ok to require gcc 4.9 (3 years old) or l

Re: Changing base compiler requirement for bootstrapping GNAT

2017-01-19 Thread Tristan Gingold
> On 19 Jan 2017, at 12:31, Richard Biener <richard.guent...@gmail.com> wrote: > > On Thu, Jan 19, 2017 at 12:04 PM, Tristan Gingold <ging...@adacore.com> wrote: >> >>> On 19 Jan 2017, at 11:46, Richard Biener <richard.guent...@gmail.com> wrote:

Re: Changing base compiler requirement for bootstrapping GNAT

2017-01-19 Thread Tristan Gingold
> On 19 Jan 2017, at 13:33, Jakub Jelinek <ja...@redhat.com> wrote: > > On Thu, Jan 19, 2017 at 01:26:32PM +0100, Andreas Schwab wrote: >> On Jan 19 2017, Tristan Gingold <ging...@adacore.com> wrote: >> >>> Is it ok to require gcc 4.9 (3 years old) or l

Re: Changing base compiler requirement for bootstrapping GNAT

2017-01-19 Thread Tristan Gingold
> On 19 Jan 2017, at 11:46, Richard Biener <richard.guent...@gmail.com> wrote: > > On Thu, Jan 19, 2017 at 11:11 AM, Tristan Gingold <ging...@adacore.com> wrote: >> Is it ok to require gcc 4.9 (3 years old) or later to build GNAT ? >> >> We plan to use g

Changing base compiler requirement for bootstrapping GNAT

2017-01-19 Thread Tristan Gingold
Is it ok to require gcc 4.9 (3 years old) or later to build GNAT ? We plan to use gcc exceptions within the GNAT front-end (previously we were using a FE specific exception mechanism). This requires a matching implementation in the runtime, which was last changed for gcc 4.9 Our idea is to

Re: [Ada] Set Always_Compatible_Rep to False everywhere

2016-10-17 Thread Tristan Gingold
> On 16 Oct 2016, at 21:40, Eric Botcazou wrote: > >> Can this be refactored to avoid having to duplicate the whole file for >> every target? > > I'm not sure, maybe entries could be omitted if they match the default though. Some files could be easily merged (like what

[Patch] sync top level configure with binutils-gdb

2015-11-20 Thread Tristan Gingold
This patch was pushed on binutils-gdb repo, so I also commit it on gcc. Tristan. 2015-11-20 Tristan Gingold <ging...@adacore.com> Sync with binutils-gdb: 2015-11-20 Tristan Gingold <ging...@adacore.com> * configure.ac: Add aarch64-*-darwin* and

Re: [Patch]: libbacktrace - add support of PE/COFF

2015-05-29 Thread Tristan Gingold
On 28 May 2015, at 17:14, Ian Lance Taylor i...@google.com wrote: On Thu, May 28, 2015 at 5:01 AM, Tristan Gingold ging...@adacore.com wrote: On 28 May 2015, at 02:26, Ian Lance Taylor i...@google.com wrote: The #include windows.h will break cross-compilers. It's not OK for trunk

Re: [Patch V2]: libbacktrace - add support of PE/COFF

2015-05-28 Thread Tristan Gingold
never happen). + /* Allocate memory for symbols are strings. */ Comment looks wrong--omit are”? Yes. Here is the new version of the patch. Regards, Tristan. libbacktrace/ 2015-05-21 Tristan Gingold ging...@adacore.com * pecoff.c: New file. * Makefile.am (FORMAT_FILES

Re: [Patch]: libbacktrace - add support of PE/COFF

2015-05-28 Thread Tristan Gingold
On 27 May 2015, at 15:36, Jeff Law l...@redhat.com wrote: +static int +coff_is_symbol (const b_coff_internal_symbol *isym) +{ + return isym-type == 0x20 isym-sec 0; +} You probably want const or enum so that you can have a symbolic name rather than 0x20 here. It also seems like the

Re: [Patch]: libbacktrace - add support of PE/COFF

2015-05-28 Thread Tristan Gingold
On 28 May 2015, at 02:26, Ian Lance Taylor i...@google.com wrote: The #include windows.h will break cross-compilers. It's not OK for trunk until that is fixed. I am confused by this comment, for two reasons: - I don’t see how that would break cross-compilers. Cross compilers hosted on

[Patch]: libbacktrace - add support of PE/COFF

2015-05-21 Thread Tristan Gingold
that feature on PE. The new file, pecoff.c, mostly follows the structure of elf.c Tested on both windows and windows64. No regression on Gnu/Linux x86. Tristan. 2015-05-21 Tristan Gingold ging...@adacore.com * pecoff.c: New file. * Makefile.am (FORMAT_FILES): Add pecoff.c

Re: [rs6000] Fix compare debug failure on AIX

2015-05-04 Thread Tristan Gingold
On 04 May 2015, at 02:32, David Edelsohn dje@gmail.com wrote: On Sat, May 2, 2015 at 6:04 AM, Eric Botcazou ebotca...@adacore.com wrote: Why should GCC unnecessarily create stack frames to avoid compare-debug testcase failures? I'm not sure I understand the question... compare-debug

Re: [Ada] Fix bootstrapping on darwin9/10 (PR ada/64349)

2015-02-04 Thread Tristan Gingold
#elif ! (defined (__vxworks)) ^^ __vxworks will not be defined by anything other than a vxworks compiler, I'd assume (it is certainly not defined by Darwin toolchains) extern char **environ; return environ; vvv so I don't see how this case will ever be exercised. #elif

Re: ping: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-01-21 Thread Tristan Gingold
On 21 Jan 2015, at 08:47, Joel Brobecker brobec...@adacore.com wrote: Thank you! :) What is the rational for having --with-zlib but --with-libz-prefix (ie zlib vs libz) ? Looks not very consistent. Tristan. On Wed, Jan 07, 2015 at 06:45:48PM +0400, Joel Brobecker wrote: Hello, This

Re: [Ada] Fix bootstrapping on darwin9/10 (PR ada/64349).

2015-01-14 Thread Tristan Gingold
On 09 Jan 2015, at 00:42, Iain Sandoe i...@codesourcery.com wrote: On 8 Jan 2015, at 13:52, Tristan Gingold wrote: On 08 Jan 2015, at 13:49, Iain Sandoe i...@codesourcery.com wrote: Hi Tristan, On 7 Jan 2015, at 10:15, Arnaud Charlet wrote: Use _NSGetEnviron to get environment

Re: [Ada] Fix bootstrapping on darwin9/10 (PR ada/64349).

2015-01-08 Thread Tristan Gingold
On 08 Jan 2015, at 13:49, Iain Sandoe i...@codesourcery.com wrote: Hi Tristan, On 7 Jan 2015, at 10:15, Arnaud Charlet wrote: Use _NSGetEnviron to get environment. Tested on x86_64-pc-linux-gnu, committed on trunk 2015-01-07 Tristan Gingold ging...@adacore.com PR ada

Re: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-01-07 Thread Tristan Gingold
On 07 Jan 2015, at 15:45, Joel Brobecker brobec...@adacore.com wrote: Hello, This patch enhances config/zlib.m4 to introduce an extra option --with-libz-prefix which allows us to provide the location of the zlib library we want to use during the build. I prefer the gcc way to provide

Re: Document __builtin_*_overflow

2014-11-12 Thread Tristan Gingold
On 12 Nov 2014, at 14:13, Jakub Jelinek ja...@redhat.com wrote: + size_t sz; + if (__builtin_mul_overflow (x, y, amp;sz) Missing right parenthesis ?

[committed]: change my email address

2014-10-29 Thread Tristan Gingold
I will commit the following change in MAINTAINERS. Tristan. 2014-10-29 Tristan Gingold ging...@adacore.com * MAINTAINERS: Change my email address. --- MAINTAINERS (revision 216822) +++ MAINTAINERS (working copy) @@ -136,7 +136,7 @@ RTEMS PortsJoel Sherrill j

Re: Trouble trying to test GCC on a simulator

2014-09-08 Thread Tristan Gingold
On 08 Sep 2014, at 12:01, Pierre-Marie de Rodat dero...@adacore.com wrote: Hello, I would like to test a GCC patch on the platform it is supposed to affect (PowerPC). In order to to this, I'm using the following documentation: https://gcc.gnu.org/simtest-howto.html However, after

Re: Trouble trying to test GCC on a simulator

2014-09-08 Thread Tristan Gingold
On 08 Sep 2014, at 16:51, Pierre-Marie de Rodat dero...@adacore.com wrote: On 09/08/2014 12:26 PM, Tristan Gingold wrote: CVS ? Binutils (and gdb) are now using git: Huh! I knew this, but not being a daily CVS user, I though the commands in the documentation would get a binutils version

Re: obsolete targets

2014-07-10 Thread Tristan Gingold
On 10 Jul 2014, at 04:23, Trevor Saunders tsaund...@mozilla.com wrote: Hi, I've noticed that the following targets are built in config-list.mk with --enable-obsolete i686-interix3 - doesn't appear to actually require --enable-obsolete though, should it be marked as obsolete in config.gcc?

Re: [RFC] Add aarch64 support for ada

2014-04-17 Thread Tristan Gingold
On 16 Apr 2014, at 17:36, Richard Henderson r...@redhat.com wrote: On 04/16/2014 12:39 AM, Eric Botcazou wrote: The primary bit of rfc here is the hunk that applies to ada/types.h with respect to Fat_Pointer. Given that the Ada type, as defined in s-stratt.ads, does not include alignment, I

Re: [RFC] Add aarch64 support for ada

2014-04-17 Thread Tristan Gingold
On 17 Apr 2014, at 16:50, Richard Henderson r...@redhat.com wrote: On 04/17/2014 02:00 AM, Tristan Gingold wrote: On 16 Apr 2014, at 17:36, Richard Henderson r...@redhat.com wrote: On 04/16/2014 12:39 AM, Eric Botcazou wrote: The primary bit of rfc here is the hunk that applies to ada

arm ttype encoding

2013-12-13 Thread Tristan Gingold
Hi, we are currently working on the use of the arm ehabi for Ada exceptions, and we aren't sure about which encoding has to be used for ttype. The patch http://gcc.gnu.org/ml/gcc-patches/2011-09/msg00765.html explains that on `Older ARM EABI toolchains set this value [ttype_encoding]

Re: [buildrobot] alpha64-dec-vms / alpha-dec-vms

2013-11-26 Thread Tristan Gingold
On 26 Nov 2013, at 04:23, Jan-Benedict Glaw jbg...@lug-owl.de wrote: Hi! Build log is available at http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=36942 http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=40027 Yes, we are aware of that. Basically, the openvms

Re: [PATCH]: Fix use of __builtin_eh_pointer in EH_ELSE

2013-09-30 Thread Tristan Gingold
On Sep 24, 2013, at 8:51 PM, Richard Henderson r...@redhat.com wrote: On 09/03/2013 07:08 AM, Tristan Gingold wrote: Hi, The field state-ehp_region wasn't updated before lowering constructs in the eh path of EH_ELSE. As a consequence, __builtin_eh_pointer is lowered to 0 (or possibly

Ping: [PATCH]: Fix use of __builtin_eh_pointer in EH_ELSE

2013-09-16 Thread Tristan Gingold
Any comment/review on this patch ? On Sep 3, 2013, at 4:08 PM, Tristan Gingold ging...@adacore.com wrote: Hi, The field state-ehp_region wasn't updated before lowering constructs in the eh path of EH_ELSE. As a consequence, __builtin_eh_pointer is lowered to 0 (or possibly to a wrong

[PATCH]: Fix use of __builtin_eh_pointer in EH_ELSE

2013-09-03 Thread Tristan Gingold
for it in tree.c. Why is 'builtin_decl_explicit_p (BUILT_IN_TM_LOAD_1)' needed in addition to flag_tm ?) No regressions (check-host) on x86-64 GNU/Linux. Ok for trunk ? Tristan. 2013-09-03 Tristan Gingold ging...@adacore.com * tree.c (set_call_expr_flags): Reject ECF_TM_PURE

Re: [RFC / musing] Scoped exception handling in Linux userspace?

2013-07-18 Thread Tristan Gingold
On Jul 19, 2013, at 2:26 AM, Andy Lutomirski wrote: Windows has a feature that I've wanted on Linux forever: stack-based (i.e. scoped) exception handling. The upshot is that you can do, roughly, this (pseudocode): [...] Indeed Windows and OpenVMS have such a mechanism. That's clean and

[VMS] Update libiberty/makefile.vms

2013-07-09 Thread Tristan Gingold
Add dwarfnames.obj, needed to build binutils on VMS hosts. Committed on both gcc and src trunks. Tristan. libiberty/ 2013-07-09 Tristan Gingold ging...@adacore.com * makefile.vms (OBJS): Add dwarfnames.obj Index: makefile.vms

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Tristan Gingold
On Feb 14, 2013, at 12:26 PM, Steven Bosscher wrote: On Wed, Feb 13, 2013 at 10:33 PM, David Edelsohn wrote: The AIX system supports DWARF debugging, but GCC does not generate it on AIX and GDB does not consume it on AIX. Is there a description for what has to be done in GCC to enable

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Tristan Gingold
On Feb 14, 2013, at 3:47 PM, Steven Bosscher wrote: On Thu, Feb 14, 2013 at 2:21 PM, Tristan Gingold wrote: Is there a description for what has to be done in GCC to enable DWARF with AIX as/ld? E.g. is it required to support the .dwsect pseudo? Is there already a plan from someone to make

Re: libbacktrace and darwin

2012-10-05 Thread Tristan Gingold
On Oct 4, 2012, at 11:23 PM, Ian Lance Taylor wrote: On Thu, Oct 4, 2012 at 1:32 PM, Jack Howarth howa...@bromo.med.uc.edu wrote: Is libbacktrace currently functional in gcc trunk and is it expected to function on darwin? While I could understand it not working on installed binaries of FSF

Re: libbacktrace and darwin

2012-10-05 Thread Tristan Gingold
On Oct 5, 2012, at 11:37 AM, Richard Guenther wrote: On Fri, Oct 5, 2012 at 9:15 AM, Tristan Gingold ging...@adacore.com wrote: On Oct 4, 2012, at 11:23 PM, Ian Lance Taylor wrote: On Thu, Oct 4, 2012 at 1:32 PM, Jack Howarth howa...@bromo.med.uc.edu wrote: Is libbacktrace currently

Re: [Patch/RFC] SEH exceptions for Win64

2012-07-25 Thread Tristan Gingold
Gerald, it is ok ? Tristan. On Jul 19, 2012, at 12:15 PM, Tristan Gingold wrote: On Jul 19, 2012, at 11:37 AM, Pedro Alves wrote: On 07/19/2012 08:30 AM, Tristan Gingold wrote: No regression on i386 GNU/Linux. Committed. Nice. Is this NEWS / docs --or wherever release notes

Re: [Patch/RFC] SEH exceptions for Win64

2012-07-20 Thread Tristan Gingold
On Jul 19, 2012, at 6:54 PM, Eric Botcazou wrote: I did a manual build and testing for x86_64-pc-mingw32 and I will run a full bootstrap and regression run on GNU/Linux. Tristan. libstdc++-v3/ * libsupc++/eh_personality.cc (__gxx_personality_seh0): New function. Adjust for

Re: [Patch/RFC] SEH exceptions for Win64

2012-07-19 Thread Tristan Gingold
On Jul 17, 2012, at 2:53 PM, Kai Tietz wrote: 2012/7/17 Richard Henderson r...@redhat.com: On 07/17/2012 12:35 AM, Tristan Gingold wrote: So, the first element of ExceptionInformation will be exc. Should I add a comment ? Ah right. Definitely. Otherwise I don't see anything else

Re: [Patch/RFC] SEH exceptions for Win64

2012-07-19 Thread Tristan Gingold
On Jul 19, 2012, at 11:37 AM, Pedro Alves wrote: On 07/19/2012 08:30 AM, Tristan Gingold wrote: No regression on i386 GNU/Linux. Committed. Nice. Is this NEWS / docs --or wherever release notes are written-- worthy? Something like this, borrowed from your own words: On Win64

Re: [Patch/RFC] SEH exceptions for Win64

2012-07-17 Thread Tristan Gingold
On Jul 16, 2012, at 6:28 PM, Richard Henderson wrote: On 07/13/2012 07:13 AM, Tristan Gingold wrote: +_Unwind_RaiseException (struct _Unwind_Exception *exc) +{ + memset (exc-private_, 0, sizeof (exc-private_)); + + RaiseException (STATUS_GCC_THROW, 0, 1, (ULONG_PTR *)exc); We almost

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Tristan Gingold
; Tested on x86_64-pc-linux-gnu, committed on trunk 2012-07-16 Tristan Gingold ging...@adacore.com * gcc-interface/decl.c (intrin_return_compatible_p): Map Address to void *.

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Tristan Gingold
On Jul 16, 2012, at 3:24 PM, Duncan Sands wrote: Hi Tristan, On 16/07/12 15:17, Tristan Gingold wrote: On Jul 16, 2012, at 3:16 PM, Duncan Sands wrote: Hi Arnaud, The natural way to import a builtin that returns void * is to use System.Address in Ada, which is in fact an integral

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Tristan Gingold
On Jul 16, 2012, at 3:58 PM, Duncan Sands wrote: Hi Tristan, indeed, for two years already. Is there any reason not to do this for all functions, rather than just limiting it to builtins? I don't understand what do you mean. We need to do this implicit conversion for builtins because

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Tristan Gingold
On Jul 16, 2012, at 4:36 PM, Duncan Sands wrote: Hi Tristan, Ah, what you want is the use of 'void *' for System.Address. We didn't choose that because the semantic of System.Address (which includes arithmetic on the whole address space) doesn't match the void * one. void* arithmetic

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Tristan Gingold
On Jul 16, 2012, at 5:16 PM, Duncan Sands wrote: PS: That said, I have to admit that using void* for builtins does cover the most important cases. Agreed. But I still think that these are two very different issues: * Doing implicit conversion to void * for builtins is just a convenient way

[Patch/RFC] SEH exceptions for Win64

2012-07-13 Thread Tristan Gingold
Hi, this is a rebase of RTH's patch posted a few years ago. It is almost unchanged, except that there is no SEH specific unwind.h header (there are 3 #if/#endif part in unwind-generic.h) and a minor cleanup in unwind-seh.c (indentation, unused variables). This patch allows to propagate GCC

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-07-05 Thread Tristan Gingold
On Jul 4, 2012, at 8:31 PM, Magnus Fromreide wrote: Hello. Ever since the ISL patch went in my builds have failed. Which ISL patch ? I am building with local copies of all the libraries, so I have added gmp, mpfr, mpc, isl and cloog from ftp://gcc.gnu.org/pub/gcc/infrastructure to my

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-07-04 Thread Tristan Gingold
On Jul 2, 2012, at 1:43 PM, Richard Guenther wrote: On Thu, 28 Jun 2012, Diego Novillo wrote: On 12-06-27 11:06 , Richard Guenther wrote: 2012-06-27 Richard Guenther rguent...@suse.de Michael Matz m...@suse.de Tobias Grosser tob...@grosser.es Sebastian Pop

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-07-04 Thread Tristan Gingold
On Jul 4, 2012, at 10:58 AM, Richard Guenther wrote: On Wed, 4 Jul 2012, Tristan Gingold wrote: On Jul 2, 2012, at 1:43 PM, Richard Guenther wrote: On Thu, 28 Jun 2012, Diego Novillo wrote: On 12-06-27 11:06 , Richard Guenther wrote: 2012-06-27 Richard Guenther rguent...@suse.de

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-07-04 Thread Tristan Gingold
On Jul 4, 2012, at 12:32 PM, Richard Guenther wrote: On Wed, 4 Jul 2012, Tristan Gingold wrote: On Jul 4, 2012, at 10:58 AM, Richard Guenther wrote: On Wed, 4 Jul 2012, Tristan Gingold wrote: On Jul 2, 2012, at 1:43 PM, Richard Guenther wrote: On Thu, 28 Jun 2012, Diego Novillo

Re: [Patch-v2] Adjustments for Windows x64 SEH

2012-06-25 Thread Tristan Gingold
On Jun 22, 2012, at 5:04 PM, Richard Henderson wrote: On 06/21/2012 12:48 AM, Tristan Gingold wrote: 2012-06-18 Tristan Gingold ging...@adacore.com * config/i386/winnt.c (i386_pe_seh_end_prologue): Move code to ... (seh_cfa_adjust_cfa): ... that function

Re: [Patch] Adjustments for Windows x64 SEH

2012-06-25 Thread Tristan Gingold
On Jun 18, 2012, at 4:28 PM, Kai Tietz wrote: Hello Tristan, patch works for me, too. Just one nit about the patch. 2012/6/18 Tristan Gingold ging...@adacore.com: @@ -8558,6 +8558,11 @@ ix86_frame_pointer_required (void) if (TARGET_32BIT_MS_ABI cfun-calls_setjmp) return true

Re: [patch] Only define JMP_BUF_SIZE in backends that also define DONT_USE_BUILTIN_SETJMP

2012-06-25 Thread Tristan Gingold
On Jun 25, 2012, at 10:26 AM, Steven Bosscher wrote: On Mon, Jun 25, 2012 at 9:13 AM, Eric Botcazou ebotca...@libertysurf.fr wrote: The PA and SPARC back ends do not define DONT_USE_BUILTIN_SETJMP, so they also do not have to define JMP_BUF_SIZE. So: * config/sparc/sparc.h

Re: [Patch-v2] Adjustments for Windows x64 SEH

2012-06-22 Thread Tristan Gingold
On Jun 21, 2012, at 8:19 PM, Richard Henderson wrote: On 2012-06-21 00:48, Tristan Gingold wrote: @@ -9142,9 +9152,12 @@ ix86_compute_frame_layout (struct ix86_frame *frame) { HOST_WIDE_INT diff; - /* If we can leave the frame pointer where it is, do so

Re: [Patch-v2] Adjustments for Windows x64 SEH

2012-06-21 Thread Tristan Gingold
On Jun 21, 2012, at 1:57 PM, Václav Zeman wrote: On 21 June 2012 09:48, Tristan Gingold wrote: Here is the new version. It is now possible to use __builtin_frame_address (0) to get the current establisher frame thanks to a tiny adjustment. No regressions on x86_64-linux, and an x86_64

Re: [Patch] Adjustments for Windows x64 SEH

2012-06-20 Thread Tristan Gingold
On Jun 19, 2012, at 6:47 PM, Richard Henderson wrote: On 2012-06-18 05:22, Tristan Gingold wrote: + /* Win64 SEH, very large frames need a frame-pointer as maximum stack + allocation is 4GB (add a safety guard for saved registers). */ + if (TARGET_64BIT_MS_ABI get_frame_size

Re: [Patch] Adjustments for Windows x64 SEH

2012-06-19 Thread Tristan Gingold
On Jun 18, 2012, at 4:28 PM, Kai Tietz wrote: Hello Tristan, patch works for me, too. Just one nit about the patch. 2012/6/18 Tristan Gingold ging...@adacore.com: @@ -8558,6 +8558,11 @@ ix86_frame_pointer_required (void) if (TARGET_32BIT_MS_ABI cfun-calls_setjmp) return true

Re: [patch] Remove VMS_TARGET

2012-06-18 Thread Tristan Gingold
On Jun 18, 2012, at 1:01 PM, Steven Bosscher wrote: Hello, There are exactly 2 references to the target macro VMS_TARGET: cp/decl.c:#if VMS_TARGET config/vax/vax.h:#define VMS_TARGET 0 I'm not sure what the macro is for. GCC seems to support a couple of VMS targets, but none of them

[Patch] Adjustments for Windows x64 SEH

2012-06-18 Thread Tristan Gingold
gdb which uses setjmp/longjmp internally for its exception mechanism. Bootstrapped and regtested on x86_64 GNU/Linux. Comments are welcome. Tristan. 2012-06-18 Tristan Gingold ging...@adacore.com * doc/extend.texi (X86 Built-in Functions): Document

Re: long long availability in host compiler (Re: constant that doesn't fit in 32bits in alpha.c)

2012-06-14 Thread Tristan Gingold
On Jun 14, 2012, at 11:12 AM, Pedro Alves wrote: On 06/13/2012 10:35 PM, Richard Henderson wrote: On 2012-06-13 02:13, Pedro Alves wrote: Related, does gcc forbid long long / ULL ? Normally, yes. The vmsdbgout.c file seems to use it all over though. And git blame shows:

Re: [Ada] Preliminary work to support SEH on Windows 64

2012-06-12 Thread Tristan Gingold
On Jun 12, 2012, at 5:00 PM, Gabriel Dos Reis wrote: On Tue, Jun 12, 2012 at 7:33 AM, Arnaud Charlet char...@adacore.com wrote: No functional change. Tested on x86_64-pc-linux-gnu, committed on trunk 2012-06-12 Tristan Gingold ging...@adacore.com * a-exexpr-gcc.adb

Re: regression due to r187199 explow.c? in target ia64-hp-hpux11.23.

2012-06-07 Thread Tristan Gingold
) #35 0xf564740:0 in main (argc=49, argv=0x7fffeee8) Regards, Kannan -Original Message- From: Mailaripillai, Kannan Jeganathan Sent: Wednesday, June 06, 2012 4:42 PM To: 'Tristan Gingold' Cc: gcc@gcc.gnu.org Subject: RE: regression due to r187199 explow.c? in target ia64-hp

Re: regression due to r187199 explow.c? in target ia64-hp-hpux11.23.

2012-06-06 Thread Tristan Gingold
, GCC configuration, etc in the original post: http://gcc.gnu.org/ml/gcc/2012-05/msg00371.html Regards, Kannan -Original Message- From: Mailaripillai, Kannan Jeganathan Sent: Wednesday, May 30, 2012 2:44 PM To: 'Tristan Gingold' Cc: gcc@gcc.gnu.org Subject: RE: regression due

Re: regression due to r187199 explow.c? in target ia64-hp-hpux11.23.

2012-06-06 Thread Tristan Gingold
Message- From: Tristan Gingold [mailto:ging...@adacore.com] Sent: Wednesday, June 06, 2012 1:17 PM To: Mailaripillai, Kannan Jeganathan Cc: gcc@gcc.gnu.org Subject: Re: regression due to r187199 explow.c? in target ia64-hp-hpux11.23. On Jun 6, 2012, at 9:10 AM, Mailaripillai, Kannan

Re: regression due to r187199 explow.c? in target ia64-hp-hpux11.23.

2012-06-06 Thread Tristan Gingold
-hpux, only for ia64-openvms (which doesn't support omp). What is the backtrace ? Tristan. Regards, Kannan -Original Message- From: Mailaripillai, Kannan Jeganathan Sent: Wednesday, June 06, 2012 2:15 PM To: 'Tristan Gingold' Cc: gcc@gcc.gnu.org Subject: RE: regression due

Re: regression due to r187199 explow.c? in target ia64-hp-hpux11.23.

2012-05-30 Thread Tristan Gingold
Hi, did you try with this patch: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg00970.html Tristan. On May 29, 2012, at 12:23 PM, Mailaripillai, Kannan Jeganathan wrote: Hi, This modification (assertion) is causing failure in ia64-hp-hpux11.23: r187199 | rsandifo | 2012-05-05 10:41:49

Ping: [Patch]: Fix call to end_prologue debug hook

2012-05-29 Thread Tristan Gingold
Ping for: On May 15, 2012, at 11:26 AM, Tristan Gingold wrote: Hi, the end_prologue debug hook (only used by dwarf on VMS) is currently called at the NOTE_INSN_FUNCTION_BEG. This is not what its name implies, neither what the VMS debugger expect and neither what a comment in dwarf2out.c

Ping: [Patch]: Fix ICE by expand_expr_addr_expr_1

2012-05-29 Thread Tristan Gingold
Ping for: On May 15, 2012, at 10:59 AM, Tristan Gingold wrote: Hi, I got ICE in plus_constant (after the assertions were added) due to expand_expr_addr_expr_1 during build on ia64/Openvms. This function is called with TMODE == SImode (32 bit pointers) but EXP designating a variable

Re: FYI: 1500+ typos, with suggested fixes

2012-05-29 Thread Tristan Gingold
On May 29, 2012, at 2:49 PM, Jim Meyering wrote: Running the following command spots over 1500 typos, and suggests fixes: $ git ls-files|misspellings -f -|grep -v '^ERROR:'|perl -pe \ 's/^(.*?)\[(\d+)\]: (\w+) - (.*?)$/sed -i '\''${2}s!$3!$4!'\'' $1/' k The misspellings command comes

Re: Regenerating fixincludes/configure

2012-05-29 Thread Tristan Gingold
On May 29, 2012, at 5:27 PM, Thomas Schwinge wrote: Hi! While looking at libatomic's, I noticed that fixincludes' configure can't be regenerated without errors. Here is a patch, in spirit of 749dea2a0549c126a0e992a6dd8e9b5eb28e1cee. OK to commit? Humm, looks like the same as Tobias

Re: [Patch]: Fix very large frame bug on i386

2012-05-25 Thread Tristan Gingold
On May 22, 2012, at 7:17 PM, Richard Henderson wrote: On 05/15/12 02:10, Tristan Gingold wrote: 2012-05-15 Tristan Gingold ging...@adacore.com * config/i386/i386.c (struct ix86_frame): Remove unused frame field. (ix86_compute_frame_layout): Fix type of stack_alignment_needed

Re: [Patch]: Fix typo in common/config/ia64/ia64-common.c

2012-05-25 Thread Tristan Gingold
On May 22, 2012, at 7:17 PM, Richard Henderson wrote: On 05/15/12 01:52, Tristan Gingold wrote: 2012-05-14 Tristan Gingold ging...@adacore.com * common/config/ia64/ia64-common.c (ia64_except_unwind_info): Fix typo. Ok. Thanks, committed. Tristan.

Re: Fix fixinclude's configure{,.ac}

2012-05-22 Thread Tristan Gingold
On May 22, 2012, at 11:20 AM, Tobias Burnus wrote: Dear all, an --enable-maintainers-build fails here with: configure.ac:99: error: possibly undefined macro: gcc_AC_FUNC_MMAP_BLACKLIST If this token and others are legitimate, please use m4_pattern_allow. See the Autoconf

Re: [Patch]: Fix very large frame bug on i386

2012-05-21 Thread Tristan Gingold
Ping for: On May 15, 2012, at 11:10 AM, Tristan Gingold wrote: Hi, use of 'unsigned int' for i386.c:ix86_compute_frame_layout stack_alignment_needed results in truncation of frame offset in code such as: offset = (offset + stack_alignment_needed - 1) -stack_alignment_needed

Re: Missing guard in ira-color.c ?

2012-05-15 Thread Tristan Gingold
On May 10, 2012, at 10:41 PM, Vladimir Makarov wrote: On 05/10/2012 09:10 AM, Tristan Gingold wrote: Hi, I am getting a segfault in ira-color.c:2945 on the trunk: Program received signal SIGSEGV, Segmentation fault. 0x00a79f37 in move_spill_restore () at ../../src/gcc/ira

[Patch]: Fix typo in common/config/ia64/ia64-common.c

2012-05-15 Thread Tristan Gingold
hi, looks like a typo... Ok for trunk ? Tristan. 2012-05-14 Tristan Gingold ging...@adacore.com * common/config/ia64/ia64-common.c (ia64_except_unwind_info): Fix typo. diff --git a/gcc/common/config/ia64/ia64-common.c b/gcc/common/config/ia64/ia64- index 1168253..79aed6a 100644

[PATCH v2]: handle size_t /= ptr size in tree-ssa-strlen.c

2012-05-15 Thread Tristan Gingold
On May 4, 2012, at 2:34 PM, Richard Guenther wrote: On Fri, May 4, 2012 at 2:27 PM, Tristan Gingold ging...@adacore.com wrote: Hi, get_string_length assumes that precision of size_t is the same as precision of a pointer, which isn't always true at least on VMS. This patch simply add

[Patch]: Fix ICE by expand_expr_addr_expr_1

2012-05-15 Thread Tristan Gingold
by the following patch. No regression on x86-64 GNU/Linux. Ok for trunk ? Tristan. 2012-05-15 Tristan Gingold ging...@adacore.com * expr.c (expand_expr_addr_expr_1): Call convert_memory_address_addr_space. diff --git a/gcc/expr.c b/gcc/expr.c index 3edb4a2..1b0ad8d 100644 --- a/gcc

[Patch]: Fix very large frame bug on i386

2012-05-15 Thread Tristan Gingold
). As a consequence, frames larger than 4GB are squeezed. Also, the frame field of struct ix86_frame is never used. Bootstrapped and reg-tested on x86_64 GNU/Linux, without regressions. Ok for trunk ? Tristan. 2012-05-15 Tristan Gingold ging...@adacore.com * config/i386/i386.c (struct

[Patch]: Fix call to end_prologue debug hook

2012-05-15 Thread Tristan Gingold
insn that corresponds to something the user wrote. These may be very different locations once scheduling is enabled. */ This is fixed by this patch. No regressions on x86_64 GNU/Linux. Ok for trunk ? Tristan. 2012-05-15 Tristan Gingold ging...@adacore.com * final.c

Re: [PATCH v2]: handle size_t /= ptr size in tree-ssa-strlen.c

2012-05-15 Thread Tristan Gingold
On May 15, 2012, at 11:13 AM, Richard Guenther wrote: On Tue, May 15, 2012 at 10:55 AM, Tristan Gingold ging...@adacore.com wrote: On May 4, 2012, at 2:34 PM, Richard Guenther wrote: On Fri, May 4, 2012 at 2:27 PM, Tristan Gingold ging...@adacore.com wrote: Hi, get_string_length

Re: Ping: always supply a mode to plus_constant

2012-05-14 Thread Tristan Gingold
:3279: error: too few arguments to function 'plus_constant' /gcc/spec/sb-terbium-head-64/gcc/gcc/config/ia64/ia64.c:3330: error: incompatible type for argument 1 of 'plus_constant' ... I think this was: 2012-05-10 Eric Botcazou ebotca...@adacore.com Tristan Gingold ging

Re: Missing guard in ira-color.c ?

2012-05-11 Thread Tristan Gingold
On May 10, 2012, at 10:41 PM, Vladimir Makarov wrote: On 05/10/2012 09:10 AM, Tristan Gingold wrote: Hi, I am getting a segfault in ira-color.c:2945 on the trunk: Program received signal SIGSEGV, Segmentation fault. 0x00a79f37 in move_spill_restore () at ../../src/gcc/ira

Re: [PATCH libcpp]: Avoid crash in interpret_float_suffix

2012-05-10 Thread Tristan Gingold
On May 8, 2012, at 5:39 PM, Tom Tromey wrote: Tristan == Tristan Gingold ging...@adacore.com writes: Tristan 2012-05-04 Tristan Gingold ging...@adacore.com Tristan * expr.c (interpret_float_suffix): Add a guard. Ok. Thanks, now committed.

Missing guard in ira-color.c ?

2012-05-10 Thread Tristan Gingold
Hi, I am getting a segfault in ira-color.c:2945 on the trunk: Program received signal SIGSEGV, Segmentation fault. 0x00a79f37 in move_spill_restore () at ../../src/gcc/ira-color.c:2945 2945 || ira_reg_equiv_const[regno] != NULL_RTX (gdb) l 2940 /* don't

[PATCH libcpp]: Avoid crash in interpret_float_suffix

2012-05-04 Thread Tristan Gingold
[0x] which is likely (and does) crash cc1. To avoid this nasty effect, I just added a guard. Bootstrapped and regtested on i386/GNU linux. Ok for trunk ? Tristan. libcpp/ 2012-05-04 Tristan Gingold ging...@adacore.com * expr.c (interpret_float_suffix): Add a guard. diff --git

[PATCH]: handle size_t /= ptr size in tree-ssa-strlen.c

2012-05-04 Thread Tristan Gingold
? Tristan. 2012-05-04 Tristan Gingold ging...@adacore.com * tree-ssa-strlen.c (get_string_length): Convert lhs if needed. diff --git a/gcc/tree-ssa-strlen.c b/gcc/tree-ssa-strlen.c index a37633a..bda5ffb 100644 --- a/gcc/tree-ssa-strlen.c +++ b/gcc/tree-ssa-strlen.c @@ -427,6 +427,12

Re: [PATCH]: handle size_t /= ptr size in tree-ssa-strlen.c

2012-05-04 Thread Tristan Gingold
On May 4, 2012, at 2:34 PM, Richard Guenther wrote: On Fri, May 4, 2012 at 2:27 PM, Tristan Gingold ging...@adacore.com wrote: Hi, get_string_length assumes that precision of size_t is the same as precision of a pointer, which isn't always true at least on VMS. This patch simply add

Re: [patch] For alpha-vms, unset flag_jump_tables if flag_pic is nonzero

2012-05-03 Thread Tristan Gingold
On May 2, 2012, at 11:45 PM, Richard Henderson wrote: On 05/02/2012 12:26 PM, Steven Bosscher wrote: * config/alpha/vms.h (SUBTARGET_OVERRIDE_OPTIONS): For pic code, unset flag_jump_tables. * stmt.c (expand_case): Remove special flag_pic case conditional on

Re: [patch] For alpha-vms, unset flag_jump_tables if flag_pic is nonzero

2012-05-03 Thread Tristan Gingold
On May 2, 2012, at 9:12 PM, Steven Bosscher wrote: Hello, alpha-vms is the only target that does not define ASM_OUTPUT_ADDR_DIFF_ELT. That makes the code in stmt.c to handle this case alpha-vms specific. But there is a better way to handle this: Just mimic -fno-jump-tables if flag_pic is

Re: [Fixinclude]: Fix typo and default to twoprocess on VMS

2012-04-24 Thread Tristan Gingold
On Apr 18, 2012, at 8:23 PM, Bruce Korb wrote: Hi, When I approved a patch in 2008, there was a typo. I didn't notice and it was fixed by removing a formatting element. Your patch corrects the error. Please apply your changes to active branches. Thank you! Regards, Bruce Thanks,

Re: [Fixinclude]: Fix typo and default to twoprocess on VMS

2012-04-18 Thread Tristan Gingold
On Apr 17, 2012, at 4:21 PM, Bruce Korb wrote: Hi Tristan, On Tue, Apr 17, 2012 at 5:57 AM, Tristan Gingold ging...@adacore.com wrote: Hi, one-process methodology cannot be used on VMS[...] But, when twoprocess is selected, gcc emits a warning[...] Ok for trunk ? diff --git

[Fixinclude]: Fix typo and default to twoprocess on VMS

2012-04-17 Thread Tristan Gingold
. fixincludes/ 2012-04-17 Tristan Gingold ging...@adacore.com * fixincl.c (fix_with_system): Add missing specifier. * configure.ac: Default to twoprocess on vms. * configure: Regenerate. diff --git a/fixincludes/configure.ac b/fixincludes/configure.ac index e7de791..f1fb2ff 100644

[Doc] Move paragraph about multiple attributes

2012-04-11 Thread Tristan Gingold
Hi, this paragraph is inside the 'i386 type attributes' section, while it isn't target specific. This patch moves it at the end of the attribute list. Maybe we should remove it completely as there is an 'attribute syntax' section. Ok for trunk ? Tristan. 2012-04-11 Tristan Gingold ging

Re: [Doc] Move paragraph about multiple attributes

2012-04-11 Thread Tristan Gingold
On Apr 11, 2012, at 12:03 PM, Richard Guenther wrote: On Wed, Apr 11, 2012 at 11:33 AM, Tristan Gingold ging...@adacore.com wrote: Hi, this paragraph is inside the 'i386 type attributes' section, while it isn't target specific. This patch moves it at the end of the attribute list

Re: intptr fixes for the testsuite

2012-04-11 Thread Tristan Gingold
On Apr 11, 2012, at 3:55 PM, Bernd Schmidt wrote: I'm working on a target where intptr_t and pointers are larger than size_t and ptrdiff_t. The testsuite has problems in this area, since we often use the latter two types for casting from/to pointers, leading to unwanted warnings. In some

Re: intptr fixes for the testsuite

2012-04-11 Thread Tristan Gingold
On Apr 11, 2012, at 4:52 PM, Mike Stump wrote: On Apr 11, 2012, at 7:04 AM, Bernd Schmidt wrote: On 04/11/2012 04:02 PM, Tristan Gingold wrote: On Apr 11, 2012, at 3:55 PM, Bernd Schmidt wrote: I'm working on a target where intptr_t and pointers are larger than size_t and ptrdiff_t

  1   2   3   >