Date: | Sun, 18 Sep 2016 16:47:16 -0700 |
---|---|
From: | Sunny Shah <shah28@xxxxxxxx> |
Subject: | [DynInst_API:] [dyninst/dyninst] 82a571: The argument to SymEvalSemantics::addWithCarries f... |
Branch: refs/heads/master Home: https://github.com/dyninst/dyninst Commit: 82a57100bb9adcc2f633f008dd94cb5fb94d8901 https://github.com/dyninst/dyninst/commit/82a57100bb9adcc2f633f008dd94cb5fb94d8901 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→] |
---|---|---|
|
Previous by Date: | [DynInst_API:] [dyninst/dyninst] 49e7cd: The argument to SymEvalSemantics::addWithCarries f..., Sunny Shah |
---|---|
Next by Date: | Re: [DynInst_API:] Directly reassemble and patch instructions, Bill Williams |
Previous by Thread: | [DynInst_API:] [dyninst/dyninst] 80eeb3: Fix for #116, John Detter |
Next by Thread: | [DynInst_API:] [dyninst/dyninst] 833960: symtabAPI: use the known type for new relocations, Bill Williams |
Indexes: | [Date] [Thread] |