Commit d49d1666 authored by Lu Hongfei's avatar Lu Hongfei Committed by Josh Poimboeuf
Browse files

tools: Remove unnecessary variables 



There are several places where warnings variables are not needed,
remove them and directly return 0.

Signed-off-by: default avatarLu Hongfei <luhongfei@vivo.com>
Link: https://lore.kernel.org/r/20230530075649.21661-1-luhongfei@vivo.com


Signed-off-by: default avatarJosh Poimboeuf <jpoimboe@kernel.org>
parent ff9a6459

tools/objtool/check.c

+3 −4
Original line number Diff line number Diff line
@@ -3799,7 +3799,7 @@ static int validate_unwind_hints(struct objtool_file *file, struct section *sec) 
static int validate_unret(struct objtool_file *file, struct instruction *insn)

{

	struct instruction *next, *dest;

	int ret, warnings = 0;

	int ret;



	for (;;) {

		next = next_insn_to_validate(file, insn);
@@ -3897,7 +3897,7 @@ static int validate_unret(struct objtool_file *file, struct instruction *insn) 
		insn = next;

	}



	return warnings;

	return 0;

}



/*
@@ -4132,7 +4132,6 @@ static int add_prefix_symbols(struct objtool_file *file) 
{

	struct section *sec;

	struct symbol *func;

	int warnings = 0;



	for_each_sec(file, sec) {

		if (!(sec->sh.sh_flags & SHF_EXECINSTR))
@@ -4146,7 +4145,7 @@ static int add_prefix_symbols(struct objtool_file *file) 
		}

	}



	return warnings;

	return 0;

}



static int validate_symbol(struct objtool_file *file, struct section *sec,