There's some duplicate logic between tryCacheGetById and tryBuildGetByIDList. We should just do self caching in tryCacheGetById, and if we fail then we should try calling tryBuildGetByIDList before giving up.
Created attachment 230342 [details] Patch
Comment on attachment 230342 [details] Patch R=me too, what is the perf impact?
Created attachment 230350 [details] Perf results Here's the results. Neutral. I re-ran sun spider with inner, outer = 8, 8, and it looked like a small progression ~0.5%.
(In reply to comment #3) > Created an attachment (id=230350) [details] > Perf results > > Here's the results. Neutral. I re-ran sun spider with inner, outer = 8, 8, and it looked like a small progression ~0.5%. Yeah that looks pretty good!
(In reply to comment #4) > (In reply to comment #3) > > Created an attachment (id=230350) [details] [details] > > Perf results > > > > Here's the results. Neutral. I re-ran sun spider with inner, outer = 8, 8, and it looked like a small progression ~0.5%. > > Yeah that looks pretty good! Wait ... did you enable FTL in that run?
(In reply to comment #5) > (In reply to comment #4) > > (In reply to comment #3) > > > Created an attachment (id=230350) [details] [details] [details] > > > Perf results > > > > > > Here's the results. Neutral. I re-ran sun spider with inner, outer = 8, 8, and it looked like a small progression ~0.5%. > > > > Yeah that looks pretty good! > > Wait ... did you enable FTL in that run? Yep. I ran with the same flags and only added the "--sunspider --inner 8 --outer 8" before the VMs. Why?
(In reply to comment #6) > (In reply to comment #5) > > (In reply to comment #4) > > > (In reply to comment #3) > > > > Created an attachment (id=230350) [details] [details] [details] [details] > > > > Perf results > > > > > > > > Here's the results. Neutral. I re-ran sun spider with inner, outer = 8, 8, and it looked like a small progression ~0.5%. > > > > > > Yeah that looks pretty good! > > > > Wait ... did you enable FTL in that run? > > Yep. I ran with the same flags and only added the "--sunspider --inner 8 --outer 8" before the VMs. Why? Just checking how you built since FTL is disabled by default on most OSes.
Committed r167922: <http://trac.webkit.org/changeset/167922>