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

Issue 578533003: Fix compiler warning: (Closed)

Created:
6 years, 3 months ago by siva
Modified:
6 years, 3 months ago
CC:
Vyacheslav Egorov (Google), reviews_dartlang.org, vm-dev_dartlang.org
Visibility:
Public.

Description

Fix compiler warning: [209/7854] CXX obj/dart/runtime/vm/libdart_vm.intermediate_language.o ../../dart/runtime/vm/intermediate_language.cc:1777:45: warning: implicit conversion of NULL constant to 'intptr_t' (aka 'long') [-Wnull-conversion] to() == kUnboxedInt32 ? deopt_id_ : NULL); R=vegorov@google.com Committed: https://code.google.com/p/dart/source/detail?r=40355

Patch Set 1 #

Total comments: 2

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -2 lines) Patch
M runtime/vm/intermediate_language.cc View 1 2 chunks +3 lines, -2 lines 0 comments Download

Messages

Total messages: 9 (2 generated)
siva
6 years, 3 months ago (2014-09-16 18:42:11 UTC) #2
srdjan
https://chromiumcodereview.appspot.com/578533003/diff/1/runtime/vm/intermediate_language.cc File runtime/vm/intermediate_language.cc (right): https://chromiumcodereview.appspot.com/578533003/diff/1/runtime/vm/intermediate_language.cc#newcode1777 runtime/vm/intermediate_language.cc:1777: to() == (kUnboxedInt32 ? deopt_id_ : 0)); kNoRepresentation? I ...
6 years, 3 months ago (2014-09-16 18:55:19 UTC) #3
Vyacheslav Egorov (Google)
https://codereview.chromium.org/578533003/diff/1/runtime/vm/intermediate_language.cc File runtime/vm/intermediate_language.cc (right): https://codereview.chromium.org/578533003/diff/1/runtime/vm/intermediate_language.cc#newcode1777 runtime/vm/intermediate_language.cc:1777: to() == (kUnboxedInt32 ? deopt_id_ : 0)); should be ...
6 years, 3 months ago (2014-09-16 18:57:05 UTC) #5
Vyacheslav Egorov (Google)
just to give more context: only conversion to Int32 can deoptimize, and thus needs deopt_id ...
6 years, 3 months ago (2014-09-16 18:58:13 UTC) #6
siva
PTAL
6 years, 3 months ago (2014-09-16 19:34:29 UTC) #7
Vyacheslav Egorov (Google)
LGTM!
6 years, 3 months ago (2014-09-16 19:37:21 UTC) #8
siva
6 years, 3 months ago (2014-09-16 22:50:52 UTC) #9
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as r40355 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698