dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.
https://docs.microsoft.com/dotnet/core/
MIT License
15.03k stars 4.68k forks source link

ARM64 - Follow-up on `x < 0` and `x >= 0` #83562

Open TIHan opened 1 year ago

TIHan commented 1 year ago

Since https://github.com/dotnet/runtime/pull/83176, there is a little more work to do.

  1. Add disasm tests
  2. Handle short types, so we can do tbz/tbnz reg, #15, IG and tbz/tbnz reg, #7, IG respectively with short and byte.
dotnet-issue-labeler[bot] commented 1 year ago

I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

ghost commented 1 year ago

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch, @kunalspathak See info in area-owners.md if you want to be subscribed.

Issue Details
Since https://github.com/dotnet/runtime/pull/83176, there is a little more work to do. 1. Add disasm tests 2. Handle short types, so we can do `tbz/tbnz reg, #15, block` or `tbz/tbnz reg, #7, block` respectively with short and byte.
Author: TIHan
Assignees: -
Labels: `tenet-performance`, `area-CodeGen-coreclr`, `untriaged`
Milestone: -