Re: [PATCH v2] Add -ftime-report-wall

2024-10-10 Thread Eric Gallager
On Mon, Oct 7, 2024 at 6:27 AM Richard Biener wrote: > > On Sat, Oct 5, 2024 at 10:17 AM Andi Kleen wrote: > > > > From: Andi Kleen > > > > Time vars normally use times(2) to get the user/sys/wall time, which is > > always a > > system call. I don't think the system time is very useful because

Re: [PATCH v2] Add -ftime-report-wall

2024-10-09 Thread Andi Kleen
> So, shouldn't we go without the new option and simply change > -ftime-report behavior? I think it's fine (given the constraints I outlined earlier). It will slightly change the output, but I guess there aren't that many users that parse it mechanically. I can do that unless someoneelse objects.

Re: [PATCH v2] Add -ftime-report-wall

2024-10-07 Thread Richard Biener
On Sat, Oct 5, 2024 at 10:17 AM Andi Kleen wrote: > > From: Andi Kleen > > Time vars normally use times(2) to get the user/sys/wall time, which is > always a > system call. I don't think the system time is very useful because most > overhead > is in user time. If we only use the wall (or monoto

[PATCH v2] Add -ftime-report-wall

2024-10-05 Thread Andi Kleen
From: Andi Kleen Time vars normally use times(2) to get the user/sys/wall time, which is always a system call. I don't think the system time is very useful because most overhead is in user time. If we only use the wall (or monotonic) time modern OS have an optimized path to get it directly from a