model, nodestack: bugfix - retain UNINIT actions across executions
authorBrian Norris <banorris@uci.edu>
Thu, 7 Mar 2013 02:40:34 +0000 (18:40 -0800)
committerBrian Norris <banorris@uci.edu>
Thu, 7 Mar 2013 02:40:34 +0000 (18:40 -0800)
commite6789461057e3395ba78575b85c114b553f4ed19
treef5a4f18125a40bcfce3446776ad56b65194fb39d
parent6fb7781c41cfed30851527911c4a48dae79cb107
model, nodestack: bugfix - retain UNINIT actions across executions

Previously, we  were allocating UNINIT actions as snapshotting memory,
just for ease of use. But because they may not immediately trigger a bug
(if they are a valid read-from), such actions must have a longer
lifetime - in fact, a lifetime similar to normal ModelActions.

So, we stash them in NodeStack alongside normal actions.

Also, improve a few ASSERT()'s and clarify a push_front() vs.
push_back() (the list is empty, so it doesn't matter; but it follows the
style of the rest of the lists to use push_front() here).
model.cc
model.h
nodestack.cc
nodestack.h