Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Issue #35. Value.merge() assumed two values were equal even if the co…

…nstVal's weren't matching. Fixed, thanks to Andrew Bate.

	modified:   src/kilim/analysis/Value.java
  • Loading branch information...
commit 4a5cd35c5bab6b80b70e65bd2fec2a68e0b07247 1 parent 69dd807
@sriram-srinivasan sriram-srinivasan authored
Showing with 1 addition and 1 deletion.
  1. +1 −1  src/kilim/analysis/Value.java
View
2  src/kilim/analysis/Value.java
@@ -84,7 +84,7 @@ public Value merge(Value other) {
newType = D_UNDEFINED;
}
Object newConst = (constVal.equals(other.constVal)) ? constVal : NO_VAL;
- if (newNumSites != numSites || newType != typeDesc) {
+ if (newNumSites != numSites || newType != typeDesc || newConst != constVal) {
return new Value(newNumSites, newSites, newType, newConst);
} else {
return this; // no change
Please sign in to comment.
Something went wrong with that request. Please try again.