changeset in /hg/icedtea6: 2009-01-12 Andrew Haley <aph at redhat...

[Andrew Haley]

changeset e34ba0ba2281 in /hg/icedtea6
details: http://icedtea.classpath.org/hg/icedtea6?cmd=changeset;node=e34ba0ba2281
description:
	2009-01-12  Andrew Haley  <aph at redhat.com>

	        * ports/hotspot/src/share/vm/shark/sharkBlock.cpp (SharkBlock::do_ldc):
	        Add logic for ldc(Class).

diffstat:

2 files changed, 29 insertions(+), 2 deletions(-)
ChangeLog                                       |    5 ++++
ports/hotspot/src/share/vm/shark/sharkBlock.cpp |   26 +++++++++++++++++++++--

diffs (72 lines):

diff -r bce8d56dfa03 -r e34ba0ba2281 ChangeLog
--- a/ChangeLog	Sun Jan 11 18:56:44 2009 +0100
+++ b/ChangeLog	Mon Jan 12 17:29:21 2009 +0000
@@ -1,3 +1,8 @@ 2009-01-11  Mark Wielaard  <mark at klomp.o
+2009-01-12  Andrew Haley  <aph at redhat.com>
+
+	* ports/hotspot/src/share/vm/shark/sharkBlock.cpp (SharkBlock::do_ldc):
+        Add logic for ldc(Class).
+
 2009-01-11  Mark Wielaard  <mark at klomp.org>
 
 	* Makefile.am: Escape single quotes in empty bootclasspath.
diff -r bce8d56dfa03 -r e34ba0ba2281 ports/hotspot/src/share/vm/shark/sharkBlock.cpp
--- a/ports/hotspot/src/share/vm/shark/sharkBlock.cpp	Sun Jan 11 18:56:44 2009 +0100
+++ b/ports/hotspot/src/share/vm/shark/sharkBlock.cpp	Mon Jan 12 17:29:21 2009 +0000
@@ -1236,10 +1236,11 @@ void SharkBlock::do_ldc()
     SharkConstantPool constants(this);
 
     BasicBlock *resolved   = function()->CreateBlock("resolved");
+    BasicBlock *resolved_class = function()->CreateBlock("resolved_class");
     BasicBlock *unresolved = function()->CreateBlock("unresolved");
     BasicBlock *unknown    = function()->CreateBlock("unknown");
     BasicBlock *done       = function()->CreateBlock("done");
-    
+
     SwitchInst *switchinst = builder()->CreateSwitch(
       constants.tag_at(iter()->get_constant_index()),
       unknown, 5);
@@ -1247,7 +1248,7 @@ void SharkBlock::do_ldc()
     switchinst->addCase(
       LLVMValue::jbyte_constant(JVM_CONSTANT_String), resolved);
     switchinst->addCase(
-      LLVMValue::jbyte_constant(JVM_CONSTANT_Class), resolved);
+      LLVMValue::jbyte_constant(JVM_CONSTANT_Class), resolved_class);
     switchinst->addCase(
       LLVMValue::jbyte_constant(JVM_CONSTANT_UnresolvedString), unresolved);
     switchinst->addCase(
@@ -1260,6 +1261,26 @@ void SharkBlock::do_ldc()
     Value *resolved_value = constants.object_at(iter()->get_constant_index());
     builder()->CreateBr(done);
 
+    builder()->SetInsertPoint(resolved_class);
+    Value *resolved_class_value
+      = constants.object_at(iter()->get_constant_index());
+    resolved_class_value
+      = builder()->CreatePtrToInt(resolved_class_value,
+                                  SharkType::intptr_type());
+    resolved_class_value
+      = (builder()->CreateAdd
+         (resolved_class_value,
+          (LLVMValue::intptr_constant
+           (klassOopDesc::klass_part_offset_in_bytes()
+            + Klass::java_mirror_offset_in_bytes()))));
+    resolved_class_value
+      // FIXME: We need a memory barrier before this load.
+      = (builder()->CreateLoad
+         (builder()->CreateIntToPtr
+          (resolved_class_value,
+           PointerType::getUnqual(SharkType::jobject_type()))));
+    builder()->CreateBr(done);
+
     builder()->SetInsertPoint(unresolved);
     builder()->CreateUnimplemented(__FILE__, __LINE__);
     Value *unresolved_value = LLVMValue::null();
@@ -1272,6 +1293,7 @@ void SharkBlock::do_ldc()
     builder()->SetInsertPoint(done);
     PHINode *phi = builder()->CreatePHI(SharkType::jobject_type(), "constant");
     phi->addIncoming(resolved_value, resolved);
+    phi->addIncoming(resolved_class_value, resolved_class);
     phi->addIncoming(unresolved_value, unresolved);
     value = SharkValue::create_jobject(phi);
   }