Browse Source

Make valgrind calls a no-op on non-supported platforms rather than print an error

gingerBill 3 years ago
parent
commit
4bdd2ff93c
1 changed files with 1 additions and 4 deletions
  1. 1 4
      src/check_builtin.cpp

+ 1 - 4
src/check_builtin.cpp

@@ -5398,10 +5398,7 @@ bool check_builtin_procedure(CheckerContext *c, Operand *operand, Ast *call, i32
 
 
 	case BuiltinProc_valgrind_client_request:
 	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};
 			enum {ARG_COUNT = 7};
 			GB_ASSERT(builtin_procs[BuiltinProc_valgrind_client_request].arg_count == ARG_COUNT);
 			GB_ASSERT(builtin_procs[BuiltinProc_valgrind_client_request].arg_count == ARG_COUNT);