C2: multiple slices for flattened value type arrays

Tobias Hartmann tobias.hartmann at oracle.com
Thu Jan 12 14:54:58 UTC 2017


Hi Roland,

On 10.01.2017 15:50, Roland Westrelin wrote:
> http://cr.openjdk.java.net/~roland/valhalla/flattenvtarray/webrev.00/

Very nice!

I noticed some style issues you may want to correct since you changed those lines anyway:
- Excess newlines that can be removed. For example, type.hpp:1208, type.cpp:4486, type.cpp:4594
- Wrong indentations. For example, type.cpp:3071
- Excess/missing whitespace around parentheses. For example:
  - TypeRawPtr( PTR ptr, address bits ) -> TypeRawPtr(PTR ptr, address bits)
  - assert( offset() >= 0, "" ) -> assert(offset() >= 0, "")
  - _bits(bits){} -> _bits(bits) { }
- Pointer asterisk position. For example:
  - static const TypePtr *make(..) -> static const TypePtr* make(..)

You don't need to send a new webrev for that or we can also do that cleanup later.

Thanks,
Tobias

> 
> Roland.
> 



More information about the valhalla-dev mailing list