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

Issue 11742045: Use RelocInfo::IsNone in a few more places. (Closed)

Created:
7 years, 11 months ago by JF
Modified:
7 years, 11 months ago
Reviewers:
ulan
CC:
v8-dev
Visibility:
Public.

Description

Use RelocInfo::IsNone in a few more places. I had missed some earlier. Followup to: https://chromiumcodereview.appspot.com/11695006/ There are now NONE and NONE64 RelocInfo types, but only ARM uses them both at the same time. They were added in: https://chromiumcodereview.appspot.com/11191029/ R= ulan@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=13317

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -10 lines) Patch
M src/ia32/assembler-ia32.h View 1 chunk +3 lines, -3 lines 0 comments Download
M src/ia32/assembler-ia32.cc View 4 chunks +6 lines, -6 lines 0 comments Download
M src/ia32/assembler-ia32-inl.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
JF
7 years, 11 months ago (2013-01-04 18:29:22 UTC) #1
ulan
7 years, 11 months ago (2013-01-07 08:55:58 UTC) #2
LGTM, I will land this for you.

Powered by Google App Engine
This is Rietveld 408576698