Bug 185188

Summary:

[JSC] Add MacroAssembler::and16 and store16

Product:

WebKit

Reporter:

Yusuke Suzuki <ysuzuki>

Component:

New Bugs

Assignee:

Yusuke Suzuki <ysuzuki>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ews-watchlist, jfbastien, keith_miller, mark.lam, msaboff, saam, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	mark.lam: review+

Yusuke Suzuki

Reported 2018-05-02 00:51:32 PDT

[JSC] Add MacroAssembler::and16

Attachments
Patch (5.13 KB, patch) 2018-05-02 01:05 PDT, Yusuke Suzuki	mark.lam: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Yusuke Suzuki

Comment 1 2018-05-02 01:05:03 PDT

Mark Lam

Comment 2 2018-05-02 09:55:39 PDT

Comment on attachment 339289 [details] Patch Sorry, I didn't see this patch before I reviewed and r+'ed https://bugs.webkit.org/show_bug.cgi?id=185196 and https://bugs.webkit.org/show_bug.cgi?id=185195. r+ anyway for Source/JavaScriptCore/assembler/MacroAssemblerARM.h part. You'll probably have to remove the ARMv7 and MIPS parts to resolve a conflict before landing.

Yusuke Suzuki

Comment 3 2018-05-02 17:51:54 PDT

(In reply to Mark Lam from comment #2) > Comment on attachment 339289 [details] > Patch > > Sorry, I didn't see this patch before I reviewed and r+'ed > https://bugs.webkit.org/show_bug.cgi?id=185196 and > https://bugs.webkit.org/show_bug.cgi?id=185195. > > r+ anyway for Source/JavaScriptCore/assembler/MacroAssemblerARM.h part. > You'll probably have to remove the ARMv7 and MIPS parts to resolve a > conflict before landing. OK, I'll land ARM traditional part.

Yusuke Suzuki

Comment 4 2018-05-02 17:53:46 PDT

Radar WebKit Bug Importer

Comment 5 2018-05-02 17:54:21 PDT

Note You need to log in before you can comment on or make changes to this bug.