[DynInst_API:] [dyninst/dyninst] e13918: Improve compiler diagnostic suppression handling (...


Date: Thu, 31 Mar 2022 05:35:41 -0700
From: kupsch <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] e13918: Improve compiler diagnostic suppression handling (...
  Branch: refs/heads/kupsch/fix-compiler-diagnostics
  Home:   https://github.com/dyninst/dyninst
  Commit: e13918d6e578ccd7fc21971985fc43df77c50b26
      https://github.com/dyninst/dyninst/commit/e13918d6e578ccd7fc21971985fc43df77c50b26
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2022-03-31 (Thu, 31 Mar 2022)

  Changed paths:
    A common/h/compiler_diagnostics.h
    M dyninstAPI/src/inst-x86.C
    M dyninstAPI_RT/h/dyninstAPI_RT.h
    M instructionAPI/src/Register.C

  Log Message:
  -----------
  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.


  Commit: d00aa5d2f5a6ea1f88c23f8d754afa4ee7f0694e
      https://github.com/dyninst/dyninst/commit/d00aa5d2f5a6ea1f88c23f8d754afa4ee7f0694e
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2022-03-31 (Thu, 31 Mar 2022)

  Changed paths:
    M common/h/Annotatable.h
    M common/h/Graph.h
    M common/h/Node.h
    M common/h/util.h
    M common/src/arch-x86.h
    M common/src/debug_common.C
    M common/src/ntHeaders.h
    M dataflowAPI/src/debug_dataflow.C
    M dyninstAPI/h/BPatch.h
    M dyninstAPI/h/BPatch_dll.h
    M dyninstAPI/h/BPatch_function.h
    M dyninstAPI/h/BPatch_image.h
    M dyninstAPI/h/BPatch_point.h
    M instructionAPI/h/BinaryFunction.h
    M instructionAPI/h/InstructionAST.h
    M instructionAPI/h/InstructionDecoder.h
    M parseAPI/src/debug_parse.C
    M proccontrol/src/handler.C
    M proccontrol/src/irpc.C
    M proccontrol/src/process.C
    M proccontrol/src/processplat.C
    M proccontrol/src/procset.C
    M proccontrol/src/windows_process.h
    M proccontrol/src/x86_process.C
    M symtabAPI/src/debug.C

  Log Message:
  -----------
  Remove MSC compiler suppression pragmas (#1239)

- These are likely no longer needed and will hide problems if MSC is is
  used in the future.


  Commit: 182084a1b175f718c5f8dbbaff87be23ab543198
      https://github.com/dyninst/dyninst/commit/182084a1b175f718c5f8dbbaff87be23ab543198
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2022-03-31 (Thu, 31 Mar 2022)

  Changed paths:
    M cmake/warnings.cmake
    M instructionAPI/CMakeLists.txt

  Log Message:
  -----------
  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


Compare: https://github.com/dyninst/dyninst/compare/ce032b92938a...182084a1b175
[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] e13918: Improve compiler diagnostic suppression handling (..., kupsch <=