-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[Arm64] Vector Load/Store structure instructions #33461
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
echesakov
merged 24 commits into
dotnet:master
from
echesakov:Arm64-Vector-Load-Store-Structure-Instructions
Mar 14, 2020
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
e31f305
Update ld1 in instrsarm64.h
echesakov cddcb09
Add ld2, ld3, ld4, st1, st2, st3, st4 in instrsarm64.h
echesakov d11a956
Add ld1, st1 operating on multiple registers in instrsarm64.h
echesakov 66cb52e
Add ld1r, ld2r, ld3r, ld4r in instrsarm64.h
echesakov d591c2b
Remove EN4J, add EN6B and EN3J in emitarm64.cpp emitfmtsarm64.h
echesakov b559403
Update LS_2D, LS_2E, LS_3F, LS_3G and add LS_2F, LS_2G in emitfmtsarm…
echesakov e8524fe
Add Arm64 emitter unit tests for "Load/Store Vector" instructions in …
echesakov b4aa743
Add emitter::emitDispElemsize in emitarm64.cpp emitarm64.h
echesakov 4e1ce14
Update functions' headers in emitarm64.cpp
echesakov 9cca19b
Add emitDispVectorRegList and emitDispVectorElemList in emitarm64.cpp…
echesakov cd88213
Add insGetLoadStoreVectorSelem in emitarm64.cpp emitarm64.h
echesakov 8fe0715
Update emitIns_R_R in emitarm64.cpp
echesakov e7ce778
Update emitIns_R_R_I in emitarm64.cpp
echesakov b493842
Update emitIns_R_R_I in emitarm64.cpp
echesakov ee4415b
Update emitIns_R_R_R in emitarm64.cpp
echesakov bdc9df6
Update emitIns_R_R_R_I in emitarm64.cpp
echesakov 66103d5
Update emitIns_R_R_I_I in emitarm64.cpp emitarm64.h
echesakov a0bfd42
Update emitDispIns in emitarm64.cpp
echesakov e77156b
Update emitOutputInstr in emitarm64.cpp
echesakov 43aed1d
Update emitInsSanityCheck in emitarm64.cpp
echesakov 683a20d
Update emitInsMayWriteToGCReg in emitarm64.cpp
echesakov 6a53804
Remove ld1 in emitInsTargetRegSize in emitarm64.cpp
echesakov 1e482c5
Update getMemoryOperation and getInsExecutionCharacteristics in emit.…
echesakov c7af7d2
Address Tanner's feedback on GitHub.
echesakov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Add ld2, ld3, ld4, st1, st2, st3, st4 in instrsarm64.h
- Loading branch information
commit cddcb09276d2aa8e8335fdf5db7fa6bfdf16e63d
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if it would be helpful to include a comment for each of these that would lead you to the correct reference manual page, e.g., "C7.2.174 LD2 (single structure)" (hopefully ARM doesn't renumber these... but at least the name would be the same)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it's a good idea - I will add