On Tue, 21 Feb 2023 at 16:27, Aditya Kumar <appu...@google.com> wrote: > > From 2d7c1398d048f11e92945c88a17a907efe8db553 Mon Sep 17 00:00:00 2001 > From: AdityaK <appu...@google.com> > Date: Tue, 21 Feb 2023 15:05:54 -0800 > Subject: [PATCH] Add attribute used to macro > > The variable gets dropped by clang compiler in an optimized builds. > Adding attribute((used)) allows the symbol to be preserved. Similar > changes have been proposed in the past e.g., > 569524741a01e1a96fc2b75dd7e5d12e41ce6c2b for ll_entry_declare macro. > > --- > include/linker_lists.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass <s...@chromium.org> > > diff --git a/include/linker_lists.h b/include/linker_lists.h > index d3da9d44e8..4cd13c3bc8 100644 > --- a/include/linker_lists.h > +++ b/include/linker_lists.h > @@ -125,7 +125,7 @@ > #define ll_entry_start(_type, _list) \ > ({ \ > static char start[0] __aligned(CONFIG_LINKER_LIST_ALIGN) \ > - __attribute__((unused)) \ > + __attribute__((unused)) __attribute__((used)) \ > __section("__u_boot_list_2_"#_list"_1"); \ > (_type *)&start; \ > }) > -- > 2.39.2.637.g21b0678d19-goog