Skip to content

Conversation

@trdthg
Copy link
Contributor

@trdthg trdthg commented Dec 19, 2024

this PR rebase from #226, and add support for zicbop and zicbop

the branch name is a typo, please don't mind

@trdthg trdthg changed the title add support for cbo.zero Add Zicbom and Zicbop extension Dec 19, 2024
@trdthg trdthg changed the title Add Zicbom and Zicbop extension [ACT] Add support for CMO Dec 19, 2024
@UmerShahidengr
Copy link
Collaborator

@trdthg the recent merge has created a merge conflict with this one, please resolve the merge conflict and update the branch. I will merge this one

@trdthg
Copy link
Contributor Author

trdthg commented Jan 20, 2025

src and dst operands to ADDI immediate must be different.
Replace with #if (imm<2048) addi swreg,swreg,imm) #else LI(imm,tmp); addi swreg,swreg,tmp; #endif

@allenjbaum I actually don't quite understand this, but I tried to updated the logic of TEST_PREFETCH (even though the sail doesn't have caches)

#define TEST_PREFETCH(swreg,testreg,rs1,inst,imm1,imm2)                       ;\

LI(rs1,imm1&(RVMODEL_CMO_BLOCKSIZE-1))                                   ;\
add rs1,rs1,swreg                                                           ;\

LI(testreg,imm2)			                                                ;\ 
sub rs1,rs1,testreg                                                         ;\

inst imm2(rs1)                                                           ;\
nop                                                                         ;\
nop                                                                         ;\

ADDI(swreg, swreg, RVMODEL_CMO_BLOCKSIZE)

@jamesbeyond jamesbeyond added unprivileged ratified ratified specs size-S Small efforts required dependencies Pull requests that update a dependency file labels Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file ratified ratified specs size-S Small efforts required unprivileged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants