From: Bart V. A. <bva...@so...> - 2024-11-18 21:41:28
|
https://sourceware.org/git/gitweb.cgi?p=valgrind.git;h=bc57ecb85b97347e31334769d615b4553d3463d4 commit bc57ecb85b97347e31334769d615b4553d3463d4 Author: Bart Van Assche <bva...@ac...> Date: Sun Nov 17 16:20:39 2024 -0800 drd: Remove a superfluous if-statement Check the client request ID once instead of twice. Diff: --- drd/drd_clientreq.c | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/drd/drd_clientreq.c b/drd/drd_clientreq.c index 79f515dda2..8a462253df 100644 --- a/drd/drd_clientreq.c +++ b/drd/drd_clientreq.c @@ -77,20 +77,6 @@ static Bool handle_client_request(ThreadId vg_tid, UWord* arg, UWord* ret) UWord result = 0; const DrdThreadId drd_tid = DRD_(thread_get_running_tid)(); - if (!VG_IS_TOOL_USERREQ('D','R',arg[0]) - && !VG_IS_TOOL_USERREQ('D','r',arg[0]) - && !VG_IS_TOOL_USERREQ('H','G',arg[0]) - && VG_USERREQ__MALLOCLIKE_BLOCK != arg[0] - && VG_USERREQ__RESIZEINPLACE_BLOCK != arg[0] - && VG_USERREQ__FREELIKE_BLOCK != arg[0] -#if defined(VGO_solaris) - && VG_USERREQ_DRD_RTLD_BIND_GUARD != arg[0] - && VG_USERREQ_DRD_RTLD_BIND_CLEAR != arg[0] -#endif /* VGO_solaris */ - && VG_USERREQ__GDB_MONITOR_COMMAND != arg[0]) { - return False; - } - tl_assert(vg_tid == VG_(get_running_tid)()); tl_assert(DRD_(VgThreadIdToDrdThreadId)(vg_tid) == drd_tid || (VG_USERREQ__GDB_MONITOR_COMMAND == arg[0] |