tree 11b782637b25bc6f7b1a6871d47eb8eaefc44d58
parent 889bbc70b640a4ba4a663fb9c49c9d5af8038511
author Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com> 1560849437 +0200
committer Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com> 1560850139 +0200

Add --check option to scripts/baremetal.sh

Only effective together with --rom, makes two changes:
- abort in case of build warnings
- skip writing statistics

The goal is to make sure we build cleanly in the configuration used for
measuring code size, with all the compilers we use, both because we care about
that configuration and those compilers, and because any warnings would cast a
shadow on the code size measurements.

Currently the build fails with armc5 due to a pre-existing warning in PK, this
will be fixed in the next commit.

The next commit will also add an all.sh component to make sure we have no
regression in the future. (Which is the motivation for --check skipping
statistics: an all.sh component should probably not leave files around.)

While at it, fix two things:

1. The call to gcc --version was redundant with the echo line below
2. WARNING_CFLAGS shouldn't be overriden with armclang, as it would remove the
-Wall -Wextra and any directory-specific warning (such as
-Wdeclaration-after-statement in library). It's meant to be overriden only
with compilers that don't accept the default value (namely armc5 here).