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

Unified Diff: sdk/lib/_internal/compiler/js_lib/js_helper.dart

Issue 1091613003: Revert "dart2js: add compiler builtins to the core-runtime." (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: sdk/lib/_internal/compiler/js_lib/js_helper.dart
diff --git a/sdk/lib/_internal/compiler/js_lib/js_helper.dart b/sdk/lib/_internal/compiler/js_lib/js_helper.dart
index defef1ed853b62b3be71b187b60addc588987a7f..bb22a2fdce13d5954ddecb7bc395941c2aa2fca4 100644
--- a/sdk/lib/_internal/compiler/js_lib/js_helper.dart
+++ b/sdk/lib/_internal/compiler/js_lib/js_helper.dart
@@ -16,7 +16,6 @@ import 'dart:_js_embedded_names' show
INTERCEPTORS_BY_TAG,
IS_HUNK_LOADED,
IS_HUNK_INITIALIZED,
- JsBuiltin,
JsGetName,
LEAF_TAGS,
METADATA,
@@ -43,20 +42,22 @@ import 'dart:async' show
import 'dart:_foreign_helper' show
DART_CLOSURE_TO_JS,
JS,
- JS_BUILTIN,
JS_CALL_IN_ISOLATE,
JS_CONST,
JS_CURRENT_ISOLATE,
JS_CURRENT_ISOLATE_CONTEXT,
+ JS_DART_OBJECT_CONSTRUCTOR,
JS_EFFECT,
JS_EMBEDDED_GLOBAL,
+ JS_FUNCTION_CLASS_NAME,
JS_FUNCTION_TYPE_NAMED_PARAMETERS_TAG,
JS_FUNCTION_TYPE_OPTIONAL_PARAMETERS_TAG,
JS_FUNCTION_TYPE_REQUIRED_PARAMETERS_TAG,
JS_FUNCTION_TYPE_RETURN_TYPE_TAG,
+ JS_FUNCTION_TYPE_TAG,
JS_FUNCTION_TYPE_VOID_RETURN_TAG,
- JS_GET_FLAG,
JS_GET_NAME,
+ JS_GET_FLAG,
JS_HAS_EQUALS,
JS_IS_INDEXABLE_FIELD_NAME,
JS_NULL_CLASS_NAME,
@@ -92,47 +93,6 @@ part 'linked_hash_map.dart';
abstract class InternalMap {
}
-/// Returns true if the given [type] is a function type object.
-// TODO(floitsch): move this to foreign_helper.dart or similar.
-@ForceInline()
-bool isDartFunctionType(Object type) {
- return JS_BUILTIN('returns:bool;effects:none;depends:none',
- JsBuiltin.isFunctionType, type);
-}
-
-
-/// Creates a function type object.
-// TODO(floitsch): move this to foreign_helper.dart or similar.
-@ForceInline()
-bool createDartFunctionType() {
- return JS_BUILTIN('returns:=Object;effects:none;depends:none',
- JsBuiltin.createFunctionType);
-}
-
-/// Returns true if the given [type] is _the_ `Function` type.
-// TODO(floitsch): move this to foreign_helper.dart or similar.
-@ForceInline()
-bool isDartFunctionTypeLiteral(Object type) {
- return JS_BUILTIN('returns:bool;effects:none;depends:none',
- JsBuiltin.isFunctionTypeLiteral, type);
-}
-
-/// Retrieves the class name from type information stored on the constructor of
-/// [type].
-// TODO(floitsch): move this to foreign_helper.dart or similar.
-@ForceInline()
-String getDartTypeName(Object type) {
- return JS_BUILTIN('String', JsBuiltin.typeName, type);
-}
-
-/// Returns the raw runtime type of the given object [o].
-// TODO(floitsch): move this to foreign_helper.dart or similar.
-@ForceInline()
-Object getRawRuntimeType(Object o) {
- return JS_BUILTIN('', JsBuiltin.rawRuntimeType, getInterceptor(o));
-}
-
-
/// No-op method that is called to inform the compiler that preambles might
/// be needed when executing the resulting JS file in a command-line
/// JS engine.
@@ -3155,7 +3115,7 @@ class RuntimeFunctionType extends RuntimeType {
}
toRti() {
- var result = createDartFunctionType();
+ var result = JS('=Object', '{ #: "dynafunc" }', JS_FUNCTION_TYPE_TAG());
if (isVoid) {
JS('', '#[#] = true', result, JS_FUNCTION_TYPE_VOID_RETURN_TAG());
} else {
« no previous file with comments | « sdk/lib/_internal/compiler/js_lib/foreign_helper.dart ('k') | sdk/lib/_internal/compiler/js_lib/js_mirrors.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698