[DynInst_API:] [dyninst/dyninst] 96f634: Remove lazy initialization of IBSTrees in Symtab (...

[Tim Haines <noreply@xxxxxxxxxx>]

  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: 96f6344d195bb74f30e209c48ee64c8397421dcd
      https://github.com/dyninst/dyninst/commit/96f6344d195bb74f30e209c48ee64c8397421dcd
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-07-24 (Mon, 24 Jul 2023)

  Changed paths:
    M symtabAPI/h/Function.h
    M symtabAPI/h/Symtab.h
    M symtabAPI/h/Variable.h
    M symtabAPI/src/Symtab-lookup.C
    M symtabAPI/src/Symtab.C

  Log Message:
  -----------
  Remove lazy initialization of IBSTrees in Symtab (#1454)

* Convert mod_lookup_ to a value type

This makes its existence a class invariant and makes Symtab::mod_lookup threadsafe.

* Fix includes in Function.h

so that it can be included in Symtab.h

* Fix includes in Variable.h

so that it can be included in Symtab.h

* Use Function.h instead of forward declarations

This removes duplicate typedefs, as well.

* Convert mod_lookup_ to a value type

This makes its existence a class invariant and makes its use in
Symtab::parseFunctionRanges threadsafe.