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