smallsolar/u-boot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

sandbox: use sections instead of symbols for getopt array boundaries

Browse Source 
In style of linked lists, instead of declaring symbols for boundaries
of getopt options array in the linker script, declare corresponding
sections and retrieve the boundaries via static inline functions.

Without this clang's LTO produces binary without any getopt options,
because for some reason it thinks that array is empty (start and end
symbols are at the same address).

Signed-off-by: Marek Behún <marek.behun@nic.cz>
Reviewed-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Marek Behún
2021-05-20 13:24:06 +02:00
committed by Tom Rini
parent be1e77f286
commit d1f81fd015
5 changed files with 32 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
arch/sandbox/include/asm/sections.h
View File

@@ -13,12 +13,27 @@
struct sandbox_cmdline_option;
extern struct sandbox_cmdline_option *__u_boot_sandbox_option_start[],
*__u_boot_sandbox_option_end[];
static inline struct sandbox_cmdline_option **
__u_boot_sandbox_option_start(void)
{
static char start[0] __aligned(4) __attribute__((unused))
__section(".u_boot_sandbox_getopt_start");
return (struct sandbox_cmdline_option **)&start;
}
static inline struct sandbox_cmdline_option **
__u_boot_sandbox_option_end(void)
{
static char end[0] __aligned(4) __attribute__((unused))
__section(".u_boot_sandbox_getopt_end");
return (struct sandbox_cmdline_option **)&end;
}
static inline size_t __u_boot_sandbox_option_count(void)
{
return __u_boot_sandbox_option_end - __u_boot_sandbox_option_start;
return __u_boot_sandbox_option_end() - __u_boot_sandbox_option_start();
}
#endif