From owner-freebsd-arm@FreeBSD.ORG Wed Feb 25 22:45:12 2015 Return-Path: Delivered-To: freebsd-arm@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id AFF00ED5 for ; Wed, 25 Feb 2015 22:45:12 +0000 (UTC) Received: from phabric-backend.isc.freebsd.org (phabric-backend.isc.freebsd.org [IPv6:2001:4f8:3:ffe0:406a:0:50:2]) (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 8CDD37D1 for ; Wed, 25 Feb 2015 22:45:12 +0000 (UTC) Received: from phabric-backend.isc.freebsd.org (phabric-backend.isc.freebsd.org [127.0.1.5]) by phabric-backend.isc.freebsd.org (8.14.9/8.14.9) with ESMTP id t1PMjCh6016079 for ; Wed, 25 Feb 2015 22:45:12 GMT (envelope-from root@phabric-backend.isc.freebsd.org) Received: (from root@localhost) by phabric-backend.isc.freebsd.org (8.14.9/8.14.9/Submit) id t1PMjCo6016078; Wed, 25 Feb 2015 22:45:12 GMT (envelope-from root) Date: Wed, 25 Feb 2015 22:45:12 +0000 To: freebsd-arm@freebsd.org From: "imp (Warner Losh)" Subject: [Differential] [Changed Subscribers] D1967: Attempt to make libcxxrt's dwarf_eh work with strict alignment Message-ID: X-Priority: 3 Thread-Topic: D1967: Attempt to make libcxxrt's dwarf_eh work with strict alignment X-Herald-Rules: none X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-Cc: X-Phabricator-Cc: X-Phabricator-Cc: In-Reply-To: References: Thread-Index: OWI2ZDlhM2EzZjk0NzYzNDhlNmU2YmYxZmFmIFTuUHg= X-Phabricator-Sent-This-Message: Yes X-Mail-Transport-Agent: MetaMTA X-Auto-Response-Suppress: All X-Phabricator-Mail-Tags: , MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="utf-8" X-BeenThere: freebsd-arm@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: "Porting FreeBSD to ARM processors." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 25 Feb 2015 22:45:12 -0000 imp added a subscriber: imp. imp added a comment. I'd just do a memcpy version of the read routines. All this #ifdef junk is insane. It will do the right thing every time. INLINE COMMENTS contrib/libcxxrt/dwarf_eh.h:154-163 uint16_t res; memcpy(&res, reinterpret_cast(*data), sizeof(uint16_)t); would be better. contrib/libcxxrt/dwarf_eh.h:176-188 see above for less insane code. contrib/libcxxrt/dwarf_eh.h:200-221 see above for less insane code. REVISION DETAIL https://reviews.freebsd.org/D1967 To: dim, andrew, bapt, ian, theraven Cc: imp, emaste, freebsd-arm