[DynInst_API:] [dyninst/dyninst] 49e7cd: The argument to SymEvalSemantics::addWithCarries f...


Date: Sun, 18 Sep 2016 16:45:41 -0700
From: Sunny Shah <shah28@xxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 49e7cd: The argument to SymEvalSemantics::addWithCarries f...
  Branch: refs/heads/arm64/feature/semantics
  Home:   https://github.com/dyninst/dyninst
  Commit: 49e7cd96b6ba4e6db7607156cf6f126c2492021c
      https://github.com/dyninst/dyninst/commit/49e7cd96b6ba4e6db7607156cf6f126c2492021c
  Author: Sunny Shah <shah28@xxxxxxxx>
  Date:   2016-09-18 (Sun, 18 Sep 2016)

  Changed paths:
    M dataflowAPI/rose/semantics/SymEvalSemantics.C

  Log Message:
  -----------
  The argument to SymEvalSemantics::addWithCarries for unsign-extending
the second addend should be the second operand from the instruction AST.

The bug here was that the first operand from the AST was being passed
for the unsign-extend operation. This caused the semantic expression and
hence the resulting assignment expansion for the instructions calling
this function to be incorrect.


[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] 49e7cd: The argument to SymEvalSemantics::addWithCarries f..., Sunny Shah <=