src/share/vm/c1/c1_LIRGenerator.cpp

@@ -392,11 +392,11 @@
 
 // This is called for each node in tree; the walk stops if a root is reached
 void LIRGenerator::walk(Value instr) {
   InstructionMark im(compilation(), instr);
   //stop walk when encounter a root
-  if (instr->is_pinned() && instr->as_Phi() == NULL || instr->operand()->is_valid()) {
+  if ((instr->is_pinned() && instr->as_Phi() == NULL) || instr->operand()->is_valid()) {
     assert(instr->operand() != LIR_OprFact::illegalOpr || instr->as_Constant() != NULL, "this root has not yet been visited");
   } else {
     assert(instr->subst() == instr, "shouldn't have missed substitution");
     instr->visit(this);
     // assert(instr->use_count() > 0 || instr->as_Phi() != NULL, "leaf instruction must have a use");

@@ -1432,10 +1432,12 @@
         if (c->as_jint_lo_bits() != other->as_jint_lo_bits()) continue;
         break;
       case T_OBJECT:
         if (c->as_jobject() != other->as_jobject()) continue;
         break;
+      default:
+        break;
       }
       return _reg_for_constants.at(i);
     }
   }
 

@@ -2802,10 +2804,12 @@
     case T_BOOLEAN:
     case T_SHORT:
     case T_CHAR:
       t = T_INT;
       break;
+    default:
+      break;
     }
 
     LIR_Opr dest = new_register(t);
     __ move(src, dest);