[Webkit-unassigned] [Bug 90095] JSC: add infrastructure for appending comments to generated bytecode
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Jun 28 11:58:21 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=90095
Geoffrey Garen <ggaren at apple.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #149842|review? |review-
Flag| |
--- Comment #4 from Geoffrey Garen <ggaren at apple.com> 2012-06-28 11:58:20 PST ---
(From update of attachment 149842)
View in context: https://bugs.webkit.org/attachment.cgi?id=149842&action=review
Design looks fine, but this patch could use some small tweaks.
> Source/JavaScriptCore/ChangeLog:10
> + development purposes. It should not be enable for product builds.
Typo: "enable" should be "enabled".
> Source/JavaScriptCore/bytecode/CodeBlock.cpp:99
> +#ifdef USE_BYTECODE_COMMENTS
The WebKit style for this kind of #ifdef is the "ENABLE()" macro.
To turn the feature on: #define ENABLE_BYTECODE_COMMENTS 1
To test the feature: #if ENABLE(BYTECODE_COMMENTS)
> Source/JavaScriptCore/bytecompiler/BytecodeGenerator.cpp:691
> +// Record a comment in the CodeBlock's comments list for the current opcode
> +// that is about to be emitted.
Better to put a comment like this in the header, by the function declaration.
> Source/JavaScriptCore/bytecompiler/BytecodeGenerator.cpp:702
> +// Register a comment to be associated with the next opcode that will be emitted.
Ditto.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list