[webkit-reviews] review granted: [Bug 171101] Refactor MASM probe to allow printing of custom types. : [Attachment 307769] updated patch w/ JF's feedback applied.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Apr 21 13:50:58 PDT 2017
JF Bastien <jfbastien at apple.com> has granted Mark Lam <mark.lam at apple.com>'s
request for review:
Bug 171101: Refactor MASM probe to allow printing of custom types.
https://bugs.webkit.org/show_bug.cgi?id=171101
Attachment 307769: updated patch w/ JF's feedback applied.
https://bugs.webkit.org/attachment.cgi?id=307769&action=review
--- Comment #8 from JF Bastien <jfbastien at apple.com> ---
Comment on attachment 307769
--> https://bugs.webkit.org/attachment.cgi?id=307769
updated patch w/ JF's feedback applied.
View in context: https://bugs.webkit.org/attachment.cgi?id=307769&action=review
r=me
> Source/JavaScriptCore/assembler/MacroAssemblerPrinter.h:117
> + explicit Memory(RegisterID& reg, size_t bytes, DumpStyle style =
GenericDump)
FWIW I only meant for the single-parameter ctors. This is fine though:
pre-C++11 it didn't mean anything, but now it applies the same rules for
initialization with curlies.
More information about the webkit-reviews
mailing list