diff options
author | Kartik Agaram <vc@akkartik.com> | 2019-02-25 00:17:46 -0800 |
---|---|---|
committer | Kartik Agaram <vc@akkartik.com> | 2019-02-25 01:50:53 -0800 |
commit | c442a5ad806b6cccbb3ec4c5744b14b0c1f31a01 (patch) | |
tree | 318fb1d56e7ee3c750635d3326ad0739dfdacefe /subx/028translate.cc | |
parent | e5998f74ac29bb4bf2aedfdd6fbea801ffdb08f6 (diff) | |
download | mu-c442a5ad806b6cccbb3ec4c5744b14b0c1f31a01.tar.gz |
4987 - support `browse_trace` tool in SubX
I've extracted it into a separate binary, independent of my Mu prototype. I also cleaned up my tracing layer to be a little nicer. Major improvements: - Realized that incremental tracing really ought to be the default. And to minimize printing traces to screen. - Finally figured out how to combine layers and call stack frames in a single dimension of depth. The answer: optimize for the experience of `browse_trace`. Instructions occupy a range of depths based on their call stack frame, and minor details of an instruction lie one level deeper in each case. Other than that, I spent some time adjusting levels everywhere to make `browse_trace` useful.
Diffstat (limited to 'subx/028translate.cc')
-rw-r--r-- | subx/028translate.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/subx/028translate.cc b/subx/028translate.cc index d4b1a3a5..42b94632 100644 --- a/subx/028translate.cc +++ b/subx/028translate.cc @@ -34,6 +34,7 @@ if (is_equal(argv[1], "translate")) { output_filename = argv[i]; } else { + trace(2, "parse") << argv[i] << end(); ifstream fin(argv[i]); if (!fin) { cerr << "could not open " << argv[i] << '\n'; @@ -53,8 +54,10 @@ if (is_equal(argv[1], "translate")) { cerr << "must provide a filename to write to using '-o'\n"; exit(1); } + trace(2, "transform") << "begin" << end(); transform(p); if (trace_contains_errors()) return 1; + trace(2, "translate") << "begin" << end(); save_elf(p, output_filename); if (trace_contains_errors()) { unlink(output_filename.c_str()); |