diff options
author | Sam McCall <sam.mccall@gmail.com> | 2022-06-21 21:22:22 +0200 |
---|---|---|
committer | Sam McCall <sam.mccall@gmail.com> | 2022-06-23 17:26:42 +0200 |
commit | e3ec054dfdf48f19cb6726cb3f4965b9ab320ed9 (patch) | |
tree | 06b8aff31694a223000ee7c084f08584ac73e949 /lldb/source/Commands/CommandObjectScript.cpp | |
parent | 9afaa158f52c13cb79fb0ab8cc73361b134c08c9 (diff) | |
download | llvm-e3ec054dfdf48f19cb6726cb3f4965b9ab320ed9.zip llvm-e3ec054dfdf48f19cb6726cb3f4965b9ab320ed9.tar.gz llvm-e3ec054dfdf48f19cb6726cb3f4965b9ab320ed9.tar.bz2 |
[pseudo] Track heads as GSS nodes, rather than as "pending actions".
IMO this model is simpler to understand (borrowed from the LR0 patch D127357).
It also makes error recovery easier to implement, as we have a simple list of
head nodes lying around to recover from when needed.
(It's not quite as nice as LR0 in this respect though).
It's slightly slower (2.24 -> 2.12 MB/S on my machine = 5%) but nothing close
to as bad as LR0.
However
- I think we'd have to eat a litle performance loss otherwise to implement
error recovery.
- this frees up some complexity budget for optimizations like fastpath push/pop
(this + fastpath is already faster than head)
- I haven't changed the data structure here and it's now pretty dumb, we can
make it faster
Differential Revision: https://reviews.llvm.org/D128297
Diffstat (limited to 'lldb/source/Commands/CommandObjectScript.cpp')
0 files changed, 0 insertions, 0 deletions