[DynInst_API:] [dyninst/dyninst] 0e73ae: Improve compiler diagnostic suppression handling


Date: Wed, 30 Mar 2022 12:49:15 -0700
From: kupsch <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 0e73ae: Improve compiler diagnostic suppression handling
  Branch: refs/heads/kupsch/fix-compiler-diagnostics
  Home:   https://github.com/dyninst/dyninst
  Commit: 0e73ae554040191a7525ecd92f830dfb257119b6
      https://github.com/dyninst/dyninst/commit/0e73ae554040191a7525ecd92f830dfb257119b6
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2022-03-30 (Wed, 30 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

- 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: 014a040f6a5376bc270b5f4da008e71f68523977
      https://github.com/dyninst/dyninst/commit/014a040f6a5376bc270b5f4da008e71f68523977
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2022-03-30 (Wed, 30 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

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


  Commit: 3b35c59e6bf60768e566c315bf7a6af79d94f26a
      https://github.com/dyninst/dyninst/commit/3b35c59e6bf60768e566c315bf7a6af79d94f26a
  Author: James A. Kupsch <kupsch@xxxxxxxxxxx>
  Date:   2022-03-30 (Wed, 30 Mar 2022)

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

  Log Message:
  -----------
  Fix frame-larger-than warning

- 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/0e73ae554040%5E...3b35c59e6bf6
[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] 0e73ae: Improve compiler diagnostic suppression handling, kupsch <=