523381 - add un-blacklisting to jitstats

Status: RESOLVED WONTFIX

(Core :: JavaScript Engine, defect)

Description

[Luke Wagner [:luke]]

Attachment: unblacklist stat

Currently, if an outer loop is blacklisted in the course of recording an inner loop, it will be un-blacklisted when the inner loop finishes recording.  The initial blacklisting is recorded in the jitstats, but not the subsequent un-blacklisting.  This attached patch adds a new "unblacklisted" jitstat so that we may better interpret the behavior of the JIT.

Comment 1

[Luke Wagner [:luke]]

Also, David: should AttemptCompilation also have:

else if (*pc == JSOP_CALL && *(pc + JSOP_CALL_LENGTH) == JSOP_NOOP) {
  AUDIT(unblacklisted);
  *(pc + JSOP_CALL_LENGTH) = JSOP_TRACE;
}

?

Comment 2

[Luke Wagner [:luke]]

Attachment: oops, forgot to print stat at the end

Comment 3

[David Anderson [:dvander] - inactive, e-mail if emergency]

(In reply to comment #1)

Yes. I'm planning on a separate bug for this because right now recursion and blacklisting is messed up in a few places.

Comment 4

[Luke Wagner [:luke]]

I'm not planning to do this.