[DynInst_API:] [dyninst/dyninst] 319513: Fix: BPatch_effectiveAddress to no longer truncate...


Date: Wed, 02 Jan 2019 16:24:02 -0800
From: Benjamin Welton <welton@xxxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 319513: Fix: BPatch_effectiveAddress to no longer truncate...
  Branch: refs/heads/welton_diogenes
  Home:   https://github.com/dyninst/dyninst
  Commit: 319513795d706380db4b18dcbb2150a1574067c5
      https://github.com/dyninst/dyninst/commit/319513795d706380db4b18dcbb2150a1574067c5
  Author: Benjamin Welton <welton@xxxxxxxxxxx>
  Date:   2019-01-02 (Wed, 02 Jan 2019)

  Changed paths:
    M dyninstAPI/src/inst-power.C

  Log Message:
  -----------
  Fix: BPatch_effectiveAddress to no longer truncate mem addresses on Power

In certain circumstances, BPatch_effectiveAddress would generate code that truncated
memory addresses to the bottom 32bits. The result is that instrimentation would see only
the bottom 32bits of any memory address.

Before this patch, the issue occured when instrimentation was applied to load/store instructions
where the address was located in registers r3-r11 (exclusive).

Ideally this function should just be straight removed in future releases (only used by emitASLoad).



      **NOTE:** This service has been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.
[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] 319513: Fix: BPatch_effectiveAddress to no longer truncate..., Benjamin Welton <=