On Tue, Aug 31, 2021 at 12:40 AM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote: > > The OBJECT_FILES_NON_STANDARD check is quite long. > > Factor it out into a new macro, objtool-enabled, to not repeat it. > > Signed-off-by: Masahiro Yamada <masahiroy@xxxxxxxxxx> Reviewed-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx> > --- > > scripts/Makefile.build | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/scripts/Makefile.build b/scripts/Makefile.build > index 021ae0146913..720a86642f48 100644 > --- a/scripts/Makefile.build > +++ b/scripts/Makefile.build > @@ -241,12 +241,12 @@ ifndef CONFIG_LTO_CLANG > # 'OBJECT_FILES_NON_STANDARD := y': skip objtool checking for a directory > # 'OBJECT_FILES_NON_STANDARD_foo.o := 'y': skip objtool checking for a file > # 'OBJECT_FILES_NON_STANDARD_foo.o := 'n': override directory skip for a file > -cmd_objtool = $(if $(patsubst y%,, \ > - $(OBJECT_FILES_NON_STANDARD_$(basetarget).o)$(OBJECT_FILES_NON_STANDARD)n), \ > - ; $(objtool) $(objtool_args) $@) > -objtool_obj = $(if $(patsubst y%,, \ > - $(OBJECT_FILES_NON_STANDARD_$(basetarget).o)$(OBJECT_FILES_NON_STANDARD)n), \ > - $(objtool)) > + > +objtool-enabled = $(if $(filter-out y%, \ > + $(OBJECT_FILES_NON_STANDARD_$(basetarget).o)$(OBJECT_FILES_NON_STANDARD)n),y) > + > +cmd_objtool = $(if $(objtool-enabled), ; $(objtool) $(objtool_args) $@) > +objtool_obj = $(if $(objtool-enabled), $(objtool)) > > endif # CONFIG_LTO_CLANG > endif # CONFIG_STACK_VALIDATION > -- > 2.30.2 > -- Thanks, ~Nick Desaulniers
