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

Issue 110443003: Make it possible to assert that certain instance types are compatible wrt HAllocate. (Closed)

Created:
7 years ago by Benedikt Meurer
Modified:
6 years, 11 months ago
CC:
v8-dev
Visibility:
Public.

Description

Make it possible to assert that certain instance types are compatible wrt HAllocate. Turns clear_next_map_word_ into a flag and adds a new method CompatibleInstanceTypes(). This is used in BuildUncheckedStringAdd() to ASSERT that there's no difference wrt. HAllocate in using CONS_STRING_TYPE vs CONS_ASCII_STRING_TYPE. R=mvstanton@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=18428

Patch Set 1 #

Total comments: 2

Patch Set 2 : Address Michael's comment. #

Patch Set 3 : Do not expose the flags. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+43 lines, -18 lines) Patch
M src/hydrogen.cc View 1 2 1 chunk +2 lines, -0 lines 0 comments Download
M src/hydrogen-instructions.h View 1 2 6 chunks +39 lines, -16 lines 0 comments Download
M src/hydrogen-instructions.cc View 1 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Benedikt Meurer
Hey Michael, Here's the HAllocate flags cleanup you suggested in https://codereview.chromium.org/106453003/. Pretty much straight-forward. PTAL ...
7 years ago (2013-12-11 12:46:04 UTC) #1
mvstanton
Cool, one comment to address and LGTM. --Michael https://codereview.chromium.org/110443003/diff/1/src/hydrogen-instructions.cc File src/hydrogen-instructions.cc (right): https://codereview.chromium.org/110443003/diff/1/src/hydrogen-instructions.cc#newcode3497 src/hydrogen-instructions.cc:3497: (flags_ ...
7 years ago (2013-12-11 13:03:21 UTC) #2
Benedikt Meurer
https://codereview.chromium.org/110443003/diff/1/src/hydrogen-instructions.cc File src/hydrogen-instructions.cc (right): https://codereview.chromium.org/110443003/diff/1/src/hydrogen-instructions.cc#newcode3497 src/hydrogen-instructions.cc:3497: (flags_ & CLEAR_NEXT_MAP_WORD)); On 2013/12/11 13:03:21, mvstanton wrote: > ...
7 years ago (2013-12-11 13:43:40 UTC) #3
Benedikt Meurer
Hey guys, here's the updated code as discussed offline.
7 years ago (2013-12-11 13:50:06 UTC) #4
mvstanton
That still works for me.
7 years ago (2013-12-11 14:16:36 UTC) #5
Benedikt Meurer
6 years, 11 months ago (2014-01-02 06:26:50 UTC) #6
Message was sent while issue was closed.
Committed patchset #3 manually as r18428 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698