Open
Bug 742627
Opened 13 years ago
Updated 4 months ago
IonMonkey: Suppress bailout spew in readFrameHeader() when called by ion::GetPcScript().
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
NEW
People
(Reporter: sstangl, Unassigned)
Details
(Whiteboard: [ion:t])
Attachments
(1 file)
4.21 KB,
patch
|
nbp
:
review+
|
Details | Diff | Splinter Review |
Using IONFLAGS=bailouts is currently a pretty miserable experience, since the majority of the spew on any major benchmark primarily consists of bogus spew from snapshot reading as part of ion::GetPcScript().
The proposed patch applies a messy spotfix that suppresses bailout spew in that context. GetPcScript() is the sole offender, and threading context data all the way through to the SnapshotReader is wasteful.
Attachment #612456 -
Flags: review?(nicolas.b.pierron)
Comment 1•13 years ago
|
||
Comment on attachment 612456 [details] [diff] [review]
Suppress spew with a spotfix.
Review of attachment 612456 [details] [diff] [review]:
-----------------------------------------------------------------
This patch mute the bailout spew produced by GetPcScript which can trouble grep commands. As GetPCScript cannot cause a bailout, this modification is good.
Attachment #612456 -
Flags: review?(nicolas.b.pierron) → review+
Updated•12 years ago
|
Whiteboard: [ion:t]
Assignee | ||
Updated•10 years ago
|
Assignee: general → nobody
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•