From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27626 invoked by alias); 10 Nov 2014 19:44:53 -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 27609 invoked by uid 89); 10 Nov 2014 19:44:52 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.2 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-yh0-f41.google.com Received: from mail-yh0-f41.google.com (HELO mail-yh0-f41.google.com) (209.85.213.41) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Mon, 10 Nov 2014 19:44:51 +0000 Received: by mail-yh0-f41.google.com with SMTP id i57so3516876yha.14 for ; Mon, 10 Nov 2014 11:44:49 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=Hwogjspd12wKuAIKFtXCU5UiWWQLVuHBWZmbEQkjcss=; b=jm2FBwNjkoHEFd+imC8XQ8iA5g2qA27peHOwAoE++LDD3vXtKKqURr/49E7xlgiwcl WZPdo00BVjZvw3H6e8A0j/oPzaDFBN9HCcYPiQjHpMkdi4IJu+s3G8hGD4dESc+Keztw R4tT4XuALTJlO8IHXBUnIEhUDVJk2tcXq022iH1eoDgyaSctcECoTP9KqiBeOcX1iuoK 4WVl9mDN776joZ+AQ+slDpz858p27bsYMl8WatWrUE1ptAmu3jzMtSJ7sMUP+qRaaoBf ZAh9Jv2Qnvx+HzpM3DNvI9NbiFSAzAgZtjSmWYZKfWr2C9H+1d7TaXHMzbxfoRcDNa8d SpgQ== X-Gm-Message-State: ALoCoQkmTQ6f/jp/mek6BMKB2A+vTIdeZRXwq85cU92sR5oUu3x9CROghsP5RrrcV9uhET2+h00s MIME-Version: 1.0 X-Received: by 10.221.66.199 with SMTP id xr7mr1572580vcb.70.1415648689350; Mon, 10 Nov 2014 11:44:49 -0800 (PST) Received: by 10.52.114.101 with HTTP; Mon, 10 Nov 2014 11:44:49 -0800 (PST) In-Reply-To: <87k333izfm.fsf@codesourcery.com> References: <1414195968-3333-1-git-send-email-yao@codesourcery.com> <1414195968-3333-3-git-send-email-yao@codesourcery.com> <21593.22575.941029.980760@ruffy2.mtv.corp.google.com> <87fvdypgap.fsf@codesourcery.com> <87k333izfm.fsf@codesourcery.com> Date: Mon, 10 Nov 2014 19:44:00 -0000 Message-ID: Subject: Re: [PATCH 2/6] DW attribute macro MACRO_AT_func and MACRO_AT_range From: Doug Evans To: Yao Qi Cc: gdb-patches Content-Type: text/plain; charset=UTF-8 X-IsSubscribed: yes X-SW-Source: 2014-11/txt/msg00170.txt.bz2 On Sun, Nov 9, 2014 at 6:04 PM, Yao Qi wrote: > I am inclined to delete them via "remote_file host delete" as many > executables func_addr[pid].x are generated. Supposing dwarf assembler > works perfectly, people have only to concentrate on their test artifacts > instead of func_addr[pid].x. OTOH, leaving such things around for debug purposes is important, or at least not making it difficult to keep such things (and editing source files to, e.g., comment out the "file delete", crosses a threshold of annoyance for me). I think the simplicity of implementation and assistance to debugability of just leaving the file there outweighs the extra code and any perceived increase in cleanliness. Plus the more remote file operations we add the more we slow things down. There's a *ton* of files we leave behind, I wouldn't worry about leaving a few more behind. If there were another reason then that might be something to consider. [At least with gcc there is -save-temps. We *could* have something like that here, but I'm ok with just leaving them.]