Merge pull request #4144 from laytan/riscv-error-on-atomics-without-extension

riscv: add an error when atomics are used without the atomics extension

1 files changed, 8 insertions, 0 deletions

diff --git a/src/check_builtin.cpp b/src/check_builtin.cpp
index fa03da946..3742caeda 100644
--- a/src/check_builtin.cpp
+++ b/src/check_builtin.cpp
@@ -2048,6 +2048,14 @@ gb_internal bool check_builtin_procedure(CheckerContext *c, Operand *operand, As
 		return ok;
 	}
 
+	if (BuiltinProc__atomic_begin < id && id < BuiltinProc__atomic_end) {
+		if (build_context.metrics.arch == TargetArch_riscv64) {
+			if (!check_target_feature_is_enabled(str_lit("a"), nullptr)) {
+				error(call, "missing required target feature \"a\" for atomics, enable it by setting a different -microarch or explicitly adding it through -target-features");
+			}
+		}
+	}
+
 	switch (id) {
 	default:
 		GB_PANIC("Implement built-in procedure: %.*s", LIT(builtin_name));