dotnet/runtime

Cleanup RegSet::verifyRegUsed

Open

#13,508 opened on 2019年9月30日

GitHub で見る
 (3 comments) (0 reactions) (0 assignees)C# (5,445 forks)batch import
area-CodeGen-coreclrhelp wanted

Repository metrics

Stars
 (17,886 stars)
PR merge metrics
 (平均マージ 12d 11h) (30d で 661 merged PRs)

説明

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

コントリビューターガイド