cache std::thread::hardware_concurrency() result at start up

[t-horikawa]

https://github.com/project-tsurugi/tsurugi-issues/issues/405

[ban-nobuhiro]

良いと思います。
プロセス実行中にコア数が変化することはあるにはあるのですが、この元のコードの頻度で再確認する必要はないはずです。

また std::thread::hardware_concurrency() から 0 が返ってきた場合に
https://en.cppreference.com/w/cpp/thread/thread/hardware_concurrency
正常に動作しないという別の問題もありますが、 PR の範囲外のことなので、別途直そうかと思います。

[ban-nobuhiro]

この部分は Debug ビルドの時にしか通らず、通常用途では使われないので、次に控えているリリースに急いで載せる必要はないかと思います。

[ban-nobuhiro]

@kuron99 特に期限は設けていませんでしたがレビュータイムアウトということで通します。

[kuron99]

すみません、リクエストに気づいていませんでした。コードは問題ないと思います。

[ban-nobuhiro]

また std::thread::hardware_concurrency() から 0 が返ってきた場合に
正常に動作しないという別の問題もありますが、PR の範囲外のことなので、別途直そうかと思います。

この点を見返してみたのですが、0 が返ってきても、全直列で動くだけで、問題というほどのことではなかったため、そのままにしました。

[t-horikawa]

はい、「std::thread::hardware_concurrency() から 0が返ってきた場合は全直列で動くだけ」です。多分1でもそうなるのではないかと思います。