Skip to content

Commit eccb557

Browse files
committed
Fix require_vector_vs checking by reverting 66c4853
The require_vector_vs, i.e., sstatus_csr_t::enabled(SSTATUS_VS), was expected to provide has_any_vector() check [1]. Unfortunately, a previous commit [2] made the sstatus_csr_t::enabled(SSTATUS_VS) true without any vector extension. The previous commit [2] was for P-extension, which has been removed from Spike [3]. This commit reverts the commit [2] and corrects require_vector_vs [1]. [1] a484f6e [2] 66c4853 [3] #1660
1 parent f7d0dba commit eccb557

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

riscv/csrs.cc

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -642,11 +642,6 @@ bool sstatus_csr_t::enabled(const reg_t which) {
642642
if (!state->v || (virt_sstatus->read() & which) != 0)
643643
return true;
644644
}
645-
646-
// If the field doesn't exist, it is always enabled. See #823.
647-
if (!orig_sstatus->field_exists(which))
648-
return true;
649-
650645
return false;
651646
}
652647

0 commit comments

Comments
 (0)