From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 34628 invoked by alias); 3 Jan 2017 10:57:21 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 34605 invoked by uid 89); 3 Jan 2017 10:57:20 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.1 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,RCVD_IN_SORBS_SPAM,SPF_PASS autolearn=ham version=3.3.2 spammy=regarded, D*foss.arm.com, jiong.wang@foss.arm.com, jiongwangfossarmcom X-Spam-User: qpsmtpd, 2 recipients X-HELO: mail-qt0-f196.google.com Received: from mail-qt0-f196.google.com (HELO mail-qt0-f196.google.com) (209.85.216.196) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 03 Jan 2017 10:57:10 +0000 Received: by mail-qt0-f196.google.com with SMTP id 3so35610463qtr.2; Tue, 03 Jan 2017 02:57:10 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=iSo1ni4L5/tt4sc8YCldwMgiJvjJATa5DDHXZqNDqo0=; b=q3rB2FKNKjHS6WfSid7xJATmELcVqwZtiS52tGX76Mib5OVgZCOwwQ6USbr5It7XtN HTrS+pPKaEatklw+ouwrgADqYyvEReVSlpnMSylwqTUJCZ6PWtV9a5VZb8P3yJijmhTA UCYW2h4DUUy594OCpzaYNDsHoZhzjMv+P9sD9s84c/0NzlvCC5D6nFUsGszXR5ItGFeJ oqdcMH81VkO1MMXER6JjTeyEJY6YH/7Vzv0ULaly/pN7Ae1c/frRD8blI/MwLTClo7pN kM6n1YfmYG8giFcpU3PLcLeyL3G7uKk0fsAUuonIZLS05MrwANe58nbgoueiHjeEnP7v WVZg== X-Gm-Message-State: AIkVDXKXev2NbNBiMeK7GDlSJi+ZAWgzCIh5hWZE+I800bmFhYWIisbyUw3WDQ1P27gjyC2b/TL0ReZ4E4k1Vg== X-Received: by 10.237.49.230 with SMTP id 93mr55629269qth.109.1483441029055; Tue, 03 Jan 2017 02:57:09 -0800 (PST) MIME-Version: 1.0 Received: by 10.12.144.11 with HTTP; Tue, 3 Jan 2017 02:57:08 -0800 (PST) In-Reply-To: <42e05bed-4819-3acb-1e3c-8982910e875f@foss.arm.com> References: <72418e98-a400-c503-e8ce-c3fbe1ecc4a7@foss.arm.com> <20161111193859.GJ3541@tucnak.redhat.com> <20161115161817.GL3541@tucnak.redhat.com> <5896be40-51de-55f7-f4a1-4c5af7ff9aec@foss.arm.com> <1479304496.14569.256.camel@redhat.com> <20161116140218.GU3541@tucnak.redhat.com> <07b84003-4e73-8a7f-f949-4c3500e4ffc4@foss.arm.com> <42e05bed-4819-3acb-1e3c-8982910e875f@foss.arm.com> From: Yao Qi Date: Tue, 03 Jan 2017 10:57:00 -0000 Message-ID: Subject: Re: [1/9][RFC][DWARF] Reserve three DW_OP numbers in vendor extension space To: Jiong Wang Cc: GDB , Binutils , "Richard Earnshaw (lists)" , Nick Clifton Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes X-SW-Source: 2017-01/txt/msg00007.txt.bz2 On Tue, Jan 3, 2017 at 10:09 AM, Jiong Wang wrote: > On 28/12/16 19:54, Cary Coutant wrote: >>> >>> OK on this proposal and to install this patch to gcc trunk? >>> >>> Hi GDB, Binutils maintainer: >>> >>> OK on this proposal and install this patch to binutils-gdb master? >>> >>> include/ >>> 2016-11-29 Richard Earnshaw >>> Jiong Wang >>> >>> * dwarf2.def (DW_OP_AARCH64_operation): Reserve the number 0xe= a. >> >> This is OK, but: >> >> +/* AARCH64 extensions. >> + DW_OP_AARCH64_operation takes one mandatory unsigned LEB128 operand. >> + Bits[6:0] of this operand is the action code, all others bits are >> initialized >> + to 0 except explicitly documented for one action. Please refer >> AArch64 DWARF >> + ABI documentation for details. */ >> >> Is it possible to include a stable URL that points to the ABI document? >> >> -cary > > > As this patch has got approve from GCC, is it OK to install into > binutils-gdb master? > > My understanding is "include/dwarf2.def" is shared between GDB and > Binutils. "include/dwarf2.def" in binutils-gdb is a copy from gcc, as the comments in the top explain "This file is part of GCC.". Since the patch was approved = in GCC, you need to commit the patch to GCC repository, and sync "include/dwarf2.def" from GCC repo to binutils-gdb repo. The last sync is done by Nick https://sourceware.org/ml/binutils/2015-09/msg00330.html > > This patch and the described AArch64 DWARF proposal has got GDB ack from > https://sourceware.org/ml/gdb-patches/2016-11/msg01008.html which, from my > understanding, is a conditional approval given there is follow up GDB > patches on multiplexing DW_CFA_GNU_window_save. > > This patch still need Binutils approval. > The sync is regarded as an obvious change, so you can commit it in binutils-gdb repo. --=20 Yao (=E9=BD=90=E5=B0=A7)