TreeContext can be moved into the Parser module, and the TreeContext module can then be renamed.
Created attachment 651977 [details] [diff] [review] Move TreeContext and related things into the Parser module. This patch moves TreeContext, frontend::GenerateBlockId and StmtInfoTC out of the |TreeContext| module into the |Parser| module. This is good because they're all things that are only needed by the Parser (i.e. not by the ByteCodeEmitter). This results in the removal of frontend/TreeContext.cpp and the addition of frontend/Parser-inl.h. I also removed some unnecessary #include statements.
Created attachment 651978 [details] [diff] [review] Rename the |TreeContext| module as |SharedContext|. This patch renames the |TreeContext| module as |SharedContext|, which better reflects its contents.
BTW, this will totally bit-rot the patches in bug 770865, but they're already rotten so it's no great loss, and they'll end up being smaller as a result of the changes in this bug. I'm happy to take over that bug.
I'd be very appreciative if this could land after bug 775323.
Created attachment 652674 [details] [diff] [review] Move TreeContext and related things into the Parser module. Updated for Luke's recent changes.
Created attachment 652675 [details] [diff] [review] Rename the |TreeContext| module as |SharedContext|. Updated for Luke's recent changes.