-
Notifications
You must be signed in to change notification settings - Fork 167
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
Adding Zcb implementation #322
Changes from 1 commit
f4317ad
bde4caf
29280ad
e19a6b2
c35a554
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,7 +16,6 @@ else | |
endif | ||
|
||
SAIL_FLEN := riscv_flen_D.sail | ||
SAIL_VLEN := riscv_vlen.sail | ||
|
||
# Instruction sources, depending on target | ||
SAIL_CHECK_SRCS = riscv_addr_checks_common.sail riscv_addr_checks.sail riscv_misa_ext.sail | ||
|
@@ -29,6 +28,8 @@ SAIL_DEFAULT_INST += riscv_insts_zbb.sail | |
SAIL_DEFAULT_INST += riscv_insts_zbc.sail | ||
SAIL_DEFAULT_INST += riscv_insts_zbs.sail | ||
|
||
SAIL_DEFAULT_INST += riscv_insts_zcb.sail | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It would be good to keep this list sorted alphabetically, i.e. move it up one. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This has been updated. |
||
|
||
SAIL_DEFAULT_INST += riscv_insts_zfh.sail | ||
# Zfa needs to be added after fext, dext and Zfh (as it needs | ||
# definitions from those) | ||
|
@@ -42,15 +43,6 @@ SAIL_DEFAULT_INST += riscv_insts_zbkx.sail | |
|
||
SAIL_DEFAULT_INST += riscv_insts_zicond.sail | ||
|
||
SAIL_DEFAULT_INST += riscv_insts_vext_utils.sail | ||
SAIL_DEFAULT_INST += riscv_insts_vext_vset.sail | ||
SAIL_DEFAULT_INST += riscv_insts_vext_arith.sail | ||
SAIL_DEFAULT_INST += riscv_insts_vext_fp.sail | ||
SAIL_DEFAULT_INST += riscv_insts_vext_mem.sail | ||
SAIL_DEFAULT_INST += riscv_insts_vext_mask.sail | ||
SAIL_DEFAULT_INST += riscv_insts_vext_vm.sail | ||
SAIL_DEFAULT_INST += riscv_insts_vext_red.sail | ||
|
||
SAIL_SEQ_INST = $(SAIL_DEFAULT_INST) riscv_jalr_seq.sail | ||
SAIL_RMEM_INST = $(SAIL_DEFAULT_INST) riscv_jalr_rmem.sail riscv_insts_rmem.sail | ||
|
||
|
@@ -59,7 +51,6 @@ SAIL_RMEM_INST_SRCS = riscv_insts_begin.sail $(SAIL_RMEM_INST) riscv_insts_end.s | |
|
||
# System and platform sources | ||
SAIL_SYS_SRCS = riscv_csr_map.sail | ||
SAIL_SYS_SRCS += riscv_vext_control.sail # helpers for the 'V' extension | ||
SAIL_SYS_SRCS += riscv_next_regs.sail | ||
SAIL_SYS_SRCS += riscv_sys_exceptions.sail # default basic helpers for exception handling | ||
SAIL_SYS_SRCS += riscv_sync_exception.sail # define the exception structure used in the model | ||
|
@@ -79,12 +70,11 @@ SAIL_VM_SRCS += $(SAIL_RV64_VM_SRCS) | |
endif | ||
|
||
# Non-instruction sources | ||
PRELUDE = prelude.sail prelude_mapping.sail $(SAIL_XLEN) $(SAIL_FLEN) $(SAIL_VLEN) prelude_mem_metadata.sail prelude_mem.sail | ||
PRELUDE = prelude.sail prelude_mapping.sail $(SAIL_XLEN) $(SAIL_FLEN) prelude_mem_metadata.sail prelude_mem.sail | ||
|
||
SAIL_REGS_SRCS = riscv_reg_type.sail riscv_freg_type.sail riscv_regs.sail riscv_pc_access.sail riscv_sys_regs.sail | ||
SAIL_REGS_SRCS += riscv_pmp_regs.sail riscv_pmp_control.sail | ||
SAIL_REGS_SRCS += riscv_ext_regs.sail $(SAIL_CHECK_SRCS) | ||
SAIL_REGS_SRCS += riscv_vreg_type.sail riscv_vext_regs.sail | ||
|
||
SAIL_ARCH_SRCS = $(PRELUDE) | ||
SAIL_ARCH_SRCS += riscv_types_common.sail riscv_types_ext.sail riscv_types.sail | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,297 @@ | ||
/*=======================================================================================*/ | ||
/* RISCV Sail Model */ | ||
/* */ | ||
/* This Sail RISC-V architecture model, comprising all files and */ | ||
/* directories except for the snapshots of the Lem and Sail libraries */ | ||
/* in the prover_snapshots directory (which include copies of their */ | ||
/* licences), is subject to the BSD two-clause licence below. */ | ||
/* */ | ||
/* Copyright (c) 2017-2023 */ | ||
/* Prashanth Mundkur */ | ||
/* Rishiyur S. Nikhil and Bluespec, Inc. */ | ||
/* Jon French */ | ||
/* Brian Campbell */ | ||
/* Robert Norton-Wright */ | ||
/* Alasdair Armstrong */ | ||
/* Thomas Bauereiss */ | ||
/* Shaked Flur */ | ||
/* Christopher Pulte */ | ||
/* Peter Sewell */ | ||
/* Alexander Richardson */ | ||
/* Hesham Almatary */ | ||
/* Jessica Clarke */ | ||
/* Microsoft, for contributions by Robert Norton-Wright and Nathaniel Wesley Filardo */ | ||
/* Peter Rugg */ | ||
/* Aril Computer Corp., for contributions by Scott Johnson */ | ||
/* Nambi JU */ | ||
/* */ | ||
/* All rights reserved. */ | ||
/* */ | ||
/* This software was developed by the above within the Rigorous */ | ||
/* Engineering of Mainstream Systems (REMS) project, partly funded by */ | ||
/* EPSRC grant EP/K008528/1, at the Universities of Cambridge and */ | ||
/* Edinburgh. */ | ||
/* */ | ||
/* This software was developed by SRI International and the University of */ | ||
/* Cambridge Computer Laboratory (Department of Computer Science and */ | ||
/* Technology) under DARPA/AFRL contract FA8650-18-C-7809 ("CIFV"), and */ | ||
/* under DARPA contract HR0011-18-C-0016 ("ECATS") as part of the DARPA */ | ||
/* SSITH research programme. */ | ||
/* */ | ||
/* This project has received funding from the European Research Council */ | ||
/* (ERC) under the European Union’s Horizon 2020 research and innovation */ | ||
/* programme (grant agreement 789108, ELVER). */ | ||
/* */ | ||
/* */ | ||
/* Redistribution and use in source and binary forms, with or without */ | ||
/* modification, are permitted provided that the following conditions */ | ||
/* are met: */ | ||
/* 1. Redistributions of source code must retain the above copyright */ | ||
/* notice, this list of conditions and the following disclaimer. */ | ||
/* 2. Redistributions in binary form must reproduce the above copyright */ | ||
/* notice, this list of conditions and the following disclaimer in */ | ||
/* the documentation and/or other materials provided with the */ | ||
/* distribution. */ | ||
/* */ | ||
/* THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' */ | ||
/* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED */ | ||
/* TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A */ | ||
/* PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR */ | ||
/* CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, */ | ||
/* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT */ | ||
/* LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF */ | ||
/* USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND */ | ||
/* ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, */ | ||
/* OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT */ | ||
/* OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF */ | ||
/* SUCH DAMAGE. */ | ||
/*=======================================================================================*/ | ||
|
||
/* ****************************************************************** */ | ||
/* This file specifies the instructions in the 'ZCEE' extension. */ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should this say Zcb not ZCEE? |
||
/* ****************************************************************** */ | ||
|
||
/* ------------------------------------------------------------------ */ | ||
/* Sail Model for the c.l and c.s encoding group */ | ||
/* ------------------------------------------------------------------ */ | ||
|
||
/* ****************************************************************** */ | ||
union clause ast = C_LBU : (bits(2), cregidx, cregidx) | ||
|
||
mapping clause encdec_compressed = C_LBU(ui65, rs1, rd) | ||
<-> 0b100 @ 0b000 @ rs1 : cregidx @ ui65 : bits(2) @ rd : cregidx @ 0b00 | ||
|
||
function clause execute (C_LBU(uimm, rsc, rdc)) = { | ||
let immext : xlenbits = zero_extend(uimm); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is unused |
||
let imm : bits(12) = zero_extend(uimm); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think this is wrong. The
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Swizzle the bits in decode not execute |
||
|
||
let rd = creg2reg_idx(rdc); | ||
let rs = creg2reg_idx(rsc); | ||
execute(LOAD(imm, rs, rd, true, BYTE, false, false)) | ||
} | ||
|
||
mapping clause assembly = C_LBU(uimm, rsc, rdc) | ||
<-> "c.lbu" ^ spc() ^ creg_name(rdc) ^ sep() ^ creg_name(rsc) ^ sep() ^ hex_bits_5(uimm @ 0b000) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
/* ****************************************************************** */ | ||
union clause ast = C_LHU : (bits(1), cregidx, cregidx) | ||
|
||
mapping clause encdec_compressed = C_LHU(ui5, rs1, rd) | ||
<-> 0b100 @ 0b001 @ rs1 : cregidx @ 0b0 @ ui5 : bits(1) @ rd : cregidx @ 0b00 | ||
|
||
function clause execute (C_LHU(uimm, rsc, rdc)) = { | ||
let immext : xlenbits = zero_extend(uimm); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also unused. Please check the other instructions too. |
||
let imm : bits(12) = zero_extend(uimm); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also wrong - should be Please check the other instructions too. |
||
|
||
let rd = creg2reg_idx(rdc); | ||
let rs = creg2reg_idx(rsc); | ||
|
||
execute(LOAD(imm, rs, rd, true, HALF, false, false)) | ||
} | ||
|
||
mapping clause assembly = C_LHU(uimm, rsc, rdc) | ||
<-> "c.lhu" ^ spc() ^ creg_name(rdc) ^ sep() ^ creg_name(rsc) ^ sep() ^ hex_bits_4(uimm @ 0b000) | ||
/* ****************************************************************** */ | ||
union clause ast = C_LH : (bits(1), cregidx, cregidx) | ||
|
||
mapping clause encdec_compressed = C_LH(ui5, rs1, rd) | ||
<-> 0b100 @ 0b001 @ rs1 : cregidx @ 0b1 @ ui5 : bits(1) @ rd : cregidx @ 0b00 | ||
|
||
function clause execute (C_LH(uimm, rsc, rdc)) = { | ||
let immext : xlenbits = zero_extend(uimm); | ||
let imm : bits(12) = zero_extend(uimm); | ||
|
||
let rd = creg2reg_idx(rdc); | ||
let rs = creg2reg_idx(rsc); | ||
|
||
let rsext = X(rs) + immext; | ||
X(rs) = rsext; | ||
execute(LOAD(imm, rs, rd, false, HALF, false, false)) | ||
} | ||
|
||
mapping clause assembly = C_LH(uimm, rsc, rdc) | ||
<-> "c.lh" ^ spc() ^ creg_name(rdc) ^ sep() ^ creg_name(rsc) ^ sep() ^ hex_bits_4(uimm @ 0b000) | ||
/* ****************************************************************** */ | ||
|
||
union clause ast = C_SB : (bits(2), cregidx, cregidx) | ||
|
||
mapping clause encdec_compressed = C_SB(ui65, rs1, rs2) | ||
<-> 0b100 @ 0b010 @ rs1 : cregidx @ ui65 : bits(2) @ rs2 : cregidx @ 0b00 | ||
|
||
function clause execute (C_SB(uimm, rs1c, rs2c)) = { | ||
let immext : xlenbits = zero_extend(uimm); | ||
let imm : bits(12) = zero_extend(uimm); | ||
|
||
let rs1c = creg2reg_idx(rs1c); | ||
let rs2c = creg2reg_idx(rs2c); | ||
execute(STORE(imm, rs2c, rs1c, BYTE, false, false)) | ||
} | ||
|
||
mapping clause assembly = C_SB(uimm, rs1, rs2) | ||
<-> "c.sb" ^ spc() ^ creg_name(rs2) ^ sep() ^ creg_name(rs1) ^ sep() ^ hex_bits_5(uimm @ 0b000) | ||
/* ****************************************************************** */ | ||
union clause ast = C_SH : (bits(1), cregidx, cregidx) | ||
|
||
mapping clause encdec_compressed = C_SH(ui5, rs1, rs2) | ||
<-> 0b100 @ 0b011 @ rs1 : cregidx @ 0b0 @ ui5 : bits(1) @ rs2 : cregidx @ 0b00 | ||
|
||
function clause execute (C_SH(uimm, rs1c, rs2c)) = { | ||
let immext : xlenbits = zero_extend(uimm); | ||
let imm : bits(12) = zero_extend(uimm); | ||
|
||
let rs2c = creg2reg_idx(rs2c); | ||
let rs1c = creg2reg_idx(rs1c); | ||
execute(STORE(imm, rs2c, rs1c, HALF, false, false)) | ||
} | ||
|
||
mapping clause assembly = C_SH(uimm, rs1, rs2) | ||
<-> "c.sh" ^ spc() ^ creg_name(rs2) ^ sep() ^ creg_name(rs1) ^ sep() ^ hex_bits_4(uimm @ 0b000) | ||
/* ****************************************************************** */ | ||
|
||
/* ------------------------------------------------------------------ */ | ||
/* Sail Model for the c.zext and c.sext encoding group */ | ||
/* ------------------------------------------------------------------ */ | ||
/* ****************************************************************** */ | ||
union clause ast = C_ZEXT_B : (cregidx) | ||
|
||
mapping clause encdec_compressed = C_ZEXT_B(rsd) | ||
<-> 0b100 @ 0b111 @ rsd : cregidx @ 0b11 @ 0b000 @ 0b01 | ||
|
||
function clause execute (C_ZEXT_B(rsd)) = { | ||
let rsdc = creg2reg_idx(rsd); | ||
|
||
X(rsdc) = zero_extend(X(rsdc)[7..0]); | ||
RETIRE_SUCCESS | ||
} | ||
|
||
mapping clause assembly = C_ZEXT_B(rsd) | ||
<-> "c.zext.b" ^ spc() ^ creg_name(rsd) | ||
|
||
/* ****************************************************************** */ | ||
union clause ast = C_SEXT_B : (cregidx) | ||
|
||
mapping clause encdec_compressed = C_SEXT_B(rsd) | ||
<-> 0b100 @ 0b111 @ rsd : cregidx @ 0b11 @ 0b001 @ 0b01 | ||
|
||
function clause execute (C_SEXT_B(rsd)) = { | ||
let rsdc = creg2reg_idx(rsd); | ||
|
||
X(rsdc) = sign_extend(X(rsdc)[7..0]); | ||
RETIRE_SUCCESS | ||
} | ||
|
||
mapping clause assembly = C_SEXT_B(rsd) | ||
<-> "c.sext.b" ^ spc() ^ creg_name(rsd) | ||
/* ****************************************************************** */ | ||
union clause ast = C_ZEXT_H : (cregidx) | ||
|
||
mapping clause encdec_compressed = C_ZEXT_H(rsd) | ||
<-> 0b100 @ 0b111 @ rsd : cregidx @ 0b11 @ 0b010 @ 0b01 | ||
|
||
function clause execute (C_ZEXT_H(rsd)) = { | ||
let rsdc = creg2reg_idx(rsd); | ||
|
||
X(rsdc) = zero_extend(X(rsdc)[15..0]); | ||
RETIRE_SUCCESS | ||
} | ||
|
||
mapping clause assembly = C_ZEXT_H(rsd) | ||
<-> "c.zext.h" ^ spc() ^ creg_name(rsd) | ||
|
||
/* ****************************************************************** */ | ||
union clause ast = C_SEXT_H : (cregidx) | ||
|
||
mapping clause encdec_compressed = C_SEXT_H(rsd) | ||
<-> 0b100 @ 0b111 @ rsd : cregidx @ 0b11 @ 0b011 @ 0b01 | ||
|
||
function clause execute (C_SEXT_H(rsd)) = { | ||
let rsdc = creg2reg_idx(rsd); | ||
|
||
X(rsdc) = sign_extend(X(rsdc)[15..0]); | ||
RETIRE_SUCCESS | ||
} | ||
|
||
mapping clause assembly = C_SEXT_H(rsd) | ||
<-> "c.sext.h" ^ spc() ^ creg_name(rsd) | ||
/* ****************************************************************** */ | ||
union clause ast = C_ZEXT_W : (cregidx) | ||
|
||
mapping clause encdec_compressed = C_ZEXT_W(rsd) | ||
<-> 0b100 @ 0b111 @ rsd : cregidx @ 0b11 @ 0b100 @ 0b01 | ||
|
||
function clause execute (C_ZEXT_W(rsd)) = { | ||
let rsdc = creg2reg_idx(rsd); | ||
|
||
X(rsdc) = zero_extend(X(rsdc)[31..0]); | ||
RETIRE_SUCCESS | ||
} | ||
|
||
mapping clause assembly = C_ZEXT_W(rsd) | ||
<-> "c.zext.w" ^ spc() ^ creg_name(rsd) | ||
|
||
/* ****************************************************************** */ | ||
|
||
/* ------------------------------------------------------------------ */ | ||
/* Sail Model for the c.mul encoding group */ | ||
/* ------------------------------------------------------------------ */ | ||
|
||
/* the assembly abstract syntax tree (AST) clause for the CMUL instructions */ | ||
union clause ast = C_MUL : (cregidx,cregidx) | ||
|
||
/* the encode/decode mapping between AST elements and 16-bit half words */ | ||
|
||
mapping clause encdec_compressed = C_MUL(rsd, rs2) | ||
<-> 0b100 @ 0b111 @ rsd : cregidx @ 0b00 @ rs2 : cregidx @ 0b01 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
|
||
function clause execute (C_MUL(rsd,rs2)) = { | ||
let rd = creg2reg_idx(rsd); | ||
let rs = creg2reg_idx(rs2); | ||
|
||
let result_wide = to_bits(2 * sizeof(xlen), signed(X(rd)) * signed(X(rs))); | ||
X(rd) = result_wide[(sizeof(xlen) - 1) .. 0]; | ||
RETIRE_SUCCESS | ||
} | ||
|
||
mapping clause assembly = C_MUL(rsd, rs2) <-> | ||
"c.mul" ^ spc() ^ creg_name(rsd) ^ spc() ^ creg_name(rs2) | ||
|
||
/* ------------------------------------------------------------------ */ | ||
/* Sail Model for the c.not encoding group */ | ||
/* ------------------------------------------------------------------ */ | ||
|
||
/* the assembly abstract syntax tree (AST) clause for the CNOT instructions */ | ||
|
||
union clause ast = C_NOT : (cregidx) | ||
|
||
/* the encode/decode mapping between AST elements and 16-bit half words */ | ||
|
||
mapping clause encdec_compressed = C_NOT(rsd) | ||
<-> 0b100 @ 0b111 @ rsd : cregidx @ 0b11 @ 0b101 @ 0b01 | ||
|
||
function clause execute (C_NOT(rsd)) = { | ||
let rsd = creg2reg_idx(rsd); | ||
X(rsd) = ~(X(rsd)); | ||
RETIRE_SUCCESS | ||
} | ||
|
||
mapping clause assembly = C_NOT(rsd) <-> | ||
"c.not" ^ spc() ^ creg_name(rsd) |
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.
Oh and this is removing vector, resolve conflicts accept all mine ?