Remove wrong assertion, and make sure we only change the type of constant lists.

Remove wrong assertion, and make sure we only change the type of constant lists.

R=floitsch@google.com

Committed: https://code.google.com/p/dart/source/detail?r=31279

[ngeoffray, 2013-12-19 16:10:46]

I added this assertion while implementing:
https://codereview.chromium.org/114613003/, to ensure we don't get weird types
for constants because of the inferrer. There are some mismatches between the
builder and the inferrer for constants that sometimes lead to assigning the
wrong type to the constant.

[ngeoffray, 2013-12-19 16:10:46]

I added this assertion while implementing:
https://codereview.chromium.org/114613003/, to ensure we don't get weird types
for constants because of the inferrer. There are some mismatches between the
builder and the inferrer for constants that sometimes lead to assigning the
wrong type to the constant.

[floitsch, 2013-12-19 16:16:29]

LGTM with comments.

https://codereview.chromium.org/109883003/diff/1/sdk/lib/_internal/compiler/i...
File sdk/lib/_internal/compiler/implementation/ssa/nodes.dart (right):

https://codereview.chromium.org/109883003/diff/1/sdk/lib/_internal/compiler/i...
sdk/lib/_internal/compiler/implementation/ssa/nodes.dart:1936: // Only lists can
be specialized.
Please add more information.

[ngeoffray, 2013-12-19 16:17:51]

Thanks Florian.

https://codereview.chromium.org/109883003/diff/1/sdk/lib/_internal/compiler/i...
File sdk/lib/_internal/compiler/implementation/ssa/nodes.dart (right):

https://codereview.chromium.org/109883003/diff/1/sdk/lib/_internal/compiler/i...
sdk/lib/_internal/compiler/implementation/ssa/nodes.dart:1936: // Only lists can
be specialized.
On 2013/12/19 16:16:29, floitsch wrote:
> Please add more information.

Done.

[ngeoffray, 2013-12-19 16:18:19]

Committed patchset #2 manually as r31279 (presubmit successful).