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

Issue 232053004: Avoid bogus type assertion on object comparison in Hydrogen (Closed)

Created:
6 years, 8 months ago by Jarin
Modified:
6 years, 8 months ago
Reviewers:
Yang
CC:
v8-dev
Visibility:
Public.

Description

Avoid type assertion on object comparison in Hydrogen - the comparison is unreachable because of previous checks. BUG= R=yangguo@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=20666

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+65 lines, -35 lines) Patch
M src/hydrogen.cc View 2 chunks +11 lines, -11 lines 1 comment Download
M test/mjsunit/regress/regress-359491.js View 1 chunk +54 lines, -24 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Jarin
Fixing another problem with the compare-object assertion in Hydrogen. https://codereview.chromium.org/232053004/diff/1/src/hydrogen.cc File src/hydrogen.cc (left): https://codereview.chromium.org/232053004/diff/1/src/hydrogen.cc#oldcode9878 src/hydrogen.cc:9878: ...
6 years, 8 months ago (2014-04-10 14:24:34 UTC) #1
Yang
On 2014/04/10 14:24:34, jarin wrote: > Fixing another problem with the compare-object assertion in Hydrogen. ...
6 years, 8 months ago (2014-04-10 14:59:05 UTC) #2
Jarin
6 years, 8 months ago (2014-04-11 06:45:32 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r20666 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698