From 362b33c918938aadd44c6516c15661c30952a584 Mon Sep 17 00:00:00 2001
From: Tobias Girstmair <t@thi3nkpad.lan>
Date: Tue, 13 Nov 2018 19:14:59 +0100
Subject: [PATCH] new version

move `o` out of g() to get rid of variable shift
---
 foo.c | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/foo.c b/foo.c
index 21b19fb..fcb07cd 100644
--- a/foo.c
+++ b/foo.c
@@ -49,7 +49,7 @@ void mod3(void) {
 	#undef tmp
 }
 void g(void) {
-	// g(i, t, o) -> t
+	// g(i, t) -> t
 	// tempvars: `x` and `_`
 	#define tmp _
 	ANDI	(t, 0x07)
@@ -67,8 +67,6 @@ void g(void) {
 	  ADC X_hi, zero
 	  LD  t, X         */
 	t = (((i1&0x1f)<<8|i0)*t)>>8; //TODO
-	t >>= o; //NOTE: o == {1, 2, 4}
-	ANDI	(t, 3)
 	RET //TODO: CALL/RET is expensive; store PC in register and RJMP, then JRMP back
 };
 
@@ -103,6 +101,8 @@ int main(void) {
 		MOV	(t, n)
 		LDI	(o, 4)
 		RCALL	g();
+	t >>= o; //NOTE: o == {1, 2, 4}
+	ANDI	(t, 3)
 		ANDI	(t, 1)
 		MOV	(acc, t)
 
@@ -121,6 +121,8 @@ int main(void) {
 		EOR	(t, n)
 		LDI	(o, 2)
 		RCALL	g();
+	t >>= o; //NOTE: o == {1, 2, 4}
+	ANDI	(t, 3)
 		AND	(t, s)
 		ADD	(acc, t)
 
@@ -151,6 +153,8 @@ int main(void) {
 		ADD	(t, n)
 		LDI	(o, 2)
 		RCALL	g();
+	t >>= o; //NOTE: o == {1, 2, 4}
+	ANDI	(t, 3)
 		MOV	(x, s)
 		INC	(x)
 		#define tmp o
@@ -200,6 +204,8 @@ int main(void) {
 		SUBI	(t, -8)
 		LDI	(o, 1)
 		RCALL	g();
+	t >>= o; //NOTE: o == {1, 2, 4}
+	ANDI	(t, 3)
 		MOV	(x, s)
 		INC	(x)
 		#define tmp o
-- 
2.39.3