Skip to content

Commit

Permalink
240501.010755.HKT revise comments in cmake.yml
Browse files Browse the repository at this point in the history
  • Loading branch information
zaikunzhang committed Apr 30, 2024
1 parent 3fd5f24 commit c8c671b
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions .github/workflows/cmake.yml
Original file line number Diff line number Diff line change
Expand Up @@ -67,16 +67,17 @@ jobs:
# 2. On windows-latest, we want to use '-Wall -W5 -Werror-all' as what we do for the intel-classic
# compiler, but the (new) intel c compiler does not recognize '-W5 -Werror-all', even though the
# official documentation of the compiler mentions them. Why?
# Comments on `cl`:
# -wd4820 disables the warning about "... bytes padding added after data member"
# -wd4464 disables the warning about "elative include path contains '..'"
# -wd4710 disables the warning about "... function is not inlined"
- os: windows-latest
toolchain: {compiler: intel, version: '2023.2', cflags: '-Wall -Werror', fflags: '/warn:all /debug:extended /Z7 /fimplicit-none /standard-semantics /assume:recursion'}
- os: windows-latest
toolchain: {compiler: intel, version: '2024.0', cflags: '-Wall -Werror', fflags: '/warn:all /debug:extended /Z7 /fimplicit-none /standard-semantics /assume:recursion'}
- os: windows-latest
toolchain: {compiler: intel, version: '2024.1', cflags: '-Wall -Werror', fflags: '/warn:all /debug:extended /Z7 /fimplicit-none /standard-semantics /assume:recursion'}
- os: windows-latest
# -wd4820 disables the warning about "... bytes padding added after data member"
# -wd4464 disables the warning about "elative include path contains '..'"
# -wd4710 disables the warning about "... function is not inlined"
toolchain: {compiler: intel, version: '2024.1', cflags: '-wd4820 -wd4464 -wd4710 -Wall -W4 -WX', fflags: '/warn:all /debug:extended /Z7 /fimplicit-none /standard-semantics /assume:recursion', cc: cl}
# N.B.: As of 20240401, setup-fortran fails constantly with windows-latest and intel-classic
# 2021.8. Thus this combination is not included.
Expand Down

0 comments on commit c8c671b

Please sign in to comment.