diff --git a/Makefile b/Makefile
index ae22d4ed3..8ab695b7d 100644
--- a/Makefile
+++ b/Makefile
@@ -5,16 +5,23 @@
 #
 #other:
 make all:
+# move these parts into compiling archtest separtately
 	cp -r addins/riscv-isa-sim/arch_test_target/spike/device/rv32i_m/I addins/riscv-isa-sim/arch_test_target/spike/device/rv32i_m/F
 	cp -r addins/riscv-isa-sim/arch_test_target/spike/device/rv64i_m/I addins/riscv-isa-sim/arch_test_target/spike/device/rv64i_m/D
+#why cat
 	cat addins/riscv-isa-sim/arch_test_target/spike/device/rv32i_m/F/Makefile.include
 	sed -i 's/--isa=rv32i /--isa=32if/' addins/riscv-isa-sim/arch_test_target/spike/device/rv32i_m/F/Makefile.include
 	sed -i 's/--isa=rv64i /--isa=64if/' addins/riscv-isa-sim/arch_test_target/spike/device/rv64i_m/D/Makefile.include
 	if [ -d "addins/riscv-isa-sim/build" ]; then echo "Build exists"; else mkdir addins/riscv-isa-sim/build; fi
 	cd addins/riscv-isa-sim/build; ../configure --prefix=/cad/riscv/gcc/bin
 	make -C addins/riscv-isa-sim/build
+# does sudo work?
+
 	sudo make install -C addins/riscv-isa-sim/build
+
 	cp addins/riscv-isa-sim/arch_test_target/spike/Makefile.include addins/riscv-arch-test/
+# update with path including $RISCV_TOOLS
+# separate into make tests and make regression
 	sed -i '/export TARGETDIR ?=/c\export TARGETDIR ?= /home/harris/riscv-wally/addins/riscv-isa-sim/arch_test_target' tests/wally-riscv-arch-test/Makefile.include
 	echo export RISCV_PREFIX = riscv64-unknown-elf- >> tests/wally-riscv-arch-test/Makefile.include
 	make -C addins/riscv-arch-test