Open Bug 742627 Opened 10 years ago Updated 8 years ago

IonMonkey: Suppress bailout spew in readFrameHeader() when called by ion::GetPcScript().

Categories

(Core :: JavaScript Engine, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

People

(Reporter: sstangl, Unassigned)

Details

(Whiteboard: [ion:t])

Attachments

(1 file)

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 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+
Assignee: general → nobody
You need to log in before you can comment on or make changes to this bug.