Re: [PATCH] Trace state variables

[Tom Tromey <tromey at redhat dot com>]

>>>>> "Stan" == Stan Shebs <stan@codesourcery.com> writes:

Stan> +     case BINOP_ASSIGN:

>> It seems a little odd to add BINOP_ASSIGN but not BINOP_ASSIGN_MODIFY.

Stan> <groan> You're going to make me do it, aren't you. :-)

:-)

It is really up to you, I won't insist on it.  I don't think it is much
code, though...

Stan> extern int remote_supports_cond_tracepoints (void);
Stan> + extern char *unpack_varlen_hex (char *buff, ULONGEST *result);

>> It seems like this could be in a header somewhere.

Stan> This is one of the temporary hacks pending target-vectorization of
Stan> tracepoint ops.

Aha, thanks.

Tom