From e09ae0920be57760fb390b6944bce420fa0b5582 Mon Sep 17 00:00:00 2001 From: Mathieu Chartier Date: Thu, 15 May 2014 12:39:19 -0700 Subject: Fix an outstanding compaction bug in interpreter. Fixed a bug in DoFieldPut where the FieldHelper GetType could cause thread suspension which would result in a stale obj. Added more handles in the class linker to facilitate moving fiels and methods in the future. Removed un-necessarly passing handle references since these are value types and don't need to be passed by reference. Added a special NullHandle type which allows null handles without a handle scope. Change-Id: I1b51723920a2e4f4f8b2907066f578a3e879fd5b --- compiler/driver/compiler_driver_test.cc | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'compiler/driver/compiler_driver_test.cc') diff --git a/compiler/driver/compiler_driver_test.cc b/compiler/driver/compiler_driver_test.cc index 4efd27d476..964dfeb5b1 100644 --- a/compiler/driver/compiler_driver_test.cc +++ b/compiler/driver/compiler_driver_test.cc @@ -152,10 +152,9 @@ TEST_F(CompilerDriverTest, AbstractMethodErrorStub) { jobject class_loader; { ScopedObjectAccess soa(Thread::Current()); - StackHandleScope<1> hs(soa.Self()); - auto null_loader(hs.NewHandle(nullptr)); - CompileVirtualMethod(null_loader, "java.lang.Class", "isFinalizable", "()Z"); - CompileDirectMethod(null_loader, "java.lang.Object", "", "()V"); + CompileVirtualMethod(NullHandle(), "java.lang.Class", "isFinalizable", + "()Z"); + CompileDirectMethod(NullHandle(), "java.lang.Object", "", "()V"); class_loader = LoadDex("AbstractMethod"); } ASSERT_TRUE(class_loader != NULL); -- cgit v1.2.3-59-g8ed1b