Chromium Code Reviews
Help | Chromium Project | Sign in
(159)

Issue 11088026: Make sure that the context argument of HAdd is never NULL. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 6 months ago by Sven Panne
Modified:
1 year, 6 months ago
Reviewers:
Jakob
CC:
v8-dev_googlegroups.com
Visibility:
Public.

Description

Make sure that the context argument of HAdd is never NULL.

Although this doesn't hurt currently, it hurts any future compiler
pass relying on the invariant that no operand is NULL.

R=jkummerow@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=12681

Patch Set 1 #

Total comments: 1

Patch Set 2 : Added comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -1 line) Lint Patch
M src/hydrogen.cc View 1 1 chunk +4 lines, -1 line 0 comments ? errors Download
M src/hydrogen-instructions.h View 2 chunks +3 lines, -0 lines 0 comments 0 errors Download
Trybot results:
Commit:

Messages

Total messages: 2
Jakob
LGTM with RFC. https://codereview.chromium.org/11088026/diff/1/src/hydrogen.cc File src/hydrogen.cc (right): https://codereview.chromium.org/11088026/diff/1/src/hydrogen.cc#newcode3570 src/hydrogen.cc:3570: HValue* context = HBinaryOperation::cast(check->index())->context(); I'd appreciate ...
1 year, 6 months ago #1
Sven Panne
1 year, 6 months ago #2
Comment added, landing...
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6