From owner-svn-src-projects@freebsd.org Tue Sep 6 20:01:16 2016 Return-Path: Delivered-To: svn-src-projects@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id AC5E3BC736E for ; Tue, 6 Sep 2016 20:01:16 +0000 (UTC) (envelope-from dim@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 6983838C; Tue, 6 Sep 2016 20:01:16 +0000 (UTC) (envelope-from dim@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id u86K1FE4001105; Tue, 6 Sep 2016 20:01:15 GMT (envelope-from dim@FreeBSD.org) Received: (from dim@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id u86K1FRK001104; Tue, 6 Sep 2016 20:01:15 GMT (envelope-from dim@FreeBSD.org) Message-Id: <201609062001.u86K1FRK001104@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: dim set sender to dim@FreeBSD.org using -f From: Dimitry Andric Date: Tue, 6 Sep 2016 20:01:15 +0000 (UTC) To: src-committers@freebsd.org, svn-src-projects@freebsd.org Subject: svn commit: r305496 - projects/clang390-import/contrib/libc++/include X-SVN-Group: projects MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-projects@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "SVN commit messages for the src " projects" tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 06 Sep 2016 20:01:16 -0000 Author: dim Date: Tue Sep 6 20:01:15 2016 New Revision: 305496 URL: https://svnweb.freebsd.org/changeset/base/305496 Log: Move inclusion of and in the -fno-exceptions case to the end of libc++'s . This is a workaround for building Firefox, which generates a rather convoluted maze of standard library wrapper headers, and this leads to an unfortunate sequence of: 1. wrapper includes libc++ , 2. which includes wrapper , 3. which includes libc++ , 4. which includes wrapper (because of -fno-exception), 5. which includes libc++ again, 6. which includes mozalloc.h, 7. which tries to declare operator new with std::bad_alloc, 8. which gives an error because std::bad_alloc is not yet defined. The inclusion at step 5 does nothing, because the header guard for was already encountered in step 1. Then when moz_alloc.h tries to use std::bad_alloc, it is not yet defined, because we are still busy processing (where this class is defined) from step 3. Mozilla has https://bugzilla.mozilla.org/show_bug.cgi?id=1269171 for this, reported by Jan Beich (jbeich@), but when the fix for it is applied to Firefox, we get into another, similar problem situation: 1. some header includes wrapper , 2. which includes libc++ , 3. which includes wrapper (because of -fno-exceptions), 4. which includes mozalloc.h, 5. which includes wrapper , 6. which includes libc++ , 7. which gives an error defining std::bad_alloc, because std::exception is not yet defined. At step 3, we were at the top of libc++'s , and at that point std::exception is not yet defined. At step 6, does include again, but similar to step 5 in the previous problem case, the header guard was already encountered, so the whole header is skipped. In upstream libc++'s later revisions r279744 and r279763, the reason for including and was nullified again, but these commits are rather large and intrusive. Therefore, move the includes to the bottom of the file, just before where they are needed. At that point, std::exception is already fully defined. Suggested by: Jörg Sonnenberger Modified: projects/clang390-import/contrib/libc++/include/exception Modified: projects/clang390-import/contrib/libc++/include/exception ============================================================================== --- projects/clang390-import/contrib/libc++/include/exception Tue Sep 6 19:54:52 2016 (r305495) +++ projects/clang390-import/contrib/libc++/include/exception Tue Sep 6 20:01:15 2016 (r305496) @@ -80,10 +80,6 @@ template void rethrow_if_neste #include <__config> #include #include -#if defined(_LIBCPP_NO_EXCEPTIONS) -#include -#include -#endif #if !defined(_LIBCPP_HAS_NO_PRAGMA_SYSTEM_HEADER) #pragma GCC system_header @@ -255,6 +251,11 @@ rethrow_if_nested(const _Ep&, typename e } // std +#if defined(_LIBCPP_NO_EXCEPTIONS) +#include +#include +#endif + _LIBCPP_BEGIN_NAMESPACE_STD template