1099517 - Non-existent category "Debugger" for JSOP_DEBUGAFTERYIELD.

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Tooru Fujisawa [:arai]]

JSOP_DEBUGAFTERYIELD has "Category: Debugger" in its documentation comment, but it's not listed in categorization comment at the top of Opcodes.h.
http://hg.mozilla.org/mozilla-central/file/acbd7b68fa8c/js/src/vm/Opcodes.h#l1622
>    /*
>     * Bytecode emitted after 'yield' statements to help the Debugger
>     * fix up the frame in the JITs. No-op in the interpreter.
>     *
>     *   Category: Debugger
>     *   Operands:
>     *   Stack: =>
>     */ \
>    macro(JSOP_DEBUGAFTERYIELD,  208, "debugafteryield",  NULL,  1,  0,  0,  JOF_BYTE) \

I guess it's "Category: Statements" and "Type: Debugger".

I updated documentation with those category/type for now.
  https://developer.mozilla.org/en-US/docs/Mozilla/Projects/SpiderMonkey/Internals/Bytecode

Comment 1

[Tooru Fujisawa [:arai]]

Attachment: Fix category/type of JSOP_DEBUGAFTERYIELD for bytecode documentation.

Simple comment only fix.

Comment 2

[Jan de Mooij [:jandem]]

Comment on attachment 8525886 [details] [diff] [review]
Fix category/type of JSOP_DEBUGAFTERYIELD for bytecode documentation.

Review of attachment 8525886 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!

::: js/src/vm/Opcodes.h
@@ +1619,5 @@
>       */ \
>      macro(JSOP_FORCEINTERPRETER, 207, "forceinterpreter", NULL,  1,  0,  0,  JOF_BYTE) \
>      \
>      /*
>       * Bytecode emitted after 'yield' statements to help the Debugger

Can you s/statements/expressions/ while you're here? See below.

@@ +1622,5 @@
>      /*
>       * Bytecode emitted after 'yield' statements to help the Debugger
>       * fix up the frame in the JITs. No-op in the interpreter.
>       *
> +     *   Category: Statements

To be pedantic, `yield` is not a statement but an expression, it can return the value you pass to next(): function*() { print(yield 3); }

So maybe it should be "Operators"?

Comment 3

[Tooru Fujisawa [:arai]]

Thank you for reviewing!

Okay, I added "Type: Debugger" to "Category: Operator".
Actually, it should be "Operators" as you wrote, but other opcodes and the list uses "Operator" by my mistake...
I'll file a bug for it.

Comment 4

[Tooru Fujisawa [:arai]]

Try run is green: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=f6cc3d2c4362

https://hg.mozilla.org/integration/mozilla-inbound/rev/ee52bc098176

Comment 5

[Tooru Fujisawa [:arai]]

filed as bug 1102315.

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/ee52bc098176