Docker: make build.sh verbose, fix file copy bug in Dockerfile.test (#1273)

- remove old code before copying code into testing container
- build with verbosity in build.sh (this helps when build errors occur)

Remove endian CMake check (#1270)

The usage of the macros was removed by d233ae759 in 2020.

Update and enforce formatting of CMake files (#1267)

Co-authored-by: Jonathan R. Madsen <jrmadsen@users.noreply.github.com>

Fix bug in processing library paths in FindLibIberty.cmake (#1266)

This was originally part of #1263

Co-authored-by: Jonathan R. Madsen <jrmadsen@users.noreply.github.com>

Fix decoding of DWARF expressions into machine registers for Intel GPUs (#1262)

We ignore the encoding as we do for all of the other GPUs. This case was missing and so hit the assert.

Docker: don't use autamus cache (#1259)

The public keys have been removed.

Remove unused build options (#1253)

* Remove BUILD_DOCS

This isn't sufficient to create the documentation as some of it is
contained in MS Word files.

* Remove BUILD_TARBALLS

We don't distrubute tarballs anymore. That's handled by GitHub.

Fix dyninstAPI_RT files to build with older glibc (#1252)

Replace feature test macro _DEFAULT_SOURCE with _GNU_SOURCE to support older version of glibc

Redo finalization to get correct function boundiaries when (#1249)

there are many tail call correction

Fix format string errors in stackwalk/callchecker.C (#1250)

These are only present when SW_ANALYSIS_STEPPER=OFF.

Make dyninstAPI_RT files build with standard C (#1246)

- use same language standards as the rest of Dyninst
- define _DEFAULT_SOURCE in some dyninstAPI_RT source files so functions
  and macros used are defined when using standard C

Add cmake options for C/C++ language standards (#1246)

Add cmake options to set C/C++ language standard versions used to build
to facilitate testing.

- DYNINST_C_LANGUAGE_STANDARD cmake option:  C Standard version
- DYNINST_CXX_LANGUAGE_STANDARD cmake option:  C++ Standard version

Use bfd linker for LTO (#1248)

The gold linker crashes and is no longer supported.

Cleanup (remove) ancient linux kernel support (#1241)

- Remove check and message for ancient linux kernel
- Remove support for old mechanism to find process's tasks

remove unused files containing pragmas (#1240)

Remove unneeded #pragma's (#1240)

- Remove once and interface pragmas
- Replace use of "external/stdint-win.h" and "external/inttypes.h" with
  <stdint.h> and <inttypes.h>

Add compiler warning related cmake options (#1239)

- DYNINST_WARNINGS_AS_ERRORS option:  if set, treat warnings as errors
- DYNINST_EXTRA_WARNINGS option:  additional warning options to test for
  and use if valid with the current compiler

Add cmake option to disable diagnostic suppressions (#1239)

- DYNINST_DISABLE_DIAGNOSTIC_SUPPRESSIONS option:  if set,
  disable all warning suppressions and frame size limit overrides

Fix frame-larger-than warning (#1239)

- Increase frame size max when using gcc 6 for non-debug builds and
  for all debug builds (needed for rhel's gcc) to compile
  instructionAPI/src/AMDGPU/cdna2/InstructionDecoder-amdgpu-cdna2.C

Remove MSC compiler warning suppressions (#1239)

Likely no longer needed with current MSC compiler and will hide
problems if MSC is used.

- Removed diagnostic suppressing pragmas
- Removed compiler command line suppression options

Improve compiler diagnostic suppression handling (#1239)

- Create compiler specific diagnostic suppression macros to suppress a
  type of warning for a region of code that are consistently defined
  based on the build environment.
- Replace current multi-line preprocessor #if and #pragma statements
  with one macro.
- Add suppression for warnings in instructionAPI/src/Register.C
  when using gcc 6-8.

Remove unused git files (#1244)

.gitmodules hasn't been used since the test suite was moved to its own repository.
.github_changelog_generate hasn't been used since v10.0.0

Docker: make compile warnings fatal (#1242)

Docker: use external-tests instead of testsuite in base image (#1209)

This is in preparation for using -Werror when building Dyninst in the Github workflow. The test suite does not currently build cleanly, so we need a different set of tests.

Docker: don't build Dyninst through spack for the environment (#1222)

This results in two installations: one in /opt/dyninst-dev/.spack/include and one in /opt/dyninst-dev/install/dyninst. The former is not updated when testing a PR, so causes public header conflicts.

Docker: use more OS packages for dependencies (#1221)

* Docker: use more OS packages for dependencies

This should have been part of #1211.

Remove void pointer arithmetic when using Valgrind annotations (#1236)

gcc allows this as an extension by considering sizeof(void) to be 1 (http://gcc.gnu.org/onlinedocs/gcc/Pointer-Arith.html).

Add parsing of names for inlined functions in DWARF (#1237)

* Clean up and enhance debug messages
* Add parsing of names for inlined functions

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

Remove BUILD_RT option (#1238)

When not enabled, the dyninstAPI_RT/cmake_install.cmake isn't present.
This has been broken since it was implemented by e1afc6918 in 2016.

Fix warnings with cmake's MINSIZEREL build type (#1235)

- Explicitly define optimization flags for cmake's MINSIZEREL build
  type.  Cmake by default includes -DNDEBUG.  This results in assert
  being defined to do nothing.  Dyninst use asserts as a fatal error
  reporting mechanism and expects assert to not return if the condition
  is always false.  If assert can return then many warnings are
  produced due to this unexpect path.
- Remove defining NDEBUG for REL build types if using the MSC compiler

Correctly propagate pc ranges for blocks and local variables (#1226)

- fix valid pc address ranges for local variables that are declared in a
  sub-block (brace, try and catch blocks) of their function
- fix Context class's constructors so they correctly initialize and copy
  all member (use in-class initialize and default constructors)

Docker: testing workflow to run libabigail (#1220)

Run libabigail's abidiff to detect ABI breakages between the current PR and the last successful build as well as the current PR and the last release.

* testing workflow to run libabigail!
I am not sure if the artifact is extracted relative or not, so will need to update
the workflow to account for that. This is also a new strategy that will try to
do the new build/retrieval of artifacts from previous containers - have not
tried this yet!

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
Co-authored-by: vsoch <vsoch@users.noreply.github.com>
Co-authored-by: Tim Haines <thaines.astro@gmail.com>

Remove usage of DW_AT_MIPS_linkage_name (#1223)

This isn't part of DWARF4 or DWARF5.

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

Docker: adding a workflow for release (#1219)

Currently we do the entire build from scratch for release, and this strategy was chosen
for the cleanest build. However we can use the same strategy as we do for testing
and take advantage of the base container. This PR adds a workflow to do that

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

Merge pull request #1217 from dyninst/thaines/docker_base_container_defaults

Docker: update dependency versions in base container config

Update dependency versions in base container config

This should have been part of #1211

Documentation update for v12.1.0 (#1216)

Correctly handle DWARF subroutines during parallel parsing (#1215)

Fix to only process the DWARF for a function (subprogram) once if it is defined
in multiple compilation units within a library or executable.  This prevents
duplication of parameters, local variables and use of thread-unsafe data
structures from concurrent threads.

Docker: simplify build script (#1212)

* Use CMake's -S instead of undocumented -H

* Remove log captures

The reports are best used within the context of the PR, so storing them
in the dashboard wouldn't be much use.

* Reduce duplication of directory names

* Use CMake commands to build/install

* Separate build and install trees for Dyninst

* Separate build and install directories for test suite

* Simplify 'git pull' for test suite

* Explicitly override test suite install directory

The test suite reads Dyninst's cache which poisons the CMAKE_INSTALL_PREFIX. This overrides that and then puts the test files in the root install directory.

Make a couple constant 64-bit ULL to avoid getting truncated during shifts (#1207)

* Ensure ROR work with data types larger than 32 bits.

* Ensure upper 32-bits of 64-bit aarch64 SIMD operand generated.

Docker: use more OS packages (#1211)

* Use perl from OS package

* Use CMake from OS package

* Use Boost from OS package

* Use TBB from OS package

* Mark all external packages as not buildable

Fix potential buffer overrun in AMDGPU decoders (#1208)

* add assert to check for potential buffer overun
* add assert to check for potential buffer overrun in vega

Code clean up for AMDGPU (#1205)

* Update the code layout for vega headers

* Remove unused cda headers

* remove duplicate cdna2 headers in instructionAPI

Tidy up classes in AMDGPU (#1204)

* Remove dead code in makeAmdgpuRegID

* Fix possible no-return cdna2 functions

* Simplify code in bitfieldInsnAliasMap and condInsnAliasMap

* Correctly initialize all members of InstructionDecoder_amdgpu_cdna2

* Inline InstructionDecoder_amdgpu_cdna2 constructor

* Explicitly default ~InstructionDecoder_amdgpu_cdna2

* Fix possible non-return in decodeSGPRorM0

Fix compiler warnings in amdgpu cdna2 code (#1198)

* Fix 'unused-variable' warnings from gcc

* Fix 'unused-parameter' warnings from gcc

Build fixes for amdgpu/cdna2 (#1203)

* Clean up duplicate cdna2 sys_reg headers (#1200)
* add install rule to install amdgpu headers under common/h (#1201)

Co-authored-by: bbiiggppiigg <bbiiggppiigg@gmail.com>

Add Support for AMDGPU CDNA2 Architectures based on XML ISA spec (#1107)

Add Initial Support for cdna based on the lastest version of XML file ( 20210720 )

Added new register names based on the latest xml-isa drop

Decoder implementation updated based on the latest xml-isa drop

renumbered registers to make low registers have the same value as full register (or its super set)

add larger memoery types for representing consecutive registers used as a single operand

Modify the format function to output multiple registers as a single operand correctly, handling for certain registers are still being worked on

split register-vector into multiple registers

fix building of symlite (#1197)

- add missing pure virtual functions isReadOnly
- fix isCode and isData methods to use program header permissions instead
  of just returning true
- fix printf compiler warning
- disable parseThat if using symlite as it is not compatible
- remove #include <linux/elf.h> from proccontrol/src/linux.h on aarch64
  if SYMLITE is configured as it is incompatible with <elf.h>

Additional cleanup of memory emulation for hybrid analysis (#1172)

* Remove vestiges of runtime functions
* Remove translation for emulation shadow pages

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

Allow zero-length ELF program headers (#1192)

CUDA-11.6 binaries do not have program headers, which causes ELF parsing to stop early and miss line info

Remove dead code and variables related to Symtab::data_ptr_ and Symtab::code_ptr_ (#1192)

Remove dead code Object::elf_vaddr_to_ptr (#1192)

glibc r_debug extensions to support multiple namespaces (#1175)

* Remove interposed definition of _r_debug

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.

* Update the elf _DYNAMIC symbol

When the symbol table is created in emitElf<ElfTypes>::createSymbolTables, also update the elf _DYNAMIC symbol to reflect the new address of the .dynamic section.

Symtab::module documentation cleanup (#1189)

* Remove docs for hasLineInformation

This was removed by 0dee1c8 in 2016

* Remove getAllVariables

This was by b3ad134 in 2021. It was removed there because this function
was never implemented.

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

Dockerfile: use explicit packages for 'spack external find' (#1195)

currently gettext in ubuntu 20.04 is being added as an external dependency (just added about 24
hours ago) and it is breaking the build. This change will test removing from the OS (probably
not recommended but okay in the context of the Dyninst image)

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

ParseAPI: Speed up the case where a function is called from many locations (#1190)

In some binaries a single function has many callers, so copying out the source edges for
the entry Block takes a while and causes significant contention on the Block's lock.
The case in Parser::finalize where this becomes a hotspot likely does not need to read
every edge, so this commit inlines those cases into methods on Block instead.

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