Skip to content

Conversation

c8ef
Copy link
Contributor

@c8ef c8ef commented Aug 26, 2025

/root/llvm-project/openmp/tools/omptest/src/OmptAssertEvent.cpp:27:3: warning: default label in switch which covers all enumeration values [-Wcovered-switch-default]
   27 |   default:
      |   ^
1 warning generated.

@llvmbot llvmbot added openmp:libomp OpenMP host runtime openmp:libomptarget OpenMP offload runtime labels Aug 26, 2025
@c8ef c8ef requested review from jprotze, mhalk and mjklemm August 26, 2025 14:45
@jprotze
Copy link
Collaborator

jprotze commented Aug 26, 2025

I think the warning is stupid, I would rather keep the default case and disable the warning.

@c8ef
Copy link
Contributor Author

c8ef commented Aug 26, 2025

I think the warning is stupid, I would rather keep the default case and disable the warning.

Maybe we should use this:

libomp_append(flags_local -Wno-covered-switch-default LIBOMP_HAVE_WCOVERED_SWITCH_DEFAULT_FLAG)

@c8ef c8ef closed this Aug 26, 2025
@c8ef c8ef deleted the omp branch August 26, 2025 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

openmp:libomp OpenMP host runtime openmp:libomptarget OpenMP offload runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants