sayurin
commented
8 years ago KCS_CUI -i "sample\3-5.json" "sample\3-5 high.map" -f 0 0 -n 10000 -t 4 -o output.jsonのプロファイル結果です。
Hot Pathは確認されていたかもしれませんが、Functions With Most Indivisual Workも重要です。
mainCRTStartupは後述しますが起動時のCSV処理の部分です。
まずVisual C++版std::vectorのcapacity()管理はかなり酷いです。
std::vector<int> v;
std::wcout << L"vector.size() = " << v.size() << L", vector.capacity() = " << v.capacity() << std::endl;
v.push_back(1);
std::wcout << L"vector.size() = " << v.size() << L", vector.capacity() = " << v.capacity() << std::endl;
v.push_back(1);
std::wcout << L"vector.size() = " << v.size() << L", vector.capacity() = " << v.capacity() << std::endl;
v.push_back(1);
std::wcout << L"vector.size() = " << v.size() << L", vector.capacity() = " << v.capacity() << std::endl;
v.push_back(1);
std::wcout << L"vector.size() = " << v.size() << L", vector.capacity() = " << v.capacity() << std::endl;
v.push_back(1);
std::wcout << L"vector.size() = " << v.size() << L", vector.capacity() = " << v.capacity() << std::endl;
v.push_back(1);
std::wcout << L"vector.size() = " << v.size() << L", vector.capacity() = " << v.capacity() << std::endl;
v.push_back(1);
std::wcout << L"vector.size() = " << v.size() << L", vector.capacity() = " << v.capacity() << std::endl;これの実行結果が
vector.size() = 0, vector.capacity() = 0
vector.size() = 1, vector.capacity() = 1
vector.size() = 2, vector.capacity() = 2
vector.size() = 3, vector.capacity() = 3
vector.size() = 4, vector.capacity() = 4
vector.size() = 5, vector.capacity() = 6
vector.size() = 6, vector.capacity() = 6
vector.size() = 7, vector.capacity() = 9となっていて、capacity()が増えるたびにoperator new[]が呼び出されているわけです。サイズをある程度見積もってreserve()するだけでstd::_Allocate、operator newの呼び出し回数を削減できるでしょう。
更にはstd::_Allocate、operator new、std::vector::vector、これらは個別の関数ですがよくよく見るとstd::vectorコピーコンストラクターの延長上で呼び出されるもので、要するにシミュレーション全体の20%はstd::vectorの操作とコピーということになります。意図しないstd::vectorコピーが大量に発生しているのではないかと。
std::vectorのコピーが発生しているということはWeapon、Kammusu、Fleet辺りのオブジェクトもコピーされまくっているということで、ところどころにあるmove()はほとんど効いていません。その結果、CSV読み込み部分もかなりのコピー＆deleteとなっていて処理コストが上がっています。
yumetodo
YSRKEN
乱数の範囲調整を高速化できないか(よく使うdistribusionをキャッシュするとか(_Evalのほうが時間かかってるからあまり意味ない？))RandBoolの高速化(50%の確率の時、distribusionをキャッシュしたら早くなったりしないか)Simulator::DetermineAttackOrderのpush_backの前にreserveSimulator::AirWarPhaseのvector<int> anti_air_scoreをarrayにFleet::RandomKammusuSSのvector<size_t> list,vector<KammusuIndex> alived_listのarray化Simulator::TorpedoPhaseのvector<vector<int>> all_damageのarray化Fleet::SearchValueの高速化(無理じゃね？)Simulator::Calcのreturn文のstd::make_tupleでvectorがコピーされている・・・？Result::JudgeWinReasonからvectorを排除するどんぶり勘定で計算時間の半分以上が乱数生成なのでネタ切れ感がある。重たかった文字列検索は @sayurin 氏のpull requestをマージしたことでぐんと下がったし、シュミレーションの一部キャッシュもそれなりに効いている
高速化の方針？