Last Comment Bug 695877 - IM: Each MBasicBlock refers to a graph and its generating CompileInfo
: IM: Each MBasicBlock refers to a graph and its generating CompileInfo
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: unspecified
: All All
: -- normal (vote)
: ---
Assigned To: Chris Leary [:cdleary] (not checking bugmail)
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-10-19 14:56 PDT by Chris Leary [:cdleary] (not checking bugmail)
Modified: 2012-01-09 13:30 PST (History)
0 users
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Inject MGraph and CompileInfo into MBasicBlock. (19.17 KB, patch)
2011-10-19 14:56 PDT, Chris Leary [:cdleary] (not checking bugmail)
dvander: review+
Details | Diff | Splinter Review

Description Chris Leary [:cdleary] (not checking bugmail) 2011-10-19 14:56:04 PDT
Created attachment 568221 [details] [diff] [review]
Inject MGraph and CompileInfo into MBasicBlock.

Decouples MBasicBlock from its MIRGenerator (which is really its IonBuilder). This way the IonBuilders can be deallocated for inline functions, with a single top-level IonBuilder. This further reduces the functionality of MIRGenerator.
Comment 1 Chris Leary [:cdleary] (not checking bugmail) 2012-01-09 13:30:40 PST
Whoops, never resolved this as fixed.

https://hg.mozilla.org/projects/ionmonkey/rev/d5403d9e1d8e

Note You need to log in before you can comment on or make changes to this bug.