From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-77195-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 3256 invoked by alias); 23 Dec 2010 19:06:25 -0000
Received: (qmail 3248 invoked by uid 22791); 23 Dec 2010 19:06:24 -0000
X-SWARE-Spam-Status: No, hits=-1.0 required=5.0	tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_SOFTFAIL
X-Spam-Check-By: sourceware.org
Received: from mtaout23.012.net.il (HELO mtaout23.012.net.il) (80.179.55.175)    by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 23 Dec 2010 19:06:17 +0000
Received: from conversion-daemon.a-mtaout23.012.net.il by a-mtaout23.012.net.il (HyperSendmail v2007.08) id <0LDW00L00ABETN00@a-mtaout23.012.net.il> for gdb-patches@sourceware.org; Thu, 23 Dec 2010 21:06:14 +0200 (IST)
Received: from HOME-C4E4A596F7 ([84.229.167.122]) by a-mtaout23.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0LDW00LPXAECLM80@a-mtaout23.012.net.il>; Thu, 23 Dec 2010 21:06:14 +0200 (IST)
Date: Thu, 23 Dec 2010 19:13:00 -0000
From: Eli Zaretskii <eliz@gnu.org>
Subject: Re: [patch 2/3] Implement support for PowerPC BookE ranged watchpoints
In-reply-to: <1293130182.14239.21.camel@hactar>
To: Thiago Jung Bauermann <bauerman@br.ibm.com>
Cc: pedro@codesourcery.com, gdb-patches@sourceware.org, jan.kratochvil@redhat.com, brobecker@adacore.com
Reply-to: Eli Zaretskii <eliz@gnu.org>
Message-id: <83tyi4e2fk.fsf@gnu.org>
References: <1290549100.3164.47.camel@hactar> <201011251731.58135.pedro@codesourcery.com> <1290806122.3009.37.camel@hactar> <201011271747.39053.pedro@codesourcery.com> <1293130182.14239.21.camel@hactar>
X-IsSubscribed: yes
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
X-SW-Source: 2010-12/txt/msg00440.txt.bz2

> From: Thiago Jung Bauermann <bauerman@br.ibm.com>
> Cc: gdb-patches@sourceware.org, Jan Kratochvil <jan.kratochvil@redhat.com>,
>         Joel Brobecker <brobecker@adacore.com>, Eli Zaretskii <eliz@gnu.org>
> Date: Thu, 23 Dec 2010 16:49:42 -0200
> 
> +When running on PowerPC embedded processors @value{GDBN} automatically uses
                                              ^
Comma here, please.

> +ranged hardware watchpoints unless the @code{exact-watchpoints} option is on,
                              ^
And here.

> +@item set powerpc exact-watchpoints
> +@itemx show powerpc exact-watchpoints
> +Allow @value{GDBN} to use only one debug register when watching a variable
> +of scalar type, thus assuming that all acesses that modify that variable
> +happen at its starting address.

I suggest a slight rewording of the last sentence:

  ... thus assuming that the variable is accessed through the address
  of its first byte.

Is that wording accurate?  If it is, the patch for the manual is okay
with those changes.

Thanks.