Update vcompress.vm to not write vstart with 0 upon completion

rbuchner-aril · rbuchner-aril · commit a9831b437729 · 2024-03-11T22:51:12.000-07:00
Vmcompress.vm requires vstart==0, so writing vstart with 0 is redundant. To do this, spin off VI_LOOP_END_BASE from VI_LOOP_END. VI_LOOP_END will contain VI_LOOP_END_BASE as well as a write of 0 to vstart. See #1623 for full discussion.
diff --git a/riscv/insns/vcompress_vm.h b/riscv/insns/vcompress_vm.h
@@ -30,4 +30,4 @@ VI_GENERAL_LOOP_BASE
 
     ++pos;
   }
-VI_LOOP_END;
+VI_LOOP_END_BASE;
diff --git a/riscv/v_ext_macros.h b/riscv/v_ext_macros.h
@@ -215,8 +215,11 @@ static inline bool is_overlapped_widen(const int astart, int asize,
     VI_GENERAL_LOOP_BASE \
     VI_LOOP_ELEMENT_SKIP();
 
+#define VI_LOOP_END_BASE \
+ }
+
 #define VI_LOOP_END \
-  } \
+  VI_LOOP_END_BASE \
   P.VU.vstart->write(0);
 
 #define VI_LOOP_REDUCTION_END(x) \

Original file line number	Diff line number	Diff line change
`@@ -30,4 +30,4 @@ VI_GENERAL_LOOP_BASE`
`30`	`30`
`31`	`31`	`++pos;`
`32`	`32`	`}`
`33`		`-VI_LOOP_END;`
	`33`	`+VI_LOOP_END_BASE;`