gdbsupport: move run_time_clock::now(user, system) out of run_time_clock

It is completely unrelated to run_time_clock, so I don't think it makes
sense to have it as a static function there.

Move it to be a free function named "get_run_time".

Change-Id: I0c3e4d3cc44ca37e523c94d72f7cd66add95645e
Approved-By: Tom Tromey <tom@tromey.com>
This commit is contained in:
Simon Marchi
2025-04-17 15:23:52 -04:00
committed by Simon Marchi
parent 48d0ac705c
commit f7a4e14a0b
3 changed files with 10 additions and 9 deletions

View File

@@ -38,8 +38,8 @@ timeval_to_microseconds (struct timeval *tv)
#endif
void
run_time_clock::now (user_cpu_time_clock::time_point &user,
system_cpu_time_clock::time_point &system) noexcept
get_run_time (user_cpu_time_clock::time_point &user,
system_cpu_time_clock::time_point &system) noexcept
{
#ifdef HAVE_GETRUSAGE
struct rusage rusage;

View File

@@ -51,6 +51,13 @@ struct system_cpu_time_clock
static time_point now () noexcept = delete;
};
/* Return the user/system time as separate time points, if
supported. If not supported, then the combined user+kernel time
is returned in USER and SYSTEM is set to zero. */
void get_run_time (user_cpu_time_clock::time_point &user,
system_cpu_time_clock::time_point &system) noexcept;
/* Count the total amount of time spent executing in userspace+kernel
mode. */
@@ -64,12 +71,6 @@ struct run_time_clock
static constexpr bool is_steady = true;
static time_point now () noexcept;
/* Return the user/system time as separate time points, if
supported. If not supported, then the combined user+kernel time
is returned in USER and SYSTEM is set to zero. */
static void now (user_cpu_time_clock::time_point &user,
system_cpu_time_clock::time_point &system) noexcept;
};
#endif /* GDBSUPPORT_RUN_TIME_CLOCK_H */