Re: [Ping] Port of VTV for Cygwin and MinGW

2015-02-02 Thread Patrick Wollgast
Hi, after the missed bug at Linux with no VTV I checked everything again on the trunk. I saw that I erroneously wrote in the changelog for libvtv/aclocal.m4 regenerate and deleted the change from the patch. The only change I made there in my working directory was the following. Index: libvtv/aclo

Re: [Ping] Port of VTV for Cygwin and MinGW

2015-01-28 Thread Patrick Wollgast
Ping. https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01270.html On 15.01.2015 22:50, Patrick Wollgast wrote: > On 15.01.2015 17:01, Ian Lance Taylor wrote: >> On Wed, Jan 14, 2015 at 11:54 PM, Patrick Wollgast >> wrote: >>> Is there something I'm still supposed to

Re: [Ping] Port of VTV for Cygwin and MinGW

2015-01-15 Thread Patrick Wollgast
On 15.01.2015 17:01, Ian Lance Taylor wrote: > On Wed, Jan 14, 2015 at 11:54 PM, Patrick Wollgast > wrote: >> Is there something I'm still supposed to do, since I don't have write >> access and this was the last part missing an "OK"? > > Somebody wit

Re: [Ping] Port of VTV for Cygwin and MinGW

2015-01-14 Thread Patrick Wollgast
On 15.01.2015 00:52, Ian Lance Taylor wrote: > On Wed, Jan 14, 2015 at 12:28 PM, Patrick Wollgast > wrote: >> On 14.01.2015 20:00, Ian Lance Taylor wrote: >>> On Thu, Jan 8, 2015 at 12:33 PM, Patrick Wollgast >>> wrote: >>>> A short recap again: &g

Re: [Ping] Port of VTV for Cygwin and MinGW

2015-01-14 Thread Patrick Wollgast
On 14.01.2015 20:00, Ian Lance Taylor wrote: > On Thu, Jan 8, 2015 at 12:33 PM, Patrick Wollgast > wrote: >> A short recap again: >> >> Latest patch, changelog and a test program (further information about >> the program in the mail): >> https://gcc.gnu.org/

Re: [Ping] Port of VTV for Cygwin and MinGW

2015-01-08 Thread Patrick Wollgast
A short recap again: Latest patch, changelog and a test program (further information about the program in the mail): https://gcc.gnu.org/ml/gcc-patches/2014-11/msg03368.html Approved: * gcc/config/i386/* * libgcc/* * libstdc++-v3/* * libvtv/* (Some changes made to three of these fil

Re: [Ping] Port of VTV for Cygwin and MinGW

2015-01-04 Thread Patrick Wollgast
On 10.12.2014 17:37, Patrick Wollgast wrote: > Ping. > > https://gcc.gnu.org/ml/gcc-patches/2014-11/msg03368.html > > On 27.11.2014 10:42, Patrick Wollgast wrote: >> On 12.11.2014 19:40, Kai Tietz wrote: >>> TerminateProcess is actually bad, as it doesn't

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-12-10 Thread Patrick Wollgast
Ping. https://gcc.gnu.org/ml/gcc-patches/2014-11/msg03368.html On 27.11.2014 10:42, Patrick Wollgast wrote: > On 12.11.2014 19:40, Kai Tietz wrote: >> TerminateProcess is actually bad, as it doesn't call any of the atexit >> handlers. You simply nuke the process off. For

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-11-27 Thread Patrick Wollgast
On 12.11.2014 19:40, Kai Tietz wrote: > TerminateProcess is actually bad, as it doesn't call any of the atexit > handlers. You simply nuke the process off. For cygwin this behavior > is inacceptable. Why a classical abort, or a classical exit call > cause for you that issues? It seems to me mor

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-11-12 Thread Patrick Wollgast
> > I don't think you have addressed all of the comments I made in the > comment, do you? > > Regards, > Kai > I added the three checks, if TARGET_PECOFF is defined, and fixed the whitespace issues. For the questions regarding C-runtime/Win32 functions I haven't changed anything in the patch b

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-11-12 Thread Patrick Wollgast
Ping for the question below. On 30.10.2014 15:29, Patrick Wollgast wrote: > Since I haven't heard back for quite a while, I wanted to ask what the > current stat of the patch is. > > Is the patch from the last mail approved ( > https://gcc.gnu.org/ml/gcc-patches/2014-10

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-10-30 Thread Patrick Wollgast
Since I haven't heard back for quite a while, I wanted to ask what the current stat of the patch is. Is the patch from the last mail approved ( https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01524.html ), or should the matters discussed further? regards, Patrick

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-10-16 Thread Patrick Wollgast
On 09.10.2014 16:42, Kai Tietz wrote: >> Not approved: >> * gcc/cp/vtable-class-hierarchy.c > > Index: gcc/cp/vtable-class-hierarchy.c > === > --- gcc/cp/vtable-class-hierarchy.c(Revision 214408) > +++ gcc/cp/vtable-class-hierarch

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-10-09 Thread Patrick Wollgast
On 27.09.2014 12:50, Kai Tietz wrote: > Hi Patrick, > > the mingw/cygwin part your patch looks fine to me. Nevertheless I > have one question regarding to you. Do you have FSF papers for gcc > already? As I asked an overseer and he didn't found you on the list. > > Regards, > Kai > The paper

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-09-24 Thread Patrick Wollgast
On 23.09.2014 12:22, Jonathan Wakely wrote: > On 19/09/14 00:23 +0200, Patrick Wollgast wrote: >> Index: libstdc++-v3/acinclude.m4 >> === >> --- libstdc++-v3/acinclude.m4(Revision 214408) >> ++

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-09-18 Thread Patrick Wollgast
Added Benjamin De Kosnik as a c++ runtime libs maintainer and Kai Tietz as Windows/Cygwin/MinGW maintainer. >> In changes to gcc/config/i386/cygwin.h mingw-w64.h and mingw32.h, you >> forgot to handle the "fvtable-verify=preinit" options. >> fvtable-veriy=preinit should cause vtv_start_preinit

[Ping] Port of VTV for Cygwin and MinGW

2014-09-10 Thread Patrick Wollgast
Ping for https://gcc.gnu.org/ml/gcc-patches/2014-08/msg02559.html Also added Caroline Tice, as libvtv maintainer, to cc and attached virtual_func_test_min_UAF.cpp, which I forgot in the original mail. Patrick On 28.08.2014 13:03, Patrick Wollgast wrote: > This patch contains a port of

Port of VTV for Cygwin and MinGW

2014-08-28 Thread Patrick Wollgast
This patch contains a port of VTV -fvtable-verify=std for Cygwin and MinGW. Since weak symbols on Windows and Linux are implemented differently, and VTV should have the possibility to be switched on and off, the structure of the feature had to be modified. On Linux libstdc++ contains the weak stub