`blake2` code size very high (overuse of`#[inline(always)]`?)
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- リファクタリング
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- rust
- 領域
- cryptography
調査の方向性
blake2/src/blake2.rs の VarBlake2b::compress から始め、#[inline(always)] アノテーションと手動でアンロールされたラウンド呼び出しに注目してください。-Copt-level=z と -Copt-level=3 で生成されたコードを比較し、そのうえで、実装が引き続き同じ Blake2b の結果を生成しながら、異常に大きい関数サイズを削減できていることを確認してください。
索引モデルが issue の本文から書いたものです。
説明
I have ~150kb of code for blake2::blake2b::VarBlake2b::compress under -Copt-level=z, which is around 3x larger than any other function in my binary. It would be nice if it were smaller.
Looking at https://github.com/RustCrypto/hashes/blob/master/blake2/src/blake2.rs, a lot is #[inline(always)], and the round calls are manually unrolled.
There's a good chance these would still get inlined and unrolled on -Copt-level=3 if it were written using #[inline] and a loop, but it would make a big difference to use cases where large code is undesirable.
- 主要言語
- Rust
- スター
- 2.3k
- フォーク
- 341
- 平均マージ
- 12時間 31分
- マージ済み PR（30日）
- 2
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
RustCrypto/hashes のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
RustCrypto/hashes#912 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 38/100
RustCrypto/hashes#909 · コメント 4 件 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
RustCrypto/hashes#908 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
RustCrypto/hashes#763 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
RustCrypto/hashes#747 · コメント 1 件 ·
RustCrypto/hashes の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
-
bug core
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
-
JIT-compiled number -> Decimal conversion silently overflows instead of raising DECIMAL_OVERFLOW オープンfuzz
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
ClickHouse/ClickHouse#122114 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
linebender/vello_svg#90 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100