Integrated: 8256453: C2: Reduce State footprint

[Claes Redestad]

On Tue, 17 Nov 2020 09:00:46 GMT, Claes Redestad <redestad at openjdk.org> wrote:

> State objects are pretty heavy-weight. On linux-x64 sizeof(State) = 2344
> 
> Much of this is due to the two 32-bit int arrays _cost and _rule.
> 
> The number of rules vary by platform, but seem to hover around 1k. A 16-bit integer seem more than enough to map all rules, so making _rule an uint16_t array saves significant space.
> 
> Also it seems we can fold the _valid bit vector into _rule and reduce the size even further. This also considerably simplify the common validity checks, which more than makes up for needing to initialize more memory when setting up the State object.
> 
> Those two optimizations reduce sizeof(State) down to 1736. Profiling compilations show a tiny win overall and instrumenting some simple compilations I see a sharp decline in Arena::grow events caused by ResourceObj allocation in Matcher::match_tree and Matcher::Label_Root

This pull request has now been integrated.

Changeset: f7f34474
Author:    Claes Redestad <redestad at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/f7f34474
Stats:     111 lines in 3 files changed: 18 ins; 37 del; 56 mod

8256453: C2: Reduce State footprint

Reviewed-by: neliasso, kvn

-------------

PR: https://git.openjdk.java.net/jdk/pull/1253