Skip to content

Commit 46e68c3

Browse files
svenvhdwoodwor-intel
authored andcommitted
Exclude some files from code style checks
Some files cannot be parsed on their own as they for example require a macro to be defined. To avoid spurious failures of clang-tidy, exclude such files from the code style checks. Remove spirv.hpp from the list as it is no longer in the tree. Original commit: KhronosGroup/SPIRV-LLVM-Translator@0974907
1 parent 7488a44 commit 46e68c3

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

llvm-spirv/.github/workflows/check-code-style.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,12 @@ jobs:
4343
- name: Gather list of changes
4444
id: gather-list-of-changes
4545
run: |
46-
git diff -U0 --no-color ${{ github.sha }}^ -- include lib ':(exclude)lib/SPIRV/libSPIRV/spirv.hpp' > diff-to-inspect.txt
46+
git diff -U0 --no-color ${{ github.sha }}^ -- include lib \
47+
':(exclude)include/LLVMSPIRVExtensions.inc' \
48+
':(exclude)lib/SPIRV/libSPIRV/SPIRVErrorEnum.h' \
49+
':(exclude)lib/SPIRV/libSPIRV/SPIRVOpCodeEnum.h' \
50+
':(exclude)lib/SPIRV/libSPIRV/SPIRVOpCodeEnumInternal.h' \
51+
> diff-to-inspect.txt
4752
if [ -s diff-to-inspect.txt ]; then
4853
# Here we set an output of our step, which is used later to either
4954
# perform or skip further steps, i.e. there is no sense to install

0 commit comments

Comments
 (0)