-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[VM] Auto-detection of null-safety mode doesn't work for AOT snapshots #42421
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
dart-bot
pushed a commit
that referenced
this issue
Jun 20, 2020
Set global null safety flag based on the snapshot instead of trying to detect it per isolate (was causing issues with the vm isolate loading). Should fix #42421 Bug:42421 Change-Id: I9143560b76fedcb991e96522cbf5d820fde99f7f Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/151866 Reviewed-by: Alexander Markov <[email protected]> Commit-Queue: Siva Annamalai <[email protected]>
skia-flutter-autoroll
added a commit
to skia-flutter-autoroll/engine
that referenced
this issue
Jun 20, 2020
https://dart.googlesource.com/sdk.git/+log/9fcac032b669..bbc8aedada6f 2020-06-20 [email protected] [VM/Runtime] - Fix for issue dart-lang/sdk#42421 2020-06-20 [email protected] Added module information to metadata and use by frontend_sever 2020-06-19 [email protected] [ddc] Normalize T to Never when T extends Never 2020-06-19 [email protected] [dart:io] HttpClientConnection is destroyed for "CONNECT" request 2020-06-19 [email protected] [dart2js] Add support for Never as a type literal. 2020-06-19 [email protected] [dart2js] Visit type parameter bounds in the scope visitor. 2020-06-19 [email protected] Migrate language_2/type_promotion to NNBD. 2020-06-19 [email protected] Add CiderCompletionComputer.warmUp() 2020-06-19 [email protected] Fix duration.toString() to conform the description If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dart-sdk-flutter-engine Please CC [email protected] on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
This was referenced Jun 20, 2020
skia-flutter-autoroll
added a commit
to skia-flutter-autoroll/engine
that referenced
this issue
Jun 20, 2020
https://dart.googlesource.com/sdk.git/+log/9fcac032b669..5316f32a6468 2020-06-20 [email protected] [benchmark] BigIntParsePrint: Hoist value in all loops 2020-06-20 [email protected] Reland : [VM/Runtime] Cleanup package config initialization code 2020-06-20 [email protected] [dart2js] final locals may be assigned 2020-06-20 [email protected] [VM/Runtime] - Fix for issue dart-lang/sdk#42421 2020-06-20 [email protected] Added module information to metadata and use by frontend_sever 2020-06-19 [email protected] [ddc] Normalize T to Never when T extends Never 2020-06-19 [email protected] [dart:io] HttpClientConnection is destroyed for "CONNECT" request 2020-06-19 [email protected] [dart2js] Add support for Never as a type literal. 2020-06-19 [email protected] [dart2js] Visit type parameter bounds in the scope visitor. 2020-06-19 [email protected] Migrate language_2/type_promotion to NNBD. 2020-06-19 [email protected] Add CiderCompletionComputer.warmUp() 2020-06-19 [email protected] Fix duration.toString() to conform the description If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dart-sdk-flutter-engine Please CC [email protected] on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
skia-flutter-autoroll
added a commit
to skia-flutter-autoroll/engine
that referenced
this issue
Jun 20, 2020
https://dart.googlesource.com/sdk.git/+log/9fcac032b669..7517a0903bc3 2020-06-20 [email protected] [cfe] Mark late setters as isGenericCovariantImpl 2020-06-20 [email protected] Add --track-widget-creation to kernel_front_end.dart 2020-06-20 [email protected] [benchmark] BigIntParsePrint: Hoist value in all loops 2020-06-20 [email protected] Reland : [VM/Runtime] Cleanup package config initialization code 2020-06-20 [email protected] [dart2js] final locals may be assigned 2020-06-20 [email protected] [VM/Runtime] - Fix for issue dart-lang/sdk#42421 2020-06-20 [email protected] Added module information to metadata and use by frontend_sever 2020-06-19 [email protected] [ddc] Normalize T to Never when T extends Never 2020-06-19 [email protected] [dart:io] HttpClientConnection is destroyed for "CONNECT" request 2020-06-19 [email protected] [dart2js] Add support for Never as a type literal. 2020-06-19 [email protected] [dart2js] Visit type parameter bounds in the scope visitor. 2020-06-19 [email protected] Migrate language_2/type_promotion to NNBD. 2020-06-19 [email protected] Add CiderCompletionComputer.warmUp() 2020-06-19 [email protected] Fix duration.toString() to conform the description If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dart-sdk-flutter-engine Please CC [email protected] on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
skia-flutter-autoroll
added a commit
to skia-flutter-autoroll/engine
that referenced
this issue
Jun 20, 2020
https://dart.googlesource.com/sdk.git/+log/9fcac032b669..d46a69695788 2020-06-20 [email protected] Revert "Reland : [VM/Runtime] Cleanup package config initialization code" 2020-06-20 [email protected] [cfe] Mark late setters as isGenericCovariantImpl 2020-06-20 [email protected] Add --track-widget-creation to kernel_front_end.dart 2020-06-20 [email protected] [benchmark] BigIntParsePrint: Hoist value in all loops 2020-06-20 [email protected] Reland : [VM/Runtime] Cleanup package config initialization code 2020-06-20 [email protected] [dart2js] final locals may be assigned 2020-06-20 [email protected] [VM/Runtime] - Fix for issue dart-lang/sdk#42421 2020-06-20 [email protected] Added module information to metadata and use by frontend_sever 2020-06-19 [email protected] [ddc] Normalize T to Never when T extends Never 2020-06-19 [email protected] [dart:io] HttpClientConnection is destroyed for "CONNECT" request 2020-06-19 [email protected] [dart2js] Add support for Never as a type literal. 2020-06-19 [email protected] [dart2js] Visit type parameter bounds in the scope visitor. 2020-06-19 [email protected] Migrate language_2/type_promotion to NNBD. 2020-06-19 [email protected] Add CiderCompletionComputer.warmUp() 2020-06-19 [email protected] Fix duration.toString() to conform the description If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dart-sdk-flutter-engine Please CC [email protected] on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
This was referenced Jun 20, 2020
skia-flutter-autoroll
added a commit
to skia-flutter-autoroll/engine
that referenced
this issue
Jun 20, 2020
https://dart.googlesource.com/sdk.git/+log/9fcac032b669..762113758de6 2020-06-20 [email protected] Enable v1.9 pedantic lints... 2020-06-20 [email protected] Revert "Reland : [VM/Runtime] Cleanup package config initialization code" 2020-06-20 [email protected] [cfe] Mark late setters as isGenericCovariantImpl 2020-06-20 [email protected] Add --track-widget-creation to kernel_front_end.dart 2020-06-20 [email protected] [benchmark] BigIntParsePrint: Hoist value in all loops 2020-06-20 [email protected] Reland : [VM/Runtime] Cleanup package config initialization code 2020-06-20 [email protected] [dart2js] final locals may be assigned 2020-06-20 [email protected] [VM/Runtime] - Fix for issue dart-lang/sdk#42421 2020-06-20 [email protected] Added module information to metadata and use by frontend_sever 2020-06-19 [email protected] [ddc] Normalize T to Never when T extends Never 2020-06-19 [email protected] [dart:io] HttpClientConnection is destroyed for "CONNECT" request 2020-06-19 [email protected] [dart2js] Add support for Never as a type literal. 2020-06-19 [email protected] [dart2js] Visit type parameter bounds in the scope visitor. 2020-06-19 [email protected] Migrate language_2/type_promotion to NNBD. 2020-06-19 [email protected] Add CiderCompletionComputer.warmUp() 2020-06-19 [email protected] Fix duration.toString() to conform the description If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dart-sdk-flutter-engine Please CC [email protected] on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
skia-flutter-autoroll
added a commit
to skia-flutter-autoroll/engine
that referenced
this issue
Jun 21, 2020
https://dart.googlesource.com/sdk.git/+log/9fcac032b669..52dd48651be4 2020-06-21 [email protected] Convert the remaining quick assists to use CorrectionProducer 2020-06-20 [email protected] Enable v1.9 pedantic lints... 2020-06-20 [email protected] Revert "Reland : [VM/Runtime] Cleanup package config initialization code" 2020-06-20 [email protected] [cfe] Mark late setters as isGenericCovariantImpl 2020-06-20 [email protected] Add --track-widget-creation to kernel_front_end.dart 2020-06-20 [email protected] [benchmark] BigIntParsePrint: Hoist value in all loops 2020-06-20 [email protected] Reland : [VM/Runtime] Cleanup package config initialization code 2020-06-20 [email protected] [dart2js] final locals may be assigned 2020-06-20 [email protected] [VM/Runtime] - Fix for issue dart-lang/sdk#42421 2020-06-20 [email protected] Added module information to metadata and use by frontend_sever 2020-06-19 [email protected] [ddc] Normalize T to Never when T extends Never 2020-06-19 [email protected] [dart:io] HttpClientConnection is destroyed for "CONNECT" request 2020-06-19 [email protected] [dart2js] Add support for Never as a type literal. 2020-06-19 [email protected] [dart2js] Visit type parameter bounds in the scope visitor. 2020-06-19 [email protected] Migrate language_2/type_promotion to NNBD. 2020-06-19 [email protected] Add CiderCompletionComputer.warmUp() 2020-06-19 [email protected] Fix duration.toString() to conform the description If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dart-sdk-flutter-engine Please CC [email protected] on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
skia-flutter-autoroll
added a commit
to skia-flutter-autoroll/engine
that referenced
this issue
Jun 21, 2020
https://dart.googlesource.com/sdk.git/+log/9fcac032b669..32d403b472b5 2020-06-21 [email protected] Add a fix to insert the keyword late 2020-06-21 [email protected] Convert the remaining quick assists to use CorrectionProducer 2020-06-20 [email protected] Enable v1.9 pedantic lints... 2020-06-20 [email protected] Revert "Reland : [VM/Runtime] Cleanup package config initialization code" 2020-06-20 [email protected] [cfe] Mark late setters as isGenericCovariantImpl 2020-06-20 [email protected] Add --track-widget-creation to kernel_front_end.dart 2020-06-20 [email protected] [benchmark] BigIntParsePrint: Hoist value in all loops 2020-06-20 [email protected] Reland : [VM/Runtime] Cleanup package config initialization code 2020-06-20 [email protected] [dart2js] final locals may be assigned 2020-06-20 [email protected] [VM/Runtime] - Fix for issue dart-lang/sdk#42421 2020-06-20 [email protected] Added module information to metadata and use by frontend_sever 2020-06-19 [email protected] [ddc] Normalize T to Never when T extends Never 2020-06-19 [email protected] [dart:io] HttpClientConnection is destroyed for "CONNECT" request 2020-06-19 [email protected] [dart2js] Add support for Never as a type literal. 2020-06-19 [email protected] [dart2js] Visit type parameter bounds in the scope visitor. 2020-06-19 [email protected] Migrate language_2/type_promotion to NNBD. 2020-06-19 [email protected] Add CiderCompletionComputer.warmUp() 2020-06-19 [email protected] Fix duration.toString() to conform the description If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dart-sdk-flutter-engine Please CC [email protected] on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
skia-flutter-autoroll
added a commit
to skia-flutter-autoroll/engine
that referenced
this issue
Jun 22, 2020
https://dart.googlesource.com/sdk.git/+log/9fcac032b669..dbefbc375376 2020-06-22 [email protected] [infra] Empty SDK commit to validate recipe changes 2020-06-21 [email protected] Add a fix to insert the keyword late 2020-06-21 [email protected] Convert the remaining quick assists to use CorrectionProducer 2020-06-20 [email protected] Enable v1.9 pedantic lints... 2020-06-20 [email protected] Revert "Reland : [VM/Runtime] Cleanup package config initialization code" 2020-06-20 [email protected] [cfe] Mark late setters as isGenericCovariantImpl 2020-06-20 [email protected] Add --track-widget-creation to kernel_front_end.dart 2020-06-20 [email protected] [benchmark] BigIntParsePrint: Hoist value in all loops 2020-06-20 [email protected] Reland : [VM/Runtime] Cleanup package config initialization code 2020-06-20 [email protected] [dart2js] final locals may be assigned 2020-06-20 [email protected] [VM/Runtime] - Fix for issue dart-lang/sdk#42421 2020-06-20 [email protected] Added module information to metadata and use by frontend_sever 2020-06-19 [email protected] [ddc] Normalize T to Never when T extends Never 2020-06-19 [email protected] [dart:io] HttpClientConnection is destroyed for "CONNECT" request 2020-06-19 [email protected] [dart2js] Add support for Never as a type literal. 2020-06-19 [email protected] [dart2js] Visit type parameter bounds in the scope visitor. 2020-06-19 [email protected] Migrate language_2/type_promotion to NNBD. 2020-06-19 [email protected] Add CiderCompletionComputer.warmUp() 2020-06-19 [email protected] Fix duration.toString() to conform the description If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dart-sdk-flutter-engine Please CC [email protected] on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
This was referenced Jun 22, 2020
skia-flutter-autoroll
added a commit
to skia-flutter-autoroll/engine
that referenced
this issue
Jun 22, 2020
https://dart.googlesource.com/sdk.git/+log/9fcac032b669..ed0eeaebcac2 2020-06-22 [email protected] [vm/aot] Move precompiled position handling to CodeSourceMap classes. 2020-06-22 [email protected] [infra] Empty SDK commit to validate recipe changes 2020-06-21 [email protected] Add a fix to insert the keyword late 2020-06-21 [email protected] Convert the remaining quick assists to use CorrectionProducer 2020-06-20 [email protected] Enable v1.9 pedantic lints... 2020-06-20 [email protected] Revert "Reland : [VM/Runtime] Cleanup package config initialization code" 2020-06-20 [email protected] [cfe] Mark late setters as isGenericCovariantImpl 2020-06-20 [email protected] Add --track-widget-creation to kernel_front_end.dart 2020-06-20 [email protected] [benchmark] BigIntParsePrint: Hoist value in all loops 2020-06-20 [email protected] Reland : [VM/Runtime] Cleanup package config initialization code 2020-06-20 [email protected] [dart2js] final locals may be assigned 2020-06-20 [email protected] [VM/Runtime] - Fix for issue dart-lang/sdk#42421 2020-06-20 [email protected] Added module information to metadata and use by frontend_sever 2020-06-19 [email protected] [ddc] Normalize T to Never when T extends Never 2020-06-19 [email protected] [dart:io] HttpClientConnection is destroyed for "CONNECT" request 2020-06-19 [email protected] [dart2js] Add support for Never as a type literal. 2020-06-19 [email protected] [dart2js] Visit type parameter bounds in the scope visitor. 2020-06-19 [email protected] Migrate language_2/type_promotion to NNBD. 2020-06-19 [email protected] Add CiderCompletionComputer.warmUp() 2020-06-19 [email protected] Fix duration.toString() to conform the description If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dart-sdk-flutter-engine Please CC [email protected] on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
skia-flutter-autoroll
added a commit
to skia-flutter-autoroll/engine
that referenced
this issue
Jun 22, 2020
https://dart.googlesource.com/sdk.git/+log/9fcac032b669..1d3fbbed7c06 2020-06-22 [email protected] Revert "Fix duration.toString() to conform the description" 2020-06-22 [email protected] [vm/aot] Move precompiled position handling to CodeSourceMap classes. 2020-06-22 [email protected] [infra] Empty SDK commit to validate recipe changes 2020-06-21 [email protected] Add a fix to insert the keyword late 2020-06-21 [email protected] Convert the remaining quick assists to use CorrectionProducer 2020-06-20 [email protected] Enable v1.9 pedantic lints... 2020-06-20 [email protected] Revert "Reland : [VM/Runtime] Cleanup package config initialization code" 2020-06-20 [email protected] [cfe] Mark late setters as isGenericCovariantImpl 2020-06-20 [email protected] Add --track-widget-creation to kernel_front_end.dart 2020-06-20 [email protected] [benchmark] BigIntParsePrint: Hoist value in all loops 2020-06-20 [email protected] Reland : [VM/Runtime] Cleanup package config initialization code 2020-06-20 [email protected] [dart2js] final locals may be assigned 2020-06-20 [email protected] [VM/Runtime] - Fix for issue dart-lang/sdk#42421 2020-06-20 [email protected] Added module information to metadata and use by frontend_sever 2020-06-19 [email protected] [ddc] Normalize T to Never when T extends Never 2020-06-19 [email protected] [dart:io] HttpClientConnection is destroyed for "CONNECT" request 2020-06-19 [email protected] [dart2js] Add support for Never as a type literal. 2020-06-19 [email protected] [dart2js] Visit type parameter bounds in the scope visitor. 2020-06-19 [email protected] Migrate language_2/type_promotion to NNBD. 2020-06-19 [email protected] Add CiderCompletionComputer.warmUp() 2020-06-19 [email protected] Fix duration.toString() to conform the description If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dart-sdk-flutter-engine Please CC [email protected] on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
skia-flutter-autoroll
added a commit
to skia-flutter-autoroll/engine
that referenced
this issue
Jun 22, 2020
https://dart.googlesource.com/sdk.git/+log/9fcac032b669..cc5df58bd229 2020-06-22 [email protected] [cfe] Refactor return type checking in preparation for nnbd spec change 2020-06-22 [email protected] Revert "Fix duration.toString() to conform the description" 2020-06-22 [email protected] [vm/aot] Move precompiled position handling to CodeSourceMap classes. 2020-06-22 [email protected] [infra] Empty SDK commit to validate recipe changes 2020-06-21 [email protected] Add a fix to insert the keyword late 2020-06-21 [email protected] Convert the remaining quick assists to use CorrectionProducer 2020-06-20 [email protected] Enable v1.9 pedantic lints... 2020-06-20 [email protected] Revert "Reland : [VM/Runtime] Cleanup package config initialization code" 2020-06-20 [email protected] [cfe] Mark late setters as isGenericCovariantImpl 2020-06-20 [email protected] Add --track-widget-creation to kernel_front_end.dart 2020-06-20 [email protected] [benchmark] BigIntParsePrint: Hoist value in all loops 2020-06-20 [email protected] Reland : [VM/Runtime] Cleanup package config initialization code 2020-06-20 [email protected] [dart2js] final locals may be assigned 2020-06-20 [email protected] [VM/Runtime] - Fix for issue dart-lang/sdk#42421 2020-06-20 [email protected] Added module information to metadata and use by frontend_sever 2020-06-19 [email protected] [ddc] Normalize T to Never when T extends Never 2020-06-19 [email protected] [dart:io] HttpClientConnection is destroyed for "CONNECT" request 2020-06-19 [email protected] [dart2js] Add support for Never as a type literal. 2020-06-19 [email protected] [dart2js] Visit type parameter bounds in the scope visitor. 2020-06-19 [email protected] Migrate language_2/type_promotion to NNBD. 2020-06-19 [email protected] Add CiderCompletionComputer.warmUp() 2020-06-19 [email protected] Fix duration.toString() to conform the description If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dart-sdk-flutter-engine Please CC [email protected] on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md
dart-bot
pushed a commit
that referenced
this issue
Jul 9, 2020
The autodetection was fixed in bbc8aed, so the workaround of passing --sound-null-safety to the AOT runtime is no longer needed. This change tests that the autodetection does in fact work. Bug: #42421 Change-Id: Ibbe9665905e817a28964b698e6c437897c9dc6ee Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/153771 Reviewed-by: Alexander Markov <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
/cc @a-siva
The text was updated successfully, but these errors were encountered: