mirror of
https://github.com/llvm/llvm-project.git
synced 2025-04-25 04:16:07 +00:00
[workflows] Fix argument passing in abi-dump jobs (#79658)
This was broken by 859e6aa1008b80d9b10657bac37822a32ee14a23, which added quotes around the EXTRA_ARGS variable.
This commit is contained in:
parent
eb56bc2b10
commit
5e3fc9c37f
4
.github/workflows/llvm-tests.yml
vendored
4
.github/workflows/llvm-tests.yml
vendored
@ -125,7 +125,7 @@ jobs:
|
||||
else
|
||||
touch llvm.symbols
|
||||
fi
|
||||
abi-dumper "$EXTRA_ARGS" -lver ${{ matrix.ref }} -skip-cxx -public-headers ./install/include/${{ needs.abi-dump-setup.outputs.ABI_HEADERS }} -o ${{ matrix.ref }}.abi ./install/lib/libLLVM.so
|
||||
abi-dumper $EXTRA_ARGS -lver ${{ matrix.ref }} -skip-cxx -public-headers ./install/include/${{ needs.abi-dump-setup.outputs.ABI_HEADERS }} -o ${{ matrix.ref }}.abi ./install/lib/libLLVM.so
|
||||
# Remove symbol versioning from dumps, so we can compare across major versions.
|
||||
sed -i 's/LLVM_${{ matrix.llvm_version_major }}/LLVM_NOVERSION/' ${{ matrix.ref }}.abi
|
||||
- name: Upload ABI file
|
||||
@ -175,7 +175,7 @@ jobs:
|
||||
# FIXME: Reading of gzip'd abi files on the GitHub runners stop
|
||||
# working some time in March of 2021, likely due to a change in the
|
||||
# runner's environment.
|
||||
abi-compliance-checker "$EXTRA_ARGS" -l libLLVM.so -old build-baseline/*.abi -new build-latest/*.abi || test "${{ needs.abi-dump-setup.outputs.ABI_HEADERS }}" = "llvm-c"
|
||||
abi-compliance-checker $EXTRA_ARGS -l libLLVM.so -old build-baseline/*.abi -new build-latest/*.abi || test "${{ needs.abi-dump-setup.outputs.ABI_HEADERS }}" = "llvm-c"
|
||||
- name: Upload ABI Comparison
|
||||
if: always()
|
||||
uses: actions/upload-artifact@v3
|
||||
|
Loading…
x
Reference in New Issue
Block a user