summaryrefslogtreecommitdiff
path: root/chapter17/src/main/java/com
diff options
context:
space:
mode:
authorCliff Click <cliffc@acm.org>2026-09-25 15:28:21 -0700
committerCliff Click <cliffc@acm.org>2026-09-25 15:28:21 -0700
commit29e0127170dce8964d7f85d808d1ec07426d2789 (patch)
treef9174685beaf274f3324a08c68f2b0a7137309c2 /chapter17/src/main/java/com
parent12bde12b2e18c8b82b1510ac4305de66d1205e1a (diff)
downloadSimple-29e0127170dce8964d7f85d808d1ec07426d2789.tar.gz
Simple-29e0127170dce8964d7f85d808d1ec07426d2789.zip
Restrict subtraction zero peepholes to integer operandspr254-fixes
Require resolved integer mode in chapter 25 so zero subtraction preserves floating-point signed zero. Keep nonnumeric operands in the graph for subtraction diagnostics. Add signed-zero coverage from chapter 12 and null-subtraction diagnostics from chapter 22. Confirmed the regressions fail on the original PR. Full Make suites pass in chapters 4-25 (5462 tests), including fresh chapter 25 sys.o compilation.
Diffstat (limited to 'chapter17/src/main/java/com')
-rw-r--r--chapter17/src/main/java/com/seaofnodes/simple/node/SubNode.java10
1 files changed, 5 insertions, 5 deletions
diff --git a/chapter17/src/main/java/com/seaofnodes/simple/node/SubNode.java b/chapter17/src/main/java/com/seaofnodes/simple/node/SubNode.java
index 044d0c62..2f2a7a05 100644
--- a/chapter17/src/main/java/com/seaofnodes/simple/node/SubNode.java
+++ b/chapter17/src/main/java/com/seaofnodes/simple/node/SubNode.java
@@ -47,11 +47,11 @@ public class SubNode extends Node {
Type t1 = lhs._type;
Type t2 = rhs._type;
- if( t2.isConstant() && t2 instanceof TypeInteger i && i.value()==0 )
- return lhs;
-
- if( t1.isConstant() && t1 instanceof TypeInteger i && i.value()==0 )
- return new MinusNode(rhs);
+ // Keep invalid operands for type checking.
+ if( t1 instanceof TypeInteger x && t2 instanceof TypeInteger y ) {
+ if( y.isConstant() && y.value()==0 ) return lhs;
+ if( x.isConstant() && x.value()==0 ) return new MinusNode(rhs);
+ }
// x - (-y) is x+y
if( rhs instanceof MinusNode minus )