Ideas for redisigning parse-args.c and main.c

[plavin]

It's clear that parse-args.c and main.c need to be cleaned up.

Issues with parse-args:

• The json parsing is far too complex
• There are likely memory errors due to how we statically allocate strings
• struct run_config is so large that it can overflow the stack

Issues with main.c:

• kernel invocation is a mess

Proposed Solutions:

I think both of these would be much easier to fix in C++.

I think the best way fix parse-args would be to use Boost.Program_options. Development on Argparse3 has stopped, which is frustrating as ASAN detects memory errors in that code that I don't want to fix. Boost.Program_options seems much more flexible, and supports the use of a config file to specify some options.

I think fixing kernel invocation in main might look better with templates.

I also think we should remove json input. It would be much simpler to just let users specify multiple invocations of spatter. This will be trivial to pass to the argument parser, instead of what we do now were we have to reverse the json. This, combined with a config file for options that are common to all invocations would greatly simply parse-args. The json file becomes

-pUNIFORM:8:1 -d 8
-pUNIFORM:8:2 -d 16
-pUNIFORM:8:4 -d 32

and the config file will hold options that will apply to every kernel invocation such as

backend=OMP
aggregate=10

[plavin]

The 2.0 refactor has overhauled the parsing, so this should no longer be an issue.