Mercurial > hg > truffle
comparison src/share/vm/opto/type.cpp @ 605:98cb887364d3
6810672: Comment typos
Summary: I have collected some typos I have found while looking at the code.
Reviewed-by: kvn, never
author | twisti |
---|---|
date | Fri, 27 Feb 2009 13:27:09 -0800 |
parents | 465813e0303a |
children | 337400e7a5dd |
comparison
equal
deleted
inserted
replaced
604:ec59443af135 | 605:98cb887364d3 |
---|---|
2453 // directly manipulates the addresses of oops. Rather, it only has a pointer | 2453 // directly manipulates the addresses of oops. Rather, it only has a pointer |
2454 // to a handle at compile time. This handle is embedded in the generated | 2454 // to a handle at compile time. This handle is embedded in the generated |
2455 // code and dereferenced at the time the nmethod is made. Until that time, | 2455 // code and dereferenced at the time the nmethod is made. Until that time, |
2456 // it is not reasonable to do arithmetic with the addresses of oops (we don't | 2456 // it is not reasonable to do arithmetic with the addresses of oops (we don't |
2457 // have access to the addresses!). This does not seem to currently happen, | 2457 // have access to the addresses!). This does not seem to currently happen, |
2458 // but this assertion here is to help prevent its occurrance. | 2458 // but this assertion here is to help prevent its occurence. |
2459 tty->print_cr("Found oop constant with non-zero offset"); | 2459 tty->print_cr("Found oop constant with non-zero offset"); |
2460 ShouldNotReachHere(); | 2460 ShouldNotReachHere(); |
2461 } | 2461 } |
2462 | 2462 |
2463 return (intptr_t)const_oop()->encoding(); | 2463 return (intptr_t)const_oop()->encoding(); |
2759 case NotNull: | 2759 case NotNull: |
2760 case BotPTR: // Fall down to object klass | 2760 case BotPTR: // Fall down to object klass |
2761 // LCA is object_klass, but if we subclass from the top we can do better | 2761 // LCA is object_klass, but if we subclass from the top we can do better |
2762 if( above_centerline(_ptr) ) { // if( _ptr == TopPTR || _ptr == AnyNull ) | 2762 if( above_centerline(_ptr) ) { // if( _ptr == TopPTR || _ptr == AnyNull ) |
2763 // If 'this' (InstPtr) is above the centerline and it is Object class | 2763 // If 'this' (InstPtr) is above the centerline and it is Object class |
2764 // then we can subclass in the Java class heirarchy. | 2764 // then we can subclass in the Java class hierarchy. |
2765 if (klass()->equals(ciEnv::current()->Object_klass())) { | 2765 if (klass()->equals(ciEnv::current()->Object_klass())) { |
2766 // that is, tp's array type is a subtype of my klass | 2766 // that is, tp's array type is a subtype of my klass |
2767 return TypeAryPtr::make(ptr, tp->ary(), tp->klass(), tp->klass_is_exact(), offset, instance_id); | 2767 return TypeAryPtr::make(ptr, tp->ary(), tp->klass(), tp->klass_is_exact(), offset, instance_id); |
2768 } | 2768 } |
2769 } | 2769 } |
3020 } | 3020 } |
3021 | 3021 |
3022 | 3022 |
3023 //------------------------------xdual------------------------------------------ | 3023 //------------------------------xdual------------------------------------------ |
3024 // Dual: do NOT dual on klasses. This means I do NOT understand the Java | 3024 // Dual: do NOT dual on klasses. This means I do NOT understand the Java |
3025 // inheritence mechanism. | 3025 // inheritance mechanism. |
3026 const Type *TypeInstPtr::xdual() const { | 3026 const Type *TypeInstPtr::xdual() const { |
3027 return new TypeInstPtr( dual_ptr(), klass(), klass_is_exact(), const_oop(), dual_offset(), dual_instance_id() ); | 3027 return new TypeInstPtr( dual_ptr(), klass(), klass_is_exact(), const_oop(), dual_offset(), dual_instance_id() ); |
3028 } | 3028 } |
3029 | 3029 |
3030 //------------------------------eq--------------------------------------------- | 3030 //------------------------------eq--------------------------------------------- |
3174 jint max_hi = max_array_length(elem()->basic_type()); | 3174 jint max_hi = max_array_length(elem()->basic_type()); |
3175 //if (index_not_size) --max_hi; // type of a valid array index, FTR | 3175 //if (index_not_size) --max_hi; // type of a valid array index, FTR |
3176 bool chg = false; | 3176 bool chg = false; |
3177 if (lo < min_lo) { lo = min_lo; chg = true; } | 3177 if (lo < min_lo) { lo = min_lo; chg = true; } |
3178 if (hi > max_hi) { hi = max_hi; chg = true; } | 3178 if (hi > max_hi) { hi = max_hi; chg = true; } |
3179 // Negative length arrays will produce weird intermediate dead fath-path code | 3179 // Negative length arrays will produce weird intermediate dead fast-path code |
3180 if (lo > hi) | 3180 if (lo > hi) |
3181 return TypeInt::ZERO; | 3181 return TypeInt::ZERO; |
3182 if (!chg) | 3182 if (!chg) |
3183 return size; | 3183 return size; |
3184 return TypeInt::make(lo, hi, Type::WidenMin); | 3184 return TypeInt::make(lo, hi, Type::WidenMin); |
3356 case NotNull: | 3356 case NotNull: |
3357 case BotPTR: // Fall down to object klass | 3357 case BotPTR: // Fall down to object klass |
3358 // LCA is object_klass, but if we subclass from the top we can do better | 3358 // LCA is object_klass, but if we subclass from the top we can do better |
3359 if (above_centerline(tp->ptr())) { | 3359 if (above_centerline(tp->ptr())) { |
3360 // If 'tp' is above the centerline and it is Object class | 3360 // If 'tp' is above the centerline and it is Object class |
3361 // then we can subclass in the Java class heirarchy. | 3361 // then we can subclass in the Java class hierarchy. |
3362 if( tp->klass()->equals(ciEnv::current()->Object_klass()) ) { | 3362 if( tp->klass()->equals(ciEnv::current()->Object_klass()) ) { |
3363 // that is, my array type is a subtype of 'tp' klass | 3363 // that is, my array type is a subtype of 'tp' klass |
3364 return make( ptr, _ary, _klass, _klass_is_exact, offset, instance_id ); | 3364 return make( ptr, _ary, _klass, _klass_is_exact, offset, instance_id ); |
3365 } | 3365 } |
3366 } | 3366 } |