Closed jimblandy closed 4 days ago
Use the more strictly-typed HandleSet for sets of arena handles, rather than the untyped BitSet.
HandleSet
BitSet
This depends on #5862, so it's marked as a draft PR for now.
[naga] Move HandleSet from compact into arena.
compact
arena
[naga] Use HandleSet in naga::proc::index.
naga::proc::index
Change naga::proc::index::find_checked_indexes to return a HandleSet<Expression>, rather than an untyped BitSet.
naga::proc::index::find_checked_indexes
HandleSet<Expression>
Fix uses in the Metal Shading Language backend.
[naga] Change HandleSet::insert to return a bool.
HandleSet::insert
bool
Change HandleSet::insert to return true if the handle was not already in the set.
true
Change uses in naga::compact which were taking advantage of the prior return type of () to avoid needing to use semicolons in match arms.
naga::compact
()
match
[naga] Use HandleSet in Validator::valid_expression_set.
Validator::valid_expression_set
Add HandleSet methods new, clear, clear_for_arena, and remove.
new
clear
clear_for_arena
remove
Change the type of naga::valid::Validator::valid_expression_set from BitSet to HandleSet. Adjust uses.
naga::valid::Validator::valid_expression_set
[naga spv-out] Use HandleSet in ExpressionConstnessTracker.
ExpressionConstnessTracker
Change naga::back::spv::ExpressionConstnessTracker::inner from a BitSet to a HandleSet<Expression>. Adjust uses.
naga::back::spv::ExpressionConstnessTracker::inner
[naga] Use HandleSet in Validator::needs_visit.
Validator::needs_visit
Change the type of naga::valid::Validator::needs_visit from BitSet to HandleSet. Adjust uses.
naga::valid::Validator::needs_visit
Add HandleSet method iter.
iter
This was also EZ to review. Kudos!
rebased, now that #5862 has been merged
Use the more strictly-typed
HandleSet
for sets of arena handles, rather than the untypedBitSet
.This depends on #5862, so it's marked as a draft PR for now.
[naga] Move
HandleSet
fromcompact
intoarena
.[naga] Use
HandleSet
innaga::proc::index
.Change
naga::proc::index::find_checked_indexes
to return aHandleSet<Expression>
, rather than an untypedBitSet
.Fix uses in the Metal Shading Language backend.
[naga] Change
HandleSet::insert
to return abool
.Change
HandleSet::insert
to returntrue
if the handle was not already in the set.Change uses in
naga::compact
which were taking advantage of the prior return type of()
to avoid needing to use semicolons inmatch
arms.[naga] Use
HandleSet
inValidator::valid_expression_set
.Add
HandleSet
methodsnew
,clear
,clear_for_arena
, andremove
.Change the type of
naga::valid::Validator::valid_expression_set
fromBitSet
toHandleSet
. Adjust uses.[naga spv-out] Use
HandleSet
inExpressionConstnessTracker
.Change
naga::back::spv::ExpressionConstnessTracker::inner
from aBitSet
to aHandleSet<Expression>
. Adjust uses.[naga] Use
HandleSet
inValidator::needs_visit
.Change the type of
naga::valid::Validator::needs_visit
fromBitSet
toHandleSet
. Adjust uses.Add
HandleSet
methoditer
.