dotnet/runtime

[JIT] Possible optimization for `x % {pow2_cns}` transformation for `ulong` types

Geschlossen

#81.189 geöffnet am 25.01.2023

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C# (5.445 Forks)batch import
area-CodeGen-coreclrhelp wantedtenet-performance

Repository-Metriken

Stars
 (17.886 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 12T 11h) (661 gemergte PRs in 30 T)

Beschreibung

At the moment, we do not optimize x % {pow2_cns} to x & (pow2_cns - 1) for ulong types as they get transformed differently in morph.cpp. I believe we should only do this for X64 and not X86.

Contributor Guide

Research-Richtung
Untersuchen Sie morph.cpp im dotnet/runtime Repository, um die bestehende Optimierung für `x % pow2 cns` für vorzeichenbehaftete und vorzeichenlose Ganzzahltypen zu verstehen. Das Problem weist darauf hin, dass ulong Typen nicht in `x & (pow2 cns 1)` transformiert werden, da sie anders behandelt werden. Überprüfen Sie den Kommentar im Issue für zusätzlichen Kontext. Stellen Sie sicher, dass die Optimierung nur für X64, nicht für X86, angewendet wird, und verifizieren Sie dies durch Tests.
Tech Stack
csharpcpp
Domain
backendperformance
Issue Type
Performance
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
4
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++ programmingJIT compiler internalsx64 architecture basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
35