Issue 1222203007: Use FullCodeGenerator::EmitGlobalVariableLoad() where possible to avoid code duplication.

Issue 1222203007: Use FullCodeGenerator::EmitGlobalVariableLoad() where possible to avoid code duplication. (Closed)

Created:
5 years, 5 months ago by Igor Sheludko

Modified:
5 years, 5 months ago

Reviewers:
Jakob Kummerow

CC:
v8-dev

Base URL:
https://chromium.googlesource.com/v8/v8.git@master

Target Ref:
refs/pending/heads/master

Project:
v8

Visibility:
Public.

Use FullCodeGenerator::EmitGlobalVariableLoad() where possible to avoid code duplication. Committed: https://crrev.com/f043ab861839402d45af3724611671daf6a798c0 Cr-Commit-Position: refs/heads/master@{#29520}

Created: 5 years, 5 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+243 lines, -455 lines)			Patch
M	src/arm/full-codegen-arm.cc	View	1 2	6 chunks	+28 lines, -55 lines	0 comments	Download
M	src/arm64/full-codegen-arm64.cc	View	1 2	6 chunks	+28 lines, -53 lines	0 comments	Download
M	src/full-codegen.h	View	1	2 chunks	+3 lines, -2 lines	0 comments	Download
M	src/full-codegen.cc	View	1	2 chunks	+16 lines, -5 lines	0 comments	Download
M	src/ia32/full-codegen-ia32.cc	View	1 2	6 chunks	+28 lines, -59 lines	0 comments	Download
M	src/mips/full-codegen-mips.cc	View	1 2	6 chunks	+28 lines, -54 lines	0 comments	Download
M	src/mips64/full-codegen-mips64.cc	View	1 2	6 chunks	+28 lines, -56 lines	0 comments	Download
M	src/ppc/full-codegen-ppc.cc	View	1 2	6 chunks	+28 lines, -54 lines	0 comments	Download
M	src/x64/full-codegen-x64.cc	View	1 2	6 chunks	+28 lines, -58 lines	0 comments	Download
M	src/x87/full-codegen-x87.cc	View	1 2	6 chunks	+28 lines, -59 lines	0 comments	Download

Total messages: 8 (3 generated)

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1222203007/80001

5 years, 5 months ago (2015-07-07 14:44:50 UTC) #6

commit-bot: I haz the power

5 years, 5 months ago (2015-07-07 15:05:00 UTC) #8

Message was sent while issue was closed.

Patchset 4 (id:??) landed as
https://crrev.com/f043ab861839402d45af3724611671daf6a798c0
Cr-Commit-Position: refs/heads/master@{#29520}