Closed m8pple closed 3 years ago
Thanks @m8pple, I've made some tweaks to the docs. I'm on holiday this week, so will aim to merge early next week after I've had chance to try it!
@m8pple Very nice, it works for me, but please see the above comment about the hardware simulation target in the POLite Makefile before we can merge.
@m8pple, I've added back the POLite target for hardware simulation, renamed from sim
to hwsim
to avoid the clash. Will merge soon, unless you have any final comments.
Thanks.
I think I just deleted the sim target as a side-effect without thinking about it.
This is an attempt to pick just the software simulator out of my previous mess.
I've modified it a bit to try to make it as isolated as possible:
apps/POLite/util
, so that it is clear it is not an application.POLiteSWSim
, which resides inPGraph.h
.POLite.h
just pulls a small number of types into the global namespace.polite.mk
is modified to addsim
andsim-release
targets (I think I did slightly more than that, but the diff is not too bad I hope).Apart from that, I also tried to make testing a bit clearer:
test_polite_sw_sim.sh
which tries to compile all the apps, and then run a sub-set which don't need input files.Simulation.
note to the POLite section of the readme.Unfortunately the actual simulation code in PGraph is still a giant blob, but I hope it is clear that it is additive and shouldn't affect other parts of the repo, and at least there is testing there to show that it works.
I'd like to add more general testing in a separate pull, but kept it out of this one to minimise the size of the patch.