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→] |
---|---|---|
|
Previous by Date: | [DynInst_API:] [dyninst/dyninst] 16ccee: When determining the ROSE register category for AR..., Sunny Shah |
---|---|
Next by Date: | [DynInst_API:] [dyninst/dyninst] 82a571: The argument to SymEvalSemantics::addWithCarries f..., Sunny Shah |
Previous by Thread: | [DynInst_API:] [dyninst/dyninst] 49accc: fix icc warning flags, Peter Foley |
Next by Thread: | [DynInst_API:] [dyninst/dyninst] 4a5008: Wrap stackanalysis in a try-catch per function, an..., Bill Williams |
Indexes: | [Date] [Thread] |