Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(52)

Issue 6597003: ARM: Align full codegen with IA32 version... (Closed)

Created:
9 years, 10 months ago by Søren Thygesen Gjesse
Modified:
9 years, 7 months ago
Reviewers:
Karl Klose
CC:
v8-dev
Visibility:
Public.

Description

ARM: Align full codegen with IA32 version Use ToBooleanStub in DoTest instead of runtime call. Try allocation in generated code in VisitRegExpLiteral. Use TranscendentalCacheStub for sin, cos and log instead of calling runtime. Minor "alignment" of the full codegen code across platforms. Committed: http://code.google.com/p/v8/source/detail?r=6953

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Patch Set 5 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+147 lines, -36 lines) Patch
M src/arm/full-codegen-arm.cc View 1 2 3 9 chunks +96 lines, -28 lines 0 comments Download
M src/ia32/full-codegen-ia32.cc View 1 2 7 chunks +22 lines, -4 lines 0 comments Download
M src/x64/full-codegen-x64.cc View 1 2 4 chunks +29 lines, -4 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Søren Thygesen Gjesse
9 years, 10 months ago (2011-02-25 10:25:23 UTC) #1
Karl Klose
9 years, 10 months ago (2011-02-25 11:35:11 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698