RE: [PATCH v4 2/2] LAM: Enable tagged pointer support for watchpoints.

["Schimpe, Christina" <christina.schimpe@intel.com>]

> > diff --git a/gdb/NEWS b/gdb/NEWS
> > index 47677cb773a..fcbb89e8bf8 100644
> > --- a/gdb/NEWS
> > +++ b/gdb/NEWS
> > @@ -3,6 +3,8 @@
> >
> >  *** Changes since GDB 15
> >
> > +* GDB now supports watchpoints for tagged data pointers on amd64.
> > +
> >  * Debugger Adapter Protocol changes
> 
> Thanks.  The above NEWS text is okay, but I wonder whether we explain
> anywhere what are "tagged data pointers".
> 
> Reviewed-By: Eli Zaretskii <eliz@gnu.org>

Thanks for the review. I understand this can be a bit confusing. 
Is " GDB now supports watchpoints for tagged pointers used for data accesses on amd64." better?
Or, we could also just write: " GDB now supports watchpoints for tagged pointers on amd64.", as watchpoints are usually just data breakpoints.

What do you think?

Christina

Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de
Managing Directors: Sean Fennelly, Jeffrey Schneiderman, Tiffany Doon Silva
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928