ELF+DWARF: always parse first entry in source file table (#1184)

rocm-4.5 starts to emit AMD GPU binaries with DWARF 5. I noticed missing source line information for these binaries. The root cause is that dyninst assumes that source file table entry 0 represents "unknown file". This no longer seems to be true in DWARF 5, where entry 0 can represents a real application source file.

Remove interposed definition of _r_debug (#1176)

Previously there was a dynamic relocation against _r_debug in the loader which picked up the interposed definition,
but glibc now uses a direct internal hidden symbol reference and thus no longer updates the interposed object.

start of work to add automated testing to dyninst! (#1183)

start of work to add automated testing to dyninst!

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
Co-authored-by: vsoch <vsoch@users.noreply.github.com>

adding dyninst release trigger (#1181)

to release a tagged (versioned) container on publication of a release we just
need to add the release -> published trigger, derive the tag from the GITHUB_REF environment
variable, and then tag and push the container

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
Co-authored-by: vsoch <vsoch@users.noreply.github.com>

fixing workflow trigger (#1182)

we do not need the build on PR, but we do need on merge into MASTER not MAIN

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
Co-authored-by: vsoch <vsoch@users.noreply.github.com>

Adding first stage of automated docker build (#1180)

* adding first stage of automated docker build

this first workflow include a base container with ubuntu 20.04 and
a set of dependencies chosen by Tim. The base container will be set (after this PR) as a scheduled, nightly build, and we do this so testing each day can simply pull it, re-add the new source code and testsuite, and quickly compile again (on the order of 3.5 minutes vs. 1 hour+ for the entire container build. This update will happen in two stages - this first to deploy the base container, and a follow up to add automated testing using it. Currently this does not upload test results to Tims server, and this is only because I want to talk to him about it first and then test the workflow before PR here.

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
Co-authored-by: vsoch <vsoch@users.noreply.github.com>

Remove erroneous use of realloc in symtabAPI/Type-mem.h (#1170)

Clean up memoryTracker usage in binaryEdit (#877)

* Store memoryTracker_ as a value instead of pointer

This provides cleaner semantics for its usage and ensures that the it's
default-constructed correctly.

* Code cleanup

* Give codeRange a virtual destructor

This will ensure that memoryTracker objects removed from a codeTree will
correctly get destroyed.

* Simplify inferiorFree

* Free any remaining memoryTrackers on destruction

Documentation updates for 12.0.1 release

Remove NVIDIA external line map configure check (#1162)

Now that we require elfutils >= 0.186, this is no longer needed.

Increase minimum elfutils version to 0.186 (#1161)

This is needed to parse NVIDIA's extended line map information.
HPCToolkit makes extensive use of this functionality.

Refactor dwarfWalker::findConst (#1160)

This function always returns 'true' and few callsites checked the bool
out variable. Using an optional here makes everything simpler. Logging
has been substantially improved for better tracing.

Add readable name for Symtab::typeRef (#1157)

Arguably, this is completely arbitrary as it could have been spelled
'ref', 'reference', etc. Using the language symbol also allows users to
reconstruct source-level syntax more directly.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

DwarfWalker: clean up interfaces for findDieName and findName (#1154)

findDieName always returned 'true', so any checks for its return value
were meaningless. It was also not clear that 'findName' would modify its
argument. The new interface removes that surprise by making the name the
return value.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Add conflict with CMake 3.19.0 (#1153)

There is a bug in 3.19.0 that treats .S files like C-language files.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Documentation update for 12.0.0 release (#1151)

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Add CMake test to check if libdw supports NVIDIA extended line map

Adjust interface changes in elfutils regarding NVIDIA extended line map

Add compile-time checking to see if elfutils support nvidia extended line map
when the user have specified ENABLE_NVIDIA_EXT_LINE_MAP

Fix compilation warning and add cmake option ENABLE_NVIDIA_EXT_LINE_MAP

1. Handle unrelocated line map entries for CUBIN
2. Remove redundant addFunctionRange call to improve performance
3. Add some debug logging

Inline context from nvidia extended line map identifies an inlined call path

Start to construct inlining call chains using Nvidia's extended line map

cleaning up code for ingesting nvidia extended linemaps

first draft of support for nvidia enhanced line maps

Improve docs for lookup functions in CodeObject (#1147)

Add wording to improve clarity about when a CodeRegion is strictly needed when using the lookup functions.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Remove AddressSpace::causeTemplateInstantiations (#1149)

This was removed by 47c4dfe96 in 2010.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Remove unregisterTrapMapping from PCProcess

This was disabled in 2013 by fce999f48.

Remove thread registration functions from PCProcess

These were disabled in 2015 by 0e791cce8.

Remove PCProcess::getDeadCode

This was disabled by ab24c9bf0 in 2012. Because it was replaced with an
'assert(0)', it's safe to say no one has used it since it was disabled.

Remove memory emulation (#1146)

This was disabled by 89919d982 in 2012.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Remove ppc32 from builds (#1145)

* Remove ppc32 from build system

* Remove ppc32 source and header files

* Clean up gitignore in parseThat

* Remove use of arch_ppc_little_endian

* Remove explict use of ppc32_linux macro

* Remove implicit use of ppc32 macro

There are likely more instances.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

fix statement-like macros (#1143)

wrap macro body in 'do {...} while (0)' so they can be used as if they were a
statement with a terminating semicolon.

changed break to return since it then works in a do/while loop and was
semantically the same in all the expansions

parenthesized the use of parameters as needed

Implement ppc64 in MachRegister::getROSERegister (#1139)

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Update copyright to 2022 (#1141)

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Unify meaning of 'cap_32_64' macro (#1136)

* Unify meaning of 'cap_32_64' macro

Currently, this is used in several different ways due to the historical
original meaning drifting over the years. It's meaning is now "this host
64-bit platform supports modifying 32-bit binaries". Currently, this is
only x86 and PPC32. Although the latter is going away soon due to
obsolescence.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Remove support for Cray CNL (#1137)

The Compute Node Linux (CNL) was for the old XT/XE platforms that
are obsolete now. This should have been removed by e5484368f in
2021. Support for this platform was removed from the test suite by
7e4ab7c12 in 2012.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Don't overflow aarch64 float register vector when setting used regs. (#1127)

Do not include the subtype when setting a float register as a used register if the registerSlot vector would be exceeded, e.g. for a value like 0x400 (Q_REG/register 0).

Remove xlc macros (#1132)

* Remove unused XLC macro in dyntypes.h

* Remove XLC check in BPatch_snippet/BPatch_effectiveAddressExpr

This was added by c604218d5 in 2004 for compiling dyninst with xlC as a
static library for AIX compatibility with DPCL- neither of which is
supported by Dyninst now.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Remove common/src/language.h (#1131)

This is never used anywhere. We also require a compiler that supports 'typename' implicitly.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Remove usage of arch_ppc and arch_ppc64 (#1129)

These were removed by f33ba7b in 2013 and should have been converted to
arch_power at that time.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

fix unused const variable warnings

fix pessimizing std::move warnings

fix xor operator used as power operator

2^32  should be  1LL << 32

fix misleading indentation warning

fix uninitialized this and variable warnings

fix float to double promotion warning

fix unused const variable warnings

Remove usage of x86_64_cnl (#1130)

This was added but never used.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Load callee's address when the callee and caller are in the same module (#1056)

If the callee and caller are in the same module and pic is not
required then the callee's address can be loaded directly without
using a relocation.

Remove DynC tests (#1126)

These have been moved to the examples repository.
https://github.com/dyninst/examples/pull/25

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Remove NO_INITIALIZER_LIST_SUPPORT (#1125)

Fixes #1124

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Remove unused generateSimple (#1122)

This is never called in Dyninst. It also has broken logic.

Fixes #1087

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Remove unused variables from Symtab

This is an ABI-breaking change.

Fixes #1062

Fix possible buffer overflow in BPatch::processCreate

Give global annotation objects internal linkage and file scope

This was technically an ODR violation because they are in two different
translation units that _could_ be linked together. However, I don't
think they ever were. This fix will preclude any such problem, though.

Fix uninitialized variable use in DispatcherARM64::iproc_init

Remove stabs from documentation (#1120)

This should have been part of #1113

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Turn on STERILE_BUILD by default (#1118)

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

Remove special Fortran debug handling

lang_Fortran_with_pretty_debug was only for Fortran with stabs.

Remove stabs from symbol demangling

Remove stabs from BPatch

Remove stabs from SymtabAPI

update minimum boost version to 1.70.0 (#1117)

Remove boost_system linking (#1112)

boost_system is header-only in 1.69.0 which is the min version required by dyninst.

Signed-off-by: sdlzx <hdu_sdlzx@163.com>

Summit fixes (#1108)

* Replace assert with throw exceptions

* fix conditional branch instruction conversion to Rose instruction for ppc64le

Enforce detection of libiberty (#1099)

This correctly requires that libiberty be present and removes building it from source. Both of these were bugs introduced by 08097f23.

Co-authored-by: Tim Haines <thaines@cs.wisc.edu>

remove executable flag from .dyninst_heap section (#1096)

This prevents the library/object with this code from having a segment violates
the W^X security property for segments as the current setting resulted in
segment with both the WRITE and EXECUTE flags set.  The function
mark_heaps_exec() restores execute protection to just the heap arrays.

fix compiler warnings to work with clang (#1092)

- detect warning flag test failure on clang correctly by updating the failure
  regular expression to include clang's unknown warning flag message

- fix preprocessor macro tests for scoped attributes in C

update optimization (-Og) and debug flags (-g3) (#1084)

use the C11 standard for C code in Dyninst (#1086)

fix broken cast of a char literal to pointer (#1090)

The code should have been a string literal not a char literal.

fix possibly uninitialized variables (#1082)

make method always assign the parameters even in the event of errors,
and check for errors and do not use them if an error occurred.

This diagnostic only occurs with -O3.  In practice this is a false
positive as this should not occur with a valid arm instruction.

fix possible null 'this' pointer dereference (#1082)

If a nullptr is passed to the second parameter of the Field class's constructor
then there is a nullptr this pointer dereference for the boost::shared_ptr that
is the second parameter as the reshare() method is called.  The second
parameter is a nullptr in BPatch_type::getComponents is the type is and enum
type.  This warning is only diagnosed with -Og (and maybe -O1).

prevent maybe uninitialized warning (#1082)

This is a false positive with -Og.  Initialize the variable to 0 to prevent
the warning.  The value is only used if a boolean is true and if it is true
then the value is set.

adjust large frame threshold for specific sources (#1082)

fix deprecated implicit assignment operator (#1082)

remove (default equivalent) copy constructors, so all implicitly
generated operators are defaulted.  THey should all be default or
explicitly defined

fix buffer overflow (#1082)

fix duplicate branch condition by removing branch (#1082)

Commented out else if condition and then block.  else if condition
happens to be identical to the if condition due to constant identifiers
having the same value.

The tests should be made more specific to distinguish the two cases or
the commented out else if should be removed.

fix out of bounds array access (#1082)

use ->uc_mcontext.regs->nip which follows the 32 array entries of the
same type in ->uc_mcontext.regs->gpr[] instead of access gpr[32] which
happens to work

fix potentially uninitialized variable warning (#1082)

use unused variable to correct code (#1082)

remove unused variables (#1082)

make printf format and argument types match (#1082)

fix broken bool expression that was always true (#1082)

add missing initializer braces (#1082)

Should be OK without extra brace, but early C++11 required it, and some
versions of gcc complain with -Wmissing-braces

make constructor public so class is usable (#1082)

remove ';' after in-class method definitions (#1082)

eliminate logical op warning (#1082)

gcc warns about clauses to a logical operation being identical.
They are identical comparisons but the values are from different
macros and/or enum values.

make implicit double promotions explicit (#1082)

annotate malloc-like functions (#1082)

make method noexcept, so noexcept expr can be true (#1082)

add missing default to switch statement (#1082)

fix int to void* cast if sizeof(int)<sizeof(void*) (#1082)

eliminate conversion of NULL to non-pointer type (#1082)

fix variable signedness (#1082)

replace if stmt with identical branches with then stmt (#1082)

fgetc returns an int not a char (#1082)

do not discard volatile type qualifier in cast (#1082)

add missing #include <assert.h> (#1082)