Perf Improvements:

[wmtorode]

StratOps (in particular Resupply helper methods) is showing high up in perf flame graphs of AI think time. Alot of time is spent:

• Creating and updating TagSets
• checking if a TagSet contains a resupply tag

To fix this:

• add a new resupply unit stat to the actor's stat collection
• on add unit, check the actor's tags, if resupply tag is present, mark it as a resupply unit.
• remove all other checks for the resupply tag, check the stat instead. This stops tagset duplication and reduces checks for its existence to a single Dictionary lookup and boolean check, instead of iterating over a list.
• Found that GetAllFriendlies & GetAllDetectedEnemies methods already filter out dead and marked for death units, removed spots where the output of these methods is filtered via linq on these same criteria, avoiding a new enumerable being created, filtered and calls that consume time checking if a unit is dead.

Overall the hope is that this creates a noticable improvement to combat/AI perf.

Notes for future:

• See if there is away to improve GetAllFriendlies & GetAllDetectedEnemies, maybe caching or something? they are frequently called and do a fair number of checks and list creation overhead.