Rust-GPU / Rust-GPU/rust-gpu

[Migrated] debug_printf requires unsafe

オープン
#96 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
Rust
スター
3.4k
フォーク
126
PR マージ指標
30日以内にマージされた PR はありません

説明

Issue automatically imported from old repo: https://github.com/EmbarkStudios/rust-gpu/issues/997
Old labels: t: enhancement
Originally creatd by charles-r-earp on 2023-03-04T00:38:43Z


The debug_printf and debug_printfln macros generate an invocation of the asm macro, which requires an unsafe block. However, the macro appears to validate it's input, so it could potentially add the unsafe block to it's output. If there are still outstanding issues, perhaps a safety section could be added to the docs to reduce confusion? This is a really useful tool, and it seems strange to require unsafe just to debug stuff.

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

debug_printf と debug_printfln のマクロ定義、およびそれらが呼び出す asm マクロから始めてください。入力検証が unsafe 要件とどのように関係するかを確認し、そのうえで、完了に必要なのが生成される呼び出しの変更なのか、安全性に関する想定の文書化なのかを判断してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
rust
領域
compilers
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
活発
明瞭さ
説明が足りない
初心者へのやさしさ
42/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。