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

Issue 15994011: MIPS: Deprecate FACTORY helper macro. (Closed)

Created:
7 years, 6 months ago by kilvadyb
Modified:
7 years, 6 months ago
Reviewers:
kisg, Paul Lind, palfia, danno
CC:
v8-dev
Visibility:
Public.

Description

MIPS: Deprecate FACTORY helper macro. Port r14931 (a914442) Original commit message: This removes the FACTORY helper macro to avoid accidental TLS access when using the factory. Most internal code has access to the Isolate by now whereas tests which are not performance critical still heavily use TLS access through explicit Isolate::Current() calls. BUG=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -5 lines) Patch
M src/mips/code-stubs-mips.cc View 1 chunk +1 line, -1 line 0 comments Download
M src/mips/full-codegen-mips.cc View 1 chunk +1 line, -1 line 0 comments Download
M src/mips/regexp-macro-assembler-mips.cc View 1 chunk +2 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
kilvadyb
7 years, 6 months ago (2013-06-04 15:36:58 UTC) #1
Paul Lind
lgtm
7 years, 6 months ago (2013-06-04 16:37:24 UTC) #2
palfia
7 years, 6 months ago (2013-06-04 20:15:45 UTC) #3
Committed as r14942.

Powered by Google App Engine
This is Rietveld 408576698