Use true not 1

CTSRD-CHERI · Feb 28, 2022 · a8fb070 · a8fb070
1 parent 3bfaae1
commit a8fb070
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/target/cheri-common/cheri_tagmem.h b/target/cheri-common/cheri_tagmem.h
@@ -56,11 +56,10 @@
  * developers want a "fast but not precise" singlecore, we could have locks off.
  *
  * If we want to throw caution to wind and really optimise the single thread
- * case, it should be noted qemu_tcg_mttcg_enabled is dynamic and so will
- * still be generating an extra branch. TARGET_SUPPORTS_MTTCG is the
- * static thing. Although qemu_tcg_mttcg_enabled will default to false if
- * TARGET_SUPPORTS_MTTCG is undefined, it can be still be overridden on the
- * command line.
+ * case, it should be noted need_concurrent_tags is dynamic and so will
+ * still be generating a few extra branchs. TARGET_SUPPORTS_MTTCG is the
+ * static thing we could use if we want to compile an extra fast but single-core
+ * only binary.
  */
 
 #define UNSAFE_SINGLE_CORE true

diff --git a/target/riscv/op_helper_cheri.c b/target/riscv/op_helper_cheri.c
@@ -408,7 +408,7 @@ static target_ulong sc_c_impl(CPUArchState *env, uint32_t addr_reg,
     log_changed_special_reg(env, "load_res", env->load_res);
     bool store_fails;
     if (addr != expected_addr) {
-        store_fails = 1;
+        store_fails = true;
         goto sc_failed;
     }
     // Now perform the "cmpxchg" operation by checking if the current values