-
Notifications
You must be signed in to change notification settings - Fork 5.4k
[RyuJIT] Enable contained form for shlx, sarx, shrx and on x86 #67314
Copy link
Copy link
Closed
Labels
area-CodeGen-coreclrCLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMICLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMIhelp wanted[up-for-grabs] Good issue for external contributors[up-for-grabs] Good issue for external contributorsin-prThere is an active PR which will close this issue when it is mergedThere is an active PR which will close this issue when it is merged
Milestone
Metadata
Metadata
Assignees
Labels
area-CodeGen-coreclrCLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMICLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMIhelp wanted[up-for-grabs] Good issue for external contributors[up-for-grabs] Good issue for external contributorsin-prThere is an active PR which will close this issue when it is mergedThere is an active PR which will close this issue when it is merged
Type
Fields
Give feedbackNo fields configured for issues without a type.
Related to #41881.
#67182 handles non contained form of shifts such that all three operands are registers.
Extend it to support contained form based on this comment.
Edit: Also, enable emitting the three instructions on x86. Need to check lsraxarch and emitter part.
category:implementation
theme:basic-cq
skill-level:intermediate
cost:medium
impact:medium