From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-return-44238-listarch-gdb=sources.redhat.com@sourceware.org>
Received: (qmail 74347 invoked by alias); 19 Mar 2015 18:50:32 -0000
Mailing-List: contact gdb-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb.sourceware.org>
List-Subscribe: <mailto:gdb-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb/>
List-Post: <mailto:gdb@sourceware.org>
List-Help: <mailto:gdb-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-owner@sourceware.org
Received: (qmail 74322 invoked by uid 89); 19 Mar 2015 18:50:31 -0000
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=2.7 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,KAM_FROM_URIBL_PCCC,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=no version=3.3.2
X-Spam-User: qpsmtpd, 2 recipients
X-HELO: mail-ob0-f173.google.com
Received: from mail-ob0-f173.google.com (HELO mail-ob0-f173.google.com) (209.85.214.173) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Thu, 19 Mar 2015 18:50:31 +0000
Received: by obdfc2 with SMTP id fc2so61712165obd.3;        Thu, 19 Mar 2015 11:50:29 -0700 (PDT)
MIME-Version: 1.0
X-Received: by 10.202.198.76 with SMTP id w73mr31319129oif.74.1426791029262; Thu, 19 Mar 2015 11:50:29 -0700 (PDT)
Received: by 10.76.134.102 with HTTP; Thu, 19 Mar 2015 11:50:29 -0700 (PDT)
In-Reply-To: <CAHACq4qwgFbz_jF5hE1WeaKfhuYPD0iDerEiu4Dzx3KMtSdnUw@mail.gmail.com>
References: <CAMe9rOp7=hH0HSVmbKohv=5JmcV25owj_RtnPYirG1Bhej=XkA@mail.gmail.com>	<550318C90200007800069E5B@mail.emea.novell.com>	<CAMe9rOoy9dbAw7Kh9+-haTE4gj4331Dk5E1YAr+1Xx+HBzCF0w@mail.gmail.com>	<55031C360200007800069EAF@mail.emea.novell.com>	<CAMe9rOqR4tgNr9CvZXoGsO4ASxZyaAp47J4DL=VNtC9AuLYKGA@mail.gmail.com>	<CAHACq4qwgFbz_jF5hE1WeaKfhuYPD0iDerEiu4Dzx3KMtSdnUw@mail.gmail.com>
Date: Thu, 19 Mar 2015 18:50:00 -0000
Message-ID: <CAMe9rOqNoECma=Xhy4wdmX=ZXegvMnbL6nW1yyWtxZVsM+V8+g@mail.gmail.com>
Subject: Re: RFC: Add support for SHF_COMPRESSED
From: "H.J. Lu" <hjl.tools@gmail.com>
To: Cary Coutant <ccoutant@google.com>
Cc: Jan Beulich <JBeulich@suse.com>, Binutils <binutils@sourceware.org>, GDB <gdb@sourceware.org>
Content-Type: text/plain; charset=UTF-8
X-SW-Source: 2015-03/txt/msg00069.txt.bz2

On Thu, Mar 19, 2015 at 11:18 AM, Cary Coutant <ccoutant@google.com> wrote:
>> I added --compressed-debug-sections=[none|zlib] to as and
>> objcopy.  I updated gold to decompress SHF_COMPRESSED
>> section.  I didn't add --compressed-debug-sections=[none|zlib]
>> to gold since I don't know if this name is a final choice nor
>> gold maintainers want such an option.
>
> objcopy, gas, and gold already have the --compress-debug-sections
> option (though only gold takes "=[none|zlib]" as a parameter; objcopy
> and gas always use zlib, and spell the negative as
> --nocompress-debug-sections).
>
> I don't see any point in adding another option, especially one spelled
> so similarly -- just use the existing option and make it generate the
> new form, after all the consumers understand both formats. (And I
> prefer a verb to an adjective.)

Which consumers do we care?

> If you want the option to generate either format, how about something
> like "--compress-debug-sections=[none|zlib|zlib-old|zlib-new]"? (Where
> "zlib" would generate the old for a while, then we'd switch it to the
> new after some time.
>

I will do it.

Thanks.


-- 
H.J.