Mercurial > hg > graal-compiler
comparison src/share/vm/gc_implementation/g1/g1ErgoVerbose.hpp @ 3927:f1b4e0e0bdad
7089625: G1: policy for how many old regions to add to the CSet (when young gen is fixed) is broken
Summary: When refactoring the code for a previous fix, a condition was not correctly negated which prevents the G1 policy from adding the correct number of old regions to the CSet when the young gen size is fixed. The changeset also fixes a small syntactical issue in g1ErgoVerbose.hpp which is causing compiler warnings.
Reviewed-by: brutisso, ysr
author | tonyp |
---|---|
date | Tue, 13 Sep 2011 12:40:14 -0400 |
parents | 20213c8a3c40 |
children | 41406797186b |
comparison
equal
deleted
inserted
replaced
3926:140317da459a | 3927:f1b4e0e0bdad |
---|---|
56 ErgoLevelMask = ~((1 << ErgoLevelShift) - 1), | 56 ErgoLevelMask = ~((1 << ErgoLevelShift) - 1), |
57 | 57 |
58 // ErgoLow is 0 so that we don't have to explicitly or a heuristic | 58 // ErgoLow is 0 so that we don't have to explicitly or a heuristic |
59 // id with ErgoLow to keep its use simpler. | 59 // id with ErgoLow to keep its use simpler. |
60 ErgoLow = 0, | 60 ErgoLow = 0, |
61 ErgoHigh = 1 << ErgoLevelShift, | 61 ErgoHigh = 1 << ErgoLevelShift |
62 } ErgoLevel; | 62 } ErgoLevel; |
63 | 63 |
64 // The available heuristics. | 64 // The available heuristics. |
65 typedef enum { | 65 typedef enum { |
66 // Determines which part of the tag is occupied by the heuristic id. | 66 // Determines which part of the tag is occupied by the heuristic id. |