Asiri Rathnayake via llvm-dev
2015-Nov-02 16:36 UTC
[llvm-dev] [cfe-dev] [RFC][libcxx] Fix and maintain the no-exceptions build of libcxx
> I find it amusing that you think that there's a "correct behavior" when the standard specifies > that an operation should throw an exception and you have disabled exceptions.You got me there! I think it's OK to be consistent across all the places and call abort(), this is what I've been doing in my current patch. I will double check this behavior with other compilers that provide similar functionality. Thanks. - Asiri
Marshall Clow via llvm-dev
2015-Nov-04 01:55 UTC
[llvm-dev] [cfe-dev] [RFC][libcxx] Fix and maintain the no-exceptions build of libcxx
On Mon, Nov 2, 2015 at 8:36 AM, Asiri Rathnayake <asiri.rathnayake at arm.com> wrote:> > I find it amusing that you think that there's a "correct behavior" when > the standard specifies > > that an operation should throw an exception and you have disabled > exceptions. > > You got me there! > > I think it's OK to be consistent across all the places and call abort(), > this is what I've been doing > in my current patch. I will double check this behavior with other > compilers that provide similar > functionality. > >I also want to point out that if you really want no exceptions, you're going to have to produce (and link to) a custom dylib; since the stuff in the dylib *will* throw. -- Marshall -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20151103/588d9e41/attachment.html>
Asiri Rathnayake via llvm-dev
2015-Nov-04 07:24 UTC
[llvm-dev] [cfe-dev] [RFC][libcxx] Fix and maintain the no-exceptions build of libcxx
> > I also want to point out that if you really want no exceptions, you're > going to have to produce (and link to) a custom dylib; since the stuff in > the dylib *will* throw. >IIUC, what you mean here is that it's not just the libcxx headers that I need to modify, but also the library code (stuff inside libcxx/src). This is exactly what I'm currently working on. I have a patch in review at http://reviews.llvm.org/D14292 which makes it possible to get this library (.so) to build. In my follow-up patch (which fixes some test failures on this new libcxx library build), there are changes to both the headers as well as the core library sources. Thanks. - Asiri> > -- Marshall > > > _______________________________________________ > cfe-dev mailing list > cfe-dev at lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev > >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20151104/108bf814/attachment.html>