Closed Bug 549513 (JaegerPIC) Opened 14 years ago Closed 11 years ago

[meta] JM: PIC

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: dmandelin, Unassigned)

References

Details

This is a meta bug for tracking work on PICs for JaegerMonkey. 

We'll start out with some analysis; for all we know the property cache is already the ideal implementation, but we need to check into it.
Depends on: 549515
Depends on: 549516
The property cache was good for Firefox 3 / JS1.8, but it's far from ideal. See bug 497789, finally about to be (mega-)patched. You should start with that patch applied, don't waste time without it.

/be
Blocks: 551339
WebKit and V8 have pretty much the same PIC design, which pretty much follows the Self paper extended to property access (the paper mainly treated method calls), so I think it makes sense to go ahead with that.

ETA is May 1 for all basic patches submitted for review.

This is a very rough estimate, really just a guess. I should be able to do better once I've gotten at least one major chunk done. Initial plan is:

1. PIC for |o.x| 
   Once this is done the rest should be easier, basically variations on this
   theme.

2. PIC for |o.x = e|

3. PIC for |o.x()| 
   This one may be a bit harder; both V8 and WebKit have specialized
   optimizations for calls. I haven't studied them yet but my guess is they 
   are similar to the ones we have for the property cache.

4. PIC for |o[e]|

5. PIC for |o[e] = e'|
Depends on: 552783
Depends on: 554535
Depends on: 557351
Update on the estimate. It's been just about 3 weeks to "basically finish" PICs for getprop. There are 5 bugs to completely finish that. I'd estimate about 0.5 weeks for those. Going with the original estimate, that leaves 3 weeks to do callprop and setprop, meaning those could be done together in about the same time it took to do just getprop. For now, I'm going to say that seems right--the knowledge and utility code I've built up should make the rest go faster. So no revision to the estimate for now.
Depends on: 557361
Depends on: 558227
PIC is now working on Win x64. Perf wins are similar to what they are on x86. There are probably still bugs in the PIC for Linux x64. I'm installing a VM to take care of that.
Depends on: 559899
Depends on: 561506
Depends on: 562793
Depends on: 562794
Depends on: 578761
Depends on: 578897
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.