From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 26923 invoked by alias); 13 Dec 2010 12:31:16 -0000 Received: (qmail 26912 invoked by uid 22791); 13 Dec 2010 12:31:15 -0000 X-SWARE-Spam-Status: No, hits=-2.1 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW X-Spam-Check-By: sourceware.org Received: from mail-wy0-f169.google.com (HELO mail-wy0-f169.google.com) (74.125.82.169) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 13 Dec 2010 12:31:09 +0000 Received: by wyj26 with SMTP id 26so6167293wyj.0 for ; Mon, 13 Dec 2010 04:31:07 -0800 (PST) Received: by 10.216.155.205 with SMTP id j55mr2837847wek.90.1292243467419; Mon, 13 Dec 2010 04:31:07 -0800 (PST) MIME-Version: 1.0 Received: by 10.216.182.206 with HTTP; Mon, 13 Dec 2010 04:30:47 -0800 (PST) In-Reply-To: <4D015A4A.7040500@codesourcery.com> References: <4D015A4A.7040500@codesourcery.com> From: Hui Zhu Date: Mon, 13 Dec 2010 12:31:00 -0000 Message-ID: Subject: Re: [PATCH] Print trace state variables To: Stan Shebs Cc: gdb-patches@sourceware.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes 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 X-SW-Source: 2010-12/txt/msg00176.txt.bz2 Cool! This function is really great! Thanks, Hui On Fri, Dec 10, 2010 at 06:38, Stan Shebs wrote: > This patch remedies an oversight in recent tracepoint work, by adding > printing of trace state variables. =A0While you can see tsv values using = "info > tvariables", it's much more useful to be able to include them in print > commands and expressions in general; you can then display them in differe= nt > formats, or compute the difference between a tsv and some other value. = =A0The > manual already documents this as working, only the code to do it has been > missing. :-) > > This has been in CodeSourcery's version for some time, but I set it aside > for awhile because it seemed a little kludgy to add a tracepoint-specific > case into general evaluation. =A0On the plus side, its effect is localize= d, > and should be safe for a 7.2 update release, where it will be useful for > work on trace support in Eclipse. =A0There are no regressions testing with > either i386-linux native or GDBserver. > > If this seems reasonable, I'll put in both trunk and 7.2 branch. > > Stan > > 2010-12-08 =A0Stan Shebs > > =A0 =A0* value.c (value_of_internalvar): Add case for trace state > =A0 =A0variables. > > =A0 =A0* gdb.trace/tsv.exp: Test print command on trace state variables. > > > >