[Webkit-unassigned] [Bug 69996] DFG should have inlining

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=69996





--- Comment #1 from Filip Pizlo <fpizlo at apple.com>  2011-10-17 22:20:46 PST ---
Created an attachment (id=111381)
 --> (https://bugs.webkit.org/attachment.cgi?id=111381&action=review)
work in progress

If this even compiles, then clang is broken.

Still working on this.  Items that are (hopefully) done:

- Flushing state related to the call that can be reflectively accessed, like the arguments, and the callee.

- Determining when it is profitable to inline based on the classic heuristics (inlinee size, inline stack depth, recursion detection, rejection of fancy operations that the inliner won't handle correctly).

- Actually inlining code, with the following special things:

- Inlining calls to single-basic-block functions does not introduce any control flow, and so the only "evidence" that there was a call is the flushing of arguments and the callee.

- Inlining calls to functions with multiple basic blocks does the right thing.

- Inlining calls to functions that don't terminate does the right thing.

- Inlining calls to functions that have multiple return statements does the right thing.

Things that aren't done:

- OSR support.

- ???

- Make it compiler, pass tests, and produce speed-ups.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.