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

Issue 23754002: Fix assert triggered by constant folding. (Closed)

Created:
7 years, 3 months ago by oliv
Modified:
7 years, 3 months ago
Reviewers:
Jakob Kummerow
CC:
v8-dev
Visibility:
Public.

Description

Fix assert triggered by constant folding. Add<> asserts the return type, which might not be an HAdd in this case because of constant folding. BUG= R=jkummerow@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=16414

Patch Set 1 #

Total comments: 1

Patch Set 2 : address review #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -3 lines) Patch
M src/hydrogen.cc View 1 1 chunk +4 lines, -3 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
oliv
7 years, 3 months ago (2013-08-29 09:50:11 UTC) #1
Jakob Kummerow
LGTM with comment. https://codereview.chromium.org/23754002/diff/1/src/hydrogen.cc File src/hydrogen.cc (right): https://codereview.chromium.org/23754002/diff/1/src/hydrogen.cc#newcode1546 src/hydrogen.cc:1546: HValue* min_growth = AddUncasted<HConstant>(16); You can ...
7 years, 3 months ago (2013-08-29 09:52:11 UTC) #2
Benedikt Meurer
IMHO we need some way for the compiler to detect this. I.e. Add<XZY>(...) should fail ...
7 years, 3 months ago (2013-08-29 09:54:09 UTC) #3
oliv
Committed patchset #2 manually as r16414 (presubmit successful).
7 years, 3 months ago (2013-08-29 09:54:59 UTC) #4
oliv
7 years, 3 months ago (2013-08-29 09:56:17 UTC) #5
Message was sent while issue was closed.
On 2013/08/29 09:54:09, Benedikt Meurer wrote:
> IMHO we need some way for the compiler to detect this. I.e. Add<XZY>(...)
should
> fail to compile if XZY::New() does not necessarily return an XYZ.

oh i agree, but i just wanted to fix this since it broke on octane....

Powered by Google App Engine
This is Rietveld 408576698