Skip to content

Commit

Permalink
.github/zephyr: force -DCONFIG_OUTPUT_DISASSEMBLY=y
Browse files Browse the repository at this point in the history
Force -DCONFIG_OUTPUT_DISASSEMBLY=y when building Zephyr to make sure
the disassembly is compared between the Windows and Linux builds.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
  • Loading branch information
marc-hb committed Feb 1, 2024
1 parent 2752cdf commit 789b5da
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion .github/workflows/zephyr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,7 @@ jobs:
--cmake-args=-DEXTRA_CXXFLAGS=-Werror
--cmake-args=-DEXTRA_AFLAGS='-Werror -Wa,--fatal-warnings'
--cmake-args=--warn-uninitialized
--cmake-args=-DCONFIG_OUTPUT_DISASSEMBLY=y
${{ matrix.build_opts }} ${{ matrix.IPC_platforms }}

- name: Upload build artifacts
Expand Down Expand Up @@ -357,7 +358,9 @@ jobs:
--cmake-args=-DEXTRA_CFLAGS=-Werror
--cmake-args=-DEXTRA_CXXFLAGS=-Werror
--cmake-args=-DEXTRA_AFLAGS='-Werror -Wa,--fatal-warnings'
--cmake-args=--warn-uninitialized ${{ matrix.build_opts }} ${{ matrix.platforms }}
--cmake-args=--warn-uninitialized
--cmake-args=-DCONFIG_OUTPUT_DISASSEMBLY=y
${{ matrix.build_opts }} ${{ matrix.platforms }}

- name: Upload build artifacts
uses: actions/upload-artifact@v3
Expand Down Expand Up @@ -415,6 +418,10 @@ jobs:
{ >&2 printf 'Missing %s\n' "${regdir}"; exit 1; }
done
# The disassembly is one of the most important things in this
# context, so make sure CONFIG_OUTPUT_DISSASSEMBLY was on.
ls -l */build-sof-staging/sof-info/*/zephyr.lst*
set -x
for windir in windows-build*; do
lindir=linux-"${windir#windows-}"
Expand Down

0 comments on commit 789b5da

Please sign in to comment.