tree a8826048603489328e6c94fcc19445ba109c141a
parent 13ef87caab19353a9f3e97ad9ff1c6bbbabd74a5
author Antonio Maiorano <amaiorano@google.com> 1618535743 +0000
committer Commit Bot service account <commit-bot@chromium.org> 1618535743 +0000

Resolver: improve array handling

type::Array was the only type with special handling in that we'd resolve
all array types first in ResolverInternal, then proceed with resolving
the AST in order of declaration. This CL removes this special handling,
and instead, we now process Arrays as we process ast::Variables of
array type. This change also allows us to pass down a Source location
for validation messages when processing Arrays.

Updated some Builder tests that weren't creating a variable of the array
type they declared.

Bug: tint:707
Change-Id: I8483b3a979bc1e5e04feb1ca4d281e96e9e654be
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/47825
Kokoro: Kokoro <noreply+kokoro@google.com>
Commit-Queue: Antonio Maiorano <amaiorano@google.com>
Reviewed-by: Ben Clayton <bclayton@chromium.org>