[DynInst_API:] [dyninst/dyninst] 428f76: Fix overflow of fields in instruction layout


Date: Thu, 10 Aug 2023 07:28:32 -0700
From: bbiiggppiigg <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 428f76: Fix overflow of fields in instruction layout
  Branch: refs/heads/fix-layout-field-overflow
  Home:   https://github.com/dyninst/dyninst
  Commit: 428f7641500b04a5ec53142ab18599f771f21b4f
      https://github.com/dyninst/dyninst/commit/428f7641500b04a5ec53142ab18599f771f21b4f
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2023-08-10 (Thu, 10 Aug 2023)

  Changed paths:
    M instructionAPI/src/AMDGPU/gfx908/amdgpu_gfx908_decoder_impl.h
    M instructionAPI/src/AMDGPU/gfx90a/amdgpu_gfx90a_decoder_impl.h
    M instructionAPI/src/AMDGPU/gfx90a/amdgpu_gfx90a_insn_entry.h

  Log Message:
  -----------
  Fix overflow of fields in instruction layout

Fields in instruction layouts can have implicit LSBs.

Previously this wasn't taken into consideration,
and causes overflow and decoding instruction.

This PR fixes the layout definition to include the LSBs.


[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] 428f76: Fix overflow of fields in instruction layout, bbiiggppiigg <=