tree d8d435fdf6d64d7ab28c05ae8be304800afa3e19
parent cc5cda0f997d8280712920f2749bbdfe7a0f4dc3
author Nicolas Capens <capn@google.com> 1589392644 -0400
committer Nicolas Capens <nicolascapens@google.com> 1590442907 +0000

Eliminate the array size from rr::Variable

We only need to know the array size during materialization of Array<>
types, so store it only for that class and make the allocation a
virtual method.

This change removes the REACTOR_MATERIALIZE_LVALUES_ON_DEFINITION
option, since it requires knowing the full size of Variable at
construction, which would require Array<> to still pass that into the
Variable constructor. If we need an debugging feature like this again,
we could also materialize variables on first use instead of at
construction.

Bug: b/155302798
Change-Id: I65b02c1258643dcbd3ae18f3f097cb815efa0830
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/45148
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>
Kokoro-Result: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>