shithub: scc

Info  •  Files  •  Log  •  Branches

Fix types of integer operands in pointer additions

The code was adding things of different type, and it was doing
casts that were not necessary. This new version does the multiplication
in size_t type, and convert the result to pointer.

--- a/cc1/expr.c

+++ b/cc1/expr.c

@@ -457,9 +457,11 @@

 	}

 	if (BTYPE(rp) != INT)

 		goto incorrect;

-	rp = node(OCAST, tp, promote(rp), NULL);

-	rp = node(OMUL, tp, rp, size);

-	return node(op, tp, lp, rp);

+	rp = convert(promote(rp), sizettype, 0);

+	rp = node(OMUL, sizettype, rp, size);

+	rp = node(OCAST, tp, rp, NULL);

+

+	return node(OADD, tp, lp, rp);

 incorrect:

 	error("incorrect arithmetic operands");