Hi Anthony, > On 14 Jul 2022, at 15:39, Anthony PERARD <anthony.per...@citrix.com> wrote: > > check-endbr.sh works well with gawk, but fails with mawk. The produced > $ALL file is smaller, it is missing 0x$vma_lo on every line. On mawk, > int(0x2A) just produce 0, instead of the expected value. > > The use of hexadecimal-constant in awk is an optional part of the > posix spec, and mawk doesn't seems to implemented. > > There is a way to convert an hexadecimal to a number be putting it in > a string, and awk as I understand is supposed to use strtod() to > convert the string to a number when needed. The expression > 'int("0x15") + 21' would produce the expected value in `mawk` but now > `gawk` won't convert the string to a number unless we use the option > "--non-decimal-data". > > So let's convert the hexadecimal number before using it in the awk > script. The shell as no issue with dealing with hexadecimal-constant > so we'll simply use the expression "$(( 0x15 ))" to convert the value > before using it in awk. > > Fixes: 4d037425dc ("x86: Build check for embedded endbr64 instructions") > Reported-by: Luca Fancellu <luca.fance...@arm.com> > Reported-by: Mathieu Tarral <mathieu.tar...@protonmail.com> > Signed-off-by: Anthony PERARD <anthony.per...@citrix.com>
Very nice solution Reviewed-by: Bertrand Marquis <bertrand.marq...@arm.com> Cheers Bertrand