RE: [PATCH v2 3/3] gdb/jit: enable tracking multiple jitter objfiles

["Aktemur, Tankut Baris" <tankut.baris.aktemur@intel.com>]

On Friday, July 3, 2020 4:20 AM, Simon Marchi wrote:
> On 2020-06-30 4:17 a.m., Aktemur, Tankut Baris wrote:
> > Hi Simon,
> >
> > On Sunday, June 21, 2020 5:33 AM, Simon Marchi wrote:
> >> Please tell me what you think about this.  If you agree with the direction, I could
> >> officially post it to the list.
> >
> > Any plan to post the series these days?
> >
> > -Baris
> 
> Arf, I don't think I'll have time to polish and post it soon :(.  If this is on your
> critical
> path, would you mind doing it?

Sure, I'll work on it.

> One thing I didn't like and wanted to re-visit before posting is the fact that in
> jit_breakpoint_re_set_internal, we always loop over all objfiles.  This means that when a
> new
> objfile appears and we call jit_breakpoint_re_set_internal, we look up the magic symbols in
> all
> the previously existing objfiles, even we have already looked them up earlier.  If we
> haven't
> found the symbols in a a given objfile before, there's no need to search in that objfile
> again.

I'll look into this as well.

Thanks.
-Baris


Intel Deutschland GmbH
Registered Address: Am Campeon 10-12, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de
Managing Directors: Christin Eisenschmid, Gary Kershaw
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928