This is the mail archive of the cygwin-patches mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 09/11] Add c++14 sized deallocation operator


On Mar 21 13:46, Peter Foley wrote:
> On Mon, Mar 21, 2016 at 1:13 PM, Corinna Vinschen
> <corinna-cygwin@cygwin.com> wrote:
> > But we export these functions as fallback functions to the applications.
> > See libstdcxx_wrapper.cc and the end of cxx.cc.  While the comment in
> > cxx.cc claims that this should "not be used in practice", there might be
> > c++14 code ending up with undefined references to the new delete
> > operator, isn't it?
> >
> > https://cygwin.com/ml/cygwin-patches/2009-q3/msg00010.html outlines
> > why these exports were necessary in the first place.
> 
> Ah, I'll look into updating those files as well then.

I realized that your orignal patch isn't invalidated by this so I tried
to apply it and we could then add the other stuff later.  However, it
doesn't compile due to a warning, and since we're always building with
-Werror...

[...]/cxx.cc:33:1: error: âvoid operator delete(void*, size_t)â is a usual (non-placement) deallocation function in C++14 (or with -fsized-deallocation) [-Werror=c++14-compat]
 operator delete (void *p, size_t)
 ^
cc1plus: all warnings being treated as errors

I'm not sure it's the right thing to switch to C++14 by default using
gcc 5.3 yet.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

Attachment: signature.asc
Description: PGP signature


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]