VM crash when type Never is used

[eernstg]

Consider the following two programs:

import 'dart:core' as core;

main() {
  core.Never n = throw "away";
}

main() {
  Never n = throw "away";
}

Both of them cause a crash in dart from commit 11d523d (but dartanalyzer accepts the programs):

> dart --enable-experiment=non-nullable n039.dart; dart --enable-experiment=non-nullable n040.dart
Unhandled exception:
'file:///usr/local/google/home/eernst/lang/dart/scratch/201911/n039.dart': error: Unexpected tag 1 (Something) in ::.main, expected expression
#0      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:307:19)
#1      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:174:12)
Unhandled exception:
'file:///usr/local/google/home/eernst/lang/dart/scratch/201911/n040.dart': error: Unexpected tag 1 (Something) in ::.main, expected expression
#0      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:307:19)
#1      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:174:12)

The NNBD specification indicates that Never is declared in 'dart:core', so both should work.

The following form causes the same crash (before execution starts), so it isn't caused by the use of throw "away" as an initializing expression:

import 'dart:core' as core;

main() {
  core.dynamic d = 42;
  core.Never n = d;
}

[crelier]

I am not sure why it works in dartanalyzer, but these errors are produced by the front-end.
I am reassigning the issue.

[sgrekhov]

See also #39461

[crelier]

Never mind :-)
Having a second look.

[crelier]

Although the type Never is always non-nullable, the CFE writes a nullability
byte in kernel files that was not properly skipped by the VM.
A CL is under review.

[crelier]

Fixed by af541cd