Request for reviews (S): 7192965: assert(is_aligned_sets(size)) failed: mask is not aligned, adjacent sets

[Christian Thalinger]

On Aug 22, 2012, at 3:20 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:

> http://cr.openjdk.java.net/~kvn/7192965/webrev
> 
> 7192965: assert(is_aligned_sets(size)) failed: mask is not aligned, adjacent sets
> 
> Bias coloring in RA does not work for vectors with size > 64 bits, it only works for pairs (64 bits).
> Change pair check to vector check. I did small refactoring and added comment.
> 
> Tested with failing test.

src/share/vm/opto/chaitin.cpp:

+ bool is_legal_reg(LRG &lrg, OptoReg::Name reg, int chunk) {

Should it be a static method?

-- Chris

> 
> Thanks,
> Vladimir