[DynInst_API:] commit: fixes for inline function bugs


Date: Wed, 24 Jun 2015 15:45:19 -0500
From: Bill Williams <bill@xxxxxxxxxxx>
Subject: [DynInst_API:] commit: fixes for inline function bugs
We were not, as a result of the memory optimizations I did this winter, handling inline functions' names correctly in Symtab. This has been fixed, and inlines should once again have correct names.

INTERFACE CHANGES involved in this fix:

* An InlineFunction and a FunctionBase both no longer inherit from Aggregate. Any user code that relies on downcasting Aggregates to these types will break. * FunctionBase is the preferred type for dealing with functions that may be symbol-backed or inlines, and should have a complete interface for doing so. * FunctionBase gets a getName() method that returns the first mangled (or only DWARF) name of a function, for convenience and consistency. FunctionBase does not have the Aggregate range-based methods for getting pretty/mangled/typed names. * Aggregates will no longer accept the addition of a name that is a substring of an existing name. In general, prefer adding symbols explicitly over adding names to aggregates.

There are also a few commits of minor code cleanup (dead code, mostly) in here, which should all be trivial.

--bw

Bill Williams
Paradyn Project
bill@xxxxxxxxxxx
[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] commit: fixes for inline function bugs, Bill Williams <=