From 4bdd2ff93cdede889d0db9dc0885dd852402ba8f Mon Sep 17 00:00:00 2001
From: gingerBill <bill@gingerbill.org>
Date: Tue, 6 Sep 2022 12:57:22 +0100
Subject: [PATCH] Make valgrind calls a no-op on non-supported platforms rather
 than print an error

---
 src/check_builtin.cpp | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/check_builtin.cpp b/src/check_builtin.cpp
index 6af1c3d44..07eeeacca 100644
--- a/src/check_builtin.cpp
+++ b/src/check_builtin.cpp
@@ -5398,10 +5398,7 @@ bool check_builtin_procedure(CheckerContext *c, Operand *operand, Ast *call, i32
 
 	case BuiltinProc_valgrind_client_request:
 		{
-			if (!is_arch_x86()) {
-				error(call, "'%.*s' is only allowed on x86 targets (i386, amd64)", LIT(builtin_name));
-				return false;
-			}
+			// NOTE(bill): Check it but make it a no-op for non x86 (i386, amd64) targets
 
 			enum {ARG_COUNT = 7};
 			GB_ASSERT(builtin_procs[BuiltinProc_valgrind_client_request].arg_count == ARG_COUNT);