target/riscv: fix memory access result type checker function return i…

…n case of assertion Fix memory access result type checker return in case of assertion Signed-off-by: Farid Khaydari <[email protected]>
riscv-collab · Nov 26, 2024 · c8ae081 · c8ae081
1 parent f51900b
commit c8ae081
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/src/target/riscv/riscv-013.c b/src/target/riscv/riscv-013.c
@@ -3427,26 +3427,32 @@ bool is_mem_access_failed(mem_access_result_t status)
 {
 	#define MEM_ACCESS_RESULT_HANDLER(name, kind, msg) \
 		case name: return kind == MEM_ACCESS_RESULT_TYPE_FAILED;
+
 	switch (status) {
 		LIST_OF_MEM_ACCESS_RESULTS
 	}
+
 	#undef MEM_ACCESS_RESULT_HANDLER
+
 	LOG_ERROR("Unknown memory access status: %d", status);
-	assert(false);
-	return false;
+	assert(false && "Unknown memory access status");
+	return true;
 }
 
 bool is_mem_access_skipped(mem_access_result_t status)
 {
 	#define MEM_ACCESS_RESULT_HANDLER(name, kind, msg) \
 		case name: return kind == MEM_ACCESS_RESULT_TYPE_SKIPPED;
+
 	switch (status) {
 		LIST_OF_MEM_ACCESS_RESULTS
 	}
+
 	#undef MEM_ACCESS_RESULT_HANDLER
+
 	LOG_ERROR("Unknown memory access status: %d", status);
-	assert(false);
-	return false;
+	assert(false && "Unknown memory access status");
+	return true;
 }
 
 const char *mem_access_result_to_str(mem_access_result_t status)