From owner-svn-ports-head@FreeBSD.ORG Thu Oct 24 14:46:27 2013 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 61CF0A17; Thu, 24 Oct 2013 14:46:27 +0000 (UTC) (envelope-from brooks@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:1900:2254:2068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 3EBAB2325; Thu, 24 Oct 2013 14:46:27 +0000 (UTC) Received: from svn.freebsd.org ([127.0.1.70]) by svn.freebsd.org (8.14.7/8.14.7) with ESMTP id r9OEkR30006757; Thu, 24 Oct 2013 14:46:27 GMT (envelope-from brooks@svn.freebsd.org) Received: (from brooks@localhost) by svn.freebsd.org (8.14.7/8.14.5/Submit) id r9OEkQ08006755; Thu, 24 Oct 2013 14:46:26 GMT (envelope-from brooks@svn.freebsd.org) Message-Id: <201310241446.r9OEkQ08006755@svn.freebsd.org> From: Brooks Davis Date: Thu, 24 Oct 2013 14:46:26 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r331490 - in head/devel/llvm32: . files X-SVN-Group: ports-head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Oct 2013 14:46:27 -0000 Author: brooks Date: Thu Oct 24 14:46:26 2013 New Revision: 331490 URL: http://svnweb.freebsd.org/changeset/ports/331490 Log: Fix build with libc++ by importing r178240. Submitted by: tijl (via commit to devel/llvm) Added: head/devel/llvm32/files/patch-svn-r178240 (contents, props changed) Modified: head/devel/llvm32/Makefile Modified: head/devel/llvm32/Makefile ============================================================================== --- head/devel/llvm32/Makefile Thu Oct 24 14:23:53 2013 (r331489) +++ head/devel/llvm32/Makefile Thu Oct 24 14:46:26 2013 (r331490) @@ -2,7 +2,7 @@ PORTNAME= llvm PORTVERSION= 3.2 -PORTREVISION= 1 +PORTREVISION= 2 CATEGORIES= devel lang MASTER_SITES= http://llvm.org/releases/${PORTVERSION}/ DISTNAME= ${PORTNAME}-${PORTVERSION}.src Added: head/devel/llvm32/files/patch-svn-r178240 ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/devel/llvm32/files/patch-svn-r178240 Thu Oct 24 14:46:26 2013 (r331490) @@ -0,0 +1,86 @@ +$FreeBSD$ +------------------------------------------------------------------------ +r178240 | hhinnant | 2013-03-28 15:47:50 +0000 (Thu, 28 Mar 2013) | 17 lines + +Seciton 24.2.2 of the C++ standard, [iterator.iterators], Table 106 +requires that the return type of *r for all iterators r be reference, +where reference is defined in [iterator.requirements.general]/p11 as +iterator_traits::reference, and X is the type of r. + +But in CFG.h, the dereference operator of PredIterator and SuccIterator +return pointer, not reference. + +Furthermore the nested type reference is value_type&, which is not the +type returned from operator*(). + +This patch simply makes the iterator::reference type value_type*, which +is what the operator*() returns, and then re-lables the return type as +reference. + +From a functionality point of view, the only difference is that the +nested reference type is now value_type* instead of value_type&. +------------------------------------------------------------------------ +Index: include/llvm/Support/CFG.h +=================================================================== +--- include/llvm/Support/CFG.h (revision 178239) ++++ include/llvm/Support/CFG.h (revision 178240) +@@ -27,8 +27,9 @@ + + template // Predecessor Iterator + class PredIterator : public std::iterator { +- typedef std::iterator super; ++ Ptr, ptrdiff_t, Ptr*, Ptr*> { ++ typedef std::iterator super; + typedef PredIterator Self; + USE_iterator It; + +@@ -40,6 +41,7 @@ + + public: + typedef typename super::pointer pointer; ++ typedef typename super::reference reference; + + PredIterator() {} + explicit inline PredIterator(Ptr *bb) : It(bb->use_begin()) { +@@ -50,7 +52,7 @@ + inline bool operator==(const Self& x) const { return It == x.It; } + inline bool operator!=(const Self& x) const { return !operator==(x); } + +- inline pointer operator*() const { ++ inline reference operator*() const { + assert(!It.atEnd() && "pred_iterator out of range!"); + return cast(*It)->getParent(); + } +@@ -100,10 +102,11 @@ + + template // Successor Iterator + class SuccIterator : public std::iterator { ++ BB_, ptrdiff_t, BB_*, BB_*> { + const Term_ Term; + unsigned idx; +- typedef std::iterator super; ++ typedef std::iterator super; + typedef SuccIterator Self; + + inline bool index_is_valid(int idx) { +@@ -112,6 +115,7 @@ + + public: + typedef typename super::pointer pointer; ++ typedef typename super::reference reference; + // TODO: This can be random access iterator, only operator[] missing. + + explicit inline SuccIterator(Term_ T) : Term(T), idx(0) {// begin iterator +@@ -142,7 +146,7 @@ + inline bool operator==(const Self& x) const { return idx == x.idx; } + inline bool operator!=(const Self& x) const { return !operator==(x); } + +- inline pointer operator*() const { return Term->getSuccessor(idx); } ++ inline reference operator*() const { return Term->getSuccessor(idx); } + inline pointer operator->() const { return operator*(); } + + inline Self& operator++() { ++idx; return *this; } // Preincrement