dotnet/runtime

Cleanup RegSet::verifyRegUsed

Open

#13,508 建立於 2019年9月30日

在 GitHub 查看
 (3 留言) (0 反應) (0 負責人)C# (5,445 fork)batch import
area-CodeGen-coreclrhelp wanted

倉庫指標

Star
 (17,886 star)
PR 合併指標
 (平均合併 12天 11小時) (30 天內合併 661 個 PR)

描述

verifyRegUsed() is supposed to be an assertion that the register is marked as used. However, its implementation is changing the register mask instead. This needs to be clean up.

dotnet/coreclr#18296 was an attempt to fix it, but it was wrong and incomplete. We need a better solution than that.

category:implementation theme:jit-coding-style skill-level:beginner cost:small

貢獻者指南

研究方向
檢查JIT編譯器程式碼庫中的verifyRegUsed函式，確保它僅進行斷言而不修改暫存器遮罩。審查dotnet/coreclr#18296中的先前嘗試以了解陷阱。
技術棧
csharp
領域
compilers
議題類型
重構
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
1
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1 小時以內
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
活躍
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
清晰
前置要求
GitC#
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
85