- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Wed, Mar 29
Tue, Mar 28
Mon, Mar 27
Associated review: https://reviews.llvm.org/D144925
Associated review: https://reviews.llvm.org/D144925
Associated review: https://reviews.llvm.org/D144925
Associated review: https://reviews.llvm.org/D144925
Associated review: https://reviews.llvm.org/D144925
Also, sorry for not maintaining a 1-1 relation between my commits and review, I'll do that in the future!
Sat, Mar 25
Fri, Mar 24
Fixed so that the policy for vecreduce instructions is ta, ma + some refactoring
Thu, Mar 23
Tue, Mar 21
Attempting to fix the diff
Wed, Mar 15
Refactoring and removing comments
Diff updated to add another commit that was missed last time (Sorry, never used commit ranges with arcanist before.)
Created a diff from a commit range this time
Tue, Mar 14
Made the SchedReads and SchedWrites sew-aware
ReadVRGatherVX -> ReadVRGather_(data|index), ReadVRGatherVI -> ReadVRGatherVI_data
Tue, Mar 7
Mon, Mar 6
Feb 28 2023
In D144925#4157772, @frasercrmck wrote:Just a glancing review but, to me, "expand pseudos" has a more immediate connotation of the many pseudo expansion passes (e.g., RISCVExpandPseudoInsts) than "expanding the set of pseudos" as you seem to intend it.
Feb 27 2023
Feb 21 2023
Dec 22 2022
Added the VT_MASKC/N changes
Pulled from main
Pulled from main
Dec 14 2022
Fixed some documentation nits
Dec 13 2022
Formatting fixed
Removed the TODO
Fixed previous erroneous revision, linked parent revision via commit message
The patch has been rebased and the test checks have been updated.
In D139948#3992911, @craig.topper wrote:In D139948#3992908, @jrtc27 wrote:I don't understand this, your patch subject says you're adding a pass to remove the W suffix from instructions, but the actual diff doesn't add a pass and the test changes all add W suffices
The last update looks like it should have gone to D139462
Updated the test checks