[DynInst_API:] [dyninst/dyninst] 6275fb: Fix overflow of fields in instruction layout


Date: Wed, 09 Aug 2023 00:38:53 -0700
From: bbiiggppiigg <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 6275fb: Fix overflow of fields in instruction layout
  Branch: refs/heads/fix-layout-field-overflow
  Home:   https://github.com/dyninst/dyninst
  Commit: 6275fb932f0c02b5566ff56c3e5d8ba87430f69a
      https://github.com/dyninst/dyninst/commit/6275fb932f0c02b5566ff56c3e5d8ba87430f69a
  Author: wuxx1279 <bbiiggppiigg@xxxxxxxxx>
  Date:   2023-08-09 (Wed, 09 Aug 2023)

  Changed paths:
    M instructionAPI/src/AMDGPU/gfx908/amdgpu_gfx908_decoder_impl.h
    M instructionAPI/src/AMDGPU/gfx90a/amdgpu_gfx90a_decoder_impl.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] 6275fb: Fix overflow of fields in instruction layout, bbiiggppiigg <=