apery_rust icon indicating copy to clipboard operation
apery_rust copied to clipboard

Slower than Apery C++ version

Open HiraokaTakuya opened this issue 5 years ago • 5 comments

The repository of Apery C++ version is https://github.com/HiraokaTakuya/apery.git Apery C++ version is more than 20% faster.

This is the repository of NPS comparison scripts. https://github.com/HiraokaTakuya/apery_nps_comparison.git

HiraokaTakuya avatar Jun 18 '19 17:06 HiraokaTakuya

  • lto = true : https://github.com/mizar/apery_rust/commit/24ae08448e7b044a7ec859668ad52f00370dbb09
  • target-cpu=native : https://github.com/mizar/apery_rust/commit/075d9f32a5c4e34d0d571b71766274ac3cef5281

例えば上記のような指定をしてみるのはどうでしょうか?

数%程度影響がありそうです。

mizar avatar Dec 24 '19 17:12 mizar

確かに設定した方が良いですね。 ltoはビルドが遅くなりすぎるので一旦別にした方が良いかも知れません。 あとmizarさんの変更にリンクが付けられているのやっと気づきました! ありがとうございます!

HiraokaTakuya avatar Dec 26 '19 16:12 HiraokaTakuya

そちらの develop/20191225/buildopt ブランチを取り込んでも良いでしょうか?

HiraokaTakuya avatar Dec 26 '19 16:12 HiraokaTakuya

https://github.com/HiraokaTakuya/apery_rust/pull/9 一応、PRの形で投げておきました。

mizar avatar Dec 26 '19 17:12 mizar

ありがとうございます!取り込ませて頂きました!

HiraokaTakuya avatar Dec 26 '19 17:12 HiraokaTakuya