[PATCH v2 14/30] test: Avoid strange symbols in the assembler file

Bin Meng bmeng.cn at gmail.com
Mon May 1 10:59:42 CEST 2023


On Sun, Apr 30, 2023 at 9:30 AM Simon Glass <sjg at chromium.org> wrote:
>
> This works correctly on Linux with ELF but not on Windows with PE, since
> it creates assembly symbols with invalid names.
>
> Use the lowest/highest valid characters instead. This should still work
> correctly, since we have no tests starting with $ and none starting with
> 'zz' at present.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> Changes in v2:
> - Update comment and use zz to make it less likely we have a problem
>
>  include/test/test.h | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>

Reviewed-by: Bin Meng <bmeng at tinylab.org>
Tested-by: Bin Meng <bmeng at tinylab.org>


More information about the U-Boot mailing list