[DynInst_API:] [dyninst/dyninst] cfbbc7: Correctly handle IP/PC for x86_64


Date: Mon, 03 Feb 2025 06:35:06 -0800
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] cfbbc7: Correctly handle IP/PC for x86_64
  Branch: refs/heads/thaines/rose_reg_x86
  Home:   https://github.com/dyninst/dyninst
  Commit: cfbbc734b5d01820376de2c10886d32960b1c07c
      https://github.com/dyninst/dyninst/commit/cfbbc734b5d01820376de2c10886d32960b1c07c
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/convert.C
    M dataflowAPI/rose/registers/x86_64.h

  Log Message:
  -----------
  Correctly handle IP/PC for x86_64


  Commit: 1ee396d62ec7c11455d4b96321f603e6284fbbb2
      https://github.com/dyninst/dyninst/commit/1ee396d62ec7c11455d4b96321f603e6284fbbb2
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/x86.h
    M dataflowAPI/rose/registers/x86_64.h

  Log Message:
  -----------
  Fix position flag for x86::X87


  Commit: fe55b33bb7b02b69501c66486e3a3e05810c220a
      https://github.com/dyninst/dyninst/commit/fe55b33bb7b02b69501c66486e3a3e05810c220a
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/x86.h

  Log Message:
  -----------
  Fix x86::x87 major version

The st_top is just for the top of the stack.


  Commit: 657e801916f126a320f4c9a38c39d365fd96b07b
      https://github.com/dyninst/dyninst/commit/657e801916f126a320f4c9a38c39d365fd96b07b
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/x86_64.h

  Log Message:
  -----------
  Fix x86_64::x87 major version

The st_top is just for the top of the stack.


  Commit: 909218817121bee83a0ab47fb606f68c745cc9b9
      https://github.com/dyninst/dyninst/commit/909218817121bee83a0ab47fb606f68c745cc9b9
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/x86.h

  Log Message:
  -----------
  Handle x86 kmask registers


  Commit: b7d0f4c2b761b498f8967fcf0f1b2c80afb1ebe4
      https://github.com/dyninst/dyninst/commit/b7d0f4c2b761b498f8967fcf0f1b2c80afb1ebe4
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/x86.h

  Log Message:
  -----------
  Use correct position flags for x86 vector registers

There is a non-obvious comment in the ROSE registers that the qword flag
is only meant for GPRs.


  Commit: a2b9b726714c643379ca205cf0b0e2eba5fe3bd4
      https://github.com/dyninst/dyninst/commit/a2b9b726714c643379ca205cf0b0e2eba5fe3bd4
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/x86_64.h

  Log Message:
  -----------
  Use correct position flags for x86_64 vector registers

There is a non-obvious comment in the ROSE registers that the qword flag
is only meant for GPRs.


  Commit: dd0439bc9253769603bdb22f15a52c67620e78bc
      https://github.com/dyninst/dyninst/commit/dd0439bc9253769603bdb22f15a52c67620e78bc
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/x86.h

  Log Message:
  -----------
  Use correct position flag for x86 segment registers

This was hidden in the ROSE register comments.


  Commit: c8d9d831619085ce19c8e3111b33a09a95f037eb
      https://github.com/dyninst/dyninst/commit/c8d9d831619085ce19c8e3111b33a09a95f037eb
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/x86_64.h

  Log Message:
  -----------
  Fix position flag for x86_64 segment registers

This was hidden in the ROSE register comments.


  Commit: f189e216629c7d6f3a1cd356c706683d22730314
      https://github.com/dyninst/dyninst/commit/f189e216629c7d6f3a1cd356c706683d22730314
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/convert.C

  Log Message:
  -----------
  Use x86_regpos_qword for rflags


  Commit: ad0c26f98570b2aafcdcbaab4afbf8a0838a6d9d
      https://github.com/dyninst/dyninst/commit/ad0c26f98570b2aafcdcbaab4afbf8a0838a6d9d
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/x86.h
    M dataflowAPI/rose/registers/x86_64.h
    M dataflowAPI/src/ExpressionConversionVisitor.C

  Log Message:
  -----------
  Use x86_regpos_dword for x86/x86_64 segment register

x86_regpos_all was being treated as a synonym, but this makes it clear
what the actual size is.


  Commit: fe75f0b1cd58aa4724479902046c4e94754539b1
      https://github.com/dyninst/dyninst/commit/fe75f0b1cd58aa4724479902046c4e94754539b1
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/x86_64.h

  Log Message:
  -----------
  Use x86_regpos_qword for x86_64 64-bit registers


  Commit: 60eb465b9d96112a78d146335aa7a2d6fb2674d0
      https://github.com/dyninst/dyninst/commit/60eb465b9d96112a78d146335aa7a2d6fb2674d0
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/x86.h

  Log Message:
  -----------
  Use x86_regpos_dword for 32-bit x86 registers


  Commit: a8c7b428fb4f8aa064d98c9465fb7fbdc05d99fd
      https://github.com/dyninst/dyninst/commit/a8c7b428fb4f8aa064d98c9465fb7fbdc05d99fd
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/x86.h
    M dataflowAPI/rose/registers/x86_64.h

  Log Message:
  -----------
  Make x86/x86_64 kmask 64-bit


  Commit: a034f4f0037c782fd038d75e26ead0dcb3613776
      https://github.com/dyninst/dyninst/commit/a034f4f0037c782fd038d75e26ead0dcb3613776
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/convert.C

  Log Message:
  -----------
  Preserve flag register parts


  Commit: 0ddbad262c47be3524998db0de5a6e170f5bee94
      https://github.com/dyninst/dyninst/commit/0ddbad262c47be3524998db0de5a6e170f5bee94
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M dataflowAPI/rose/registers/convert.C

  Log Message:
  -----------
  Fix comment


  Commit: 03838c5b0f6513871d30a3621df603f606f98e8b
      https://github.com/dyninst/dyninst/commit/03838c5b0f6513871d30a3621df603f606f98e8b
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M tests/CMakeLists.txt
    A tests/dataflowAPI/CMakeLists.txt
    A tests/dataflowAPI/rose/CMakeLists.txt
    A tests/dataflowAPI/rose/registers/CMakeLists.txt
    A tests/dataflowAPI/rose/registers/rose_reg_check.h
    A tests/dataflowAPI/rose/registers/x86.cpp
    A tests/dataflowAPI/rose/registers/x86_64.cpp

  Log Message:
  -----------
  Add unit tests


  Commit: 5b137c8329819160a5ac40d97b5464e243ac1e91
      https://github.com/dyninst/dyninst/commit/5b137c8329819160a5ac40d97b5464e243ac1e91
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M tests/dataflowAPI/CMakeLists.txt

  Log Message:
  -----------
  cmake formatting


  Commit: a9463c2e7b257eaee6dcddff9d8760cb5a465423
      https://github.com/dyninst/dyninst/commit/a9463c2e7b257eaee6dcddff9d8760cb5a465423
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2025-02-02 (Sun, 02 Feb 2025)

  Changed paths:
    M tests/dataflowAPI/rose/registers/x86.cpp
    M tests/dataflowAPI/rose/registers/x86_64.cpp

  Log Message:
  -----------
  Update flag and segment registers


Compare: https://github.com/dyninst/dyninst/compare/945f3c71805f...a9463c2e7b25

To unsubscribe from these emails, change your notification settings at https://github.com/dyninst/dyninst/settings/notifications
[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] cfbbc7: Correctly handle IP/PC for x86_64, Tim Haines <=