Revert "Introduce IKG into kernel-service to support incremental compilation."

runtime/vm/unit_test.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/unit_test.h"
 
 #include <stdio.h>
 
 #include "bin/builtin.h"
 #include "bin/dartutils.h"
@@ skipping 146 matching lines @@
 static Dart_Handle ResolvePackageUri(const char* uri_chars) {
   const int kNumArgs = 1;
   Dart_Handle dart_args[kNumArgs];
   dart_args[0] = DartUtils::NewString(uri_chars);
   return Dart_Invoke(DartUtils::BuiltinLib(),
                      DartUtils::NewString("_filePathFromUri"), kNumArgs,
                      dart_args);
 }
 
 static ThreadLocalKey script_reload_key = kUnsetThreadLocalKey;
-static ThreadLocalKey kernel_reload_key = kUnsetThreadLocalKey;
 
 char* TestCase::CompileTestScriptWithDFE(const char* url,
                                          const char* source,
-                                         void** kernel_pgm,
-                                         bool incrementally) {
+                                         void** kernel_pgm) {
+  Zone* zone = Thread::Current()->zone();
   // clang-format off
   Dart_SourceFile sourcefiles[] = {
     {
       url, source,
     },
     {
       "file:///.packages", "untitled:/"
     }};
   // clang-format on
-  return CompileTestScriptWithDFE(url,
-                                  sizeof(sourcefiles) / sizeof(Dart_SourceFile),
-                                  sourcefiles, kernel_pgm, incrementally);
-}
-
-char* TestCase::CompileTestScriptWithDFE(const char* url,
-                                         int sourcefiles_count,
-                                         Dart_SourceFile sourcefiles[],
-                                         void** kernel_pgm,
-                                         bool incrementally) {
-  Zone* zone = Thread::Current()->zone();
-  Dart_KernelCompilationResult compilation_result = Dart_CompileSourcesToKernel(
-      url, sourcefiles_count, sourcefiles, incrementally);
+  int sourcefiles_count = sizeof(sourcefiles) / sizeof(Dart_SourceFile);
+  Dart_KernelCompilationResult compilation_result =
+      Dart_CompileSourcesToKernel(url, sourcefiles_count, sourcefiles);
 
   if (compilation_result.status != Dart_KernelCompilationStatus_Ok) {
     return OS::SCreate(zone, "Compilation failed %s", compilation_result.error);
   }
   const uint8_t* kernel_file = compilation_result.kernel;
   intptr_t kernel_length = compilation_result.kernel_size;
   if (kernel_file == NULL) {
     return OS::SCreate(zone, "front end generated a NULL kernel file");
   }
   *kernel_pgm = Dart_ReadKernelBinary(kernel_file, kernel_length);
   if (*kernel_pgm == NULL) {
     return OS::SCreate(zone, "Failed to read generated kernel binary");
   }
   return NULL;
 }
 
 static Dart_Handle LibraryTagHandler(Dart_LibraryTag tag,
                                      Dart_Handle library,
                                      Dart_Handle url) {
   if (FLAG_use_dart_frontend) {
     // Reload request.
-
+    ASSERT(script_reload_key != kUnsetThreadLocalKey);
+    const char* script_source = reinterpret_cast<const char*>(
+        OSThread::GetThreadLocal(script_reload_key));
+    ASSERT(script_source != NULL);
+    OSThread::SetThreadLocal(script_reload_key, 0);
     const char* urlstr = NULL;
     Dart_Handle result = Dart_StringToCString(url, &urlstr);
     if (Dart_IsError(result)) {
       return Dart_NewApiError("accessing url characters failed");
     }
-
-    // Updated library either arrives as dart source or as
-    // a precompiled kernel binary.
     void* kernel_pgm;
-    if (script_reload_key != kUnsetThreadLocalKey) {
-      const char* script_source = reinterpret_cast<const char*>(
-          OSThread::GetThreadLocal(script_reload_key));
-      ASSERT(script_source != NULL);
-      OSThread::SetThreadLocal(script_reload_key, 0);
-      char* error = TestCase::CompileTestScriptWithDFE(urlstr, script_source,
-                                                       &kernel_pgm);
-      if (error != NULL) {
-        return Dart_NewApiError(error);
-      }
+    char* error =
+        TestCase::CompileTestScriptWithDFE(urlstr, script_source, &kernel_pgm);
+    if (error == NULL) {
+      return Dart_LoadScript(url, Dart_Null(),
+                             reinterpret_cast<Dart_Handle>(kernel_pgm), 0, 0);
     } else {
-      ASSERT(kernel_reload_key != kUnsetThreadLocalKey);
-      kernel_pgm =
-          reinterpret_cast<void*>(OSThread::GetThreadLocal(kernel_reload_key));
-      ASSERT(kernel_pgm != NULL);
-      OSThread::SetThreadLocal(kernel_reload_key, 0);
+      return Dart_NewApiError(error);
     }
-    return Dart_LoadScript(url, Dart_Null(),
-                           reinterpret_cast<Dart_Handle>(kernel_pgm), 0, 0);
   }
   if (tag == Dart_kCanonicalizeUrl) {
     Dart_Handle library_url = Dart_LibraryUrl(library);
     if (Dart_IsError(library_url)) {
       return library_url;
     }
     return Dart_DefaultCanonicalizeUrl(library_url, url);
   }
   if (tag == Dart_kScriptTag) {
     // Reload request.
@@ skipping 87 matching lines @@
   DART_CHECK_VALID(lib);
   result = Dart_SetNativeResolver(lib, resolver, NULL);
   DART_CHECK_VALID(result);
   if (finalize_classes) {
     result = Dart_FinalizeLoading(false);
     DART_CHECK_VALID(result);
   }
   return lib;
 }
 
+static Dart_Handle LoadTestScriptWithDFE(const char* script,
+                                         Dart_NativeEntryResolver resolver,
+                                         const char* lib_url,
+                                         bool finalize_classes) {
+  Dart_Handle url = NewString(lib_url);
+  Dart_Handle result = Dart_SetLibraryTagHandler(LibraryTagHandler);
+  EXPECT_VALID(result);
+  void* kernel_pgm = NULL;
+  char* error =
+      TestCase::CompileTestScriptWithDFE(lib_url, script, &kernel_pgm);
+  if (error == NULL) {
+    Dart_Handle lib = Dart_LoadScript(
+        url, Dart_Null(), reinterpret_cast<Dart_Handle>(kernel_pgm), 0, 0);
+    DART_CHECK_VALID(lib);
+    result = Dart_SetNativeResolver(lib, resolver, NULL);
+    DART_CHECK_VALID(result);
+    if (finalize_classes) {
+      result = Dart_FinalizeLoading(false);
+      DART_CHECK_VALID(result);
+    }
+    return lib;
+  } else {
+    return Dart_NewApiError(error);
+  }
+}
+
 Dart_Handle TestCase::LoadTestScript(const char* script,
                                      Dart_NativeEntryResolver resolver,
                                      const char* lib_url,
                                      bool finalize_classes) {
-  return FLAG_use_dart_frontend
-             ? LoadTestScriptWithDFE(
-                   1,
-                   (Dart_SourceFile[1]){{OS::SCreate(Thread::Current()->zone(),
-                                                     "file:///%s", lib_url),
-                                         script}},
-                   resolver, finalize_classes)
-             : LoadTestScriptWithVMParser(script, resolver, lib_url,
-                                          finalize_classes);
-}
-
-Dart_Handle TestCase::LoadTestScriptWithDFE(int sourcefiles_count,
-                                            Dart_SourceFile sourcefiles[],
-                                            Dart_NativeEntryResolver resolver,
-                                            bool finalize,
-                                            bool incrementally) {
-  // First script is the main script.
-  Dart_Handle url = NewString(sourcefiles[0].uri);
-  Dart_Handle result = Dart_SetLibraryTagHandler(LibraryTagHandler);
-  EXPECT_VALID(result);
-  void* kernel_pgm = NULL;
-  char* error = TestCase::CompileTestScriptWithDFE(
-      sourcefiles[0].uri, sourcefiles_count, sourcefiles, &kernel_pgm,
-      incrementally);
-  if (error != NULL) {
-    return Dart_NewApiError(error);
+  if (!FLAG_use_dart_frontend) {
+    return LoadTestScriptWithVMParser(script, resolver, lib_url,
+                                      finalize_classes);
+  } else {
+    Zone* zone = Thread::Current()->zone();
+    char* resolved_lib_url = OS::SCreate(zone, "file:///%s", lib_url);
+    return LoadTestScriptWithDFE(script, resolver, resolved_lib_url,
+                                 finalize_classes);
   }
-  Dart_Handle lib = Dart_LoadScript(
-      url, Dart_Null(), reinterpret_cast<Dart_Handle>(kernel_pgm), 0, 0);
-  DART_CHECK_VALID(lib);
-  result = Dart_SetNativeResolver(lib, resolver, NULL);
-  DART_CHECK_VALID(result);
-  if (finalize) {
-    result = Dart_FinalizeLoading(false);
-    DART_CHECK_VALID(result);
-  }
-  return lib;
 }
 
 #ifndef PRODUCT
 
 void TestCase::SetReloadTestScript(const char* script) {
   if (script_reload_key == kUnsetThreadLocalKey) {
     script_reload_key = OSThread::CreateThreadLocal();
   }
   ASSERT(script_reload_key != kUnsetThreadLocalKey);
   ASSERT(OSThread::GetThreadLocal(script_reload_key) == 0);
   // Store the new script in TLS.
   OSThread::SetThreadLocal(script_reload_key, reinterpret_cast<uword>(script));
 }
 
-void TestCase::SetReloadTestKernel(const void* kernel) {
-  if (kernel_reload_key == kUnsetThreadLocalKey) {
-    kernel_reload_key = OSThread::CreateThreadLocal();
-  }
-  ASSERT(kernel_reload_key != kUnsetThreadLocalKey);
-  ASSERT(OSThread::GetThreadLocal(kernel_reload_key) == 0);
-  // Store the new script in TLS.
-  OSThread::SetThreadLocal(kernel_reload_key, reinterpret_cast<uword>(kernel));
-}
-
 Dart_Handle TestCase::TriggerReload() {
   Isolate* isolate = Isolate::Current();
   JSONStream js;
   bool success = false;
   {
     TransitionNativeToVM transition(Thread::Current());
     success = isolate->ReloadSources(&js,
                                      false,  // force_reload
                                      NULL, NULL,
                                      true);  // dont_delete_reload_context
@@ skipping 34 matching lines @@
     TransitionNativeToVM transition(thread);
     Isolate* isolate = thread->isolate();
     if (isolate->reload_context() != NULL) {
       isolate->DeleteReloadContext();
     }
   }
 
   return result;
 }
 
-Dart_Handle TestCase::ReloadTestKernel(const void* kernel) {
-  SetReloadTestKernel(kernel);
-
-  Dart_Handle result = TriggerReload();
-  if (Dart_IsError(result)) {
-    return result;
-  }
-
-  result = GetReloadErrorOrRootLibrary();
-
-  {
-    Thread* thread = Thread::Current();
-    TransitionNativeToVM transition(thread);
-    Isolate* isolate = thread->isolate();
-    if (isolate->reload_context() != NULL) {
-      isolate->DeleteReloadContext();
-    }
-  }
-
-  return result;
-}
-
 #endif  // !PRODUCT
 
 Dart_Handle TestCase::LoadCoreTestScript(const char* script,
                                          Dart_NativeEntryResolver resolver) {
   return LoadTestScript(script, resolver, CORELIB_TEST_URI);
 }
 
 Dart_Handle TestCase::lib() {
   Dart_Handle url = NewString(TestCase::url());
   Dart_Handle lib = Dart_LookupLibrary(url);
@@ skipping 118 matching lines @@
     pos = strstr(in, prefix);
   }
   // Copy the remainder of in to out.
   while (*in != '\0') {
     *out++ = *in++;
   }
   *out = '\0';
 }
 
 }  // namespace dart