Created attachment 403609 [details] [diff] [review] import TM changes This patch pulls in some reset-simplification and asserts from TM's assembler. Applies to TR after ... several of the existing changes (best to land bug 519535 and its dependencies first). It doesn't logically depend on them, I don't think, but might not apply without.
Created attachment 405110 [details] [diff] [review] assm reset changes Pure speculation, but might it be worthwhile to also call reset() in endAssembly() otherwise we'll have some variables laying about that are non-null, potentially gumming up the gc.
Comment on attachment 405110 [details] [diff] [review] assm reset changes Assembler is short-lived and stack allocated in Tamarin, so I don't see any reason to do any more cleanup work than necessary unless we have a legitimate GC bug or performance problem.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Resolved fixed engineering / work item that has been pushed. Setting status to verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.