Skip to content

Conversation

@KornevNikita
Copy link
Contributor

If assertions are disabled this warning is treated as an error: warning C4101: 'e': unreferenced local variable

If assertions are disabled this warning is treated as an error:
warning C4101: 'e': unreferenced local variable
@KornevNikita KornevNikita changed the title [SYCL] Mark variable as maybe_unused [SYCL] Void unused variable Apr 1, 2025
@KornevNikita
Copy link
Contributor Author

The linux build with --no-assertions passes without this fix, but windows needs it. Test run - https://github.com/intel/llvm/actions/runs/14193500815

@KornevNikita
Copy link
Contributor Author

failure: #17777
@intel/llvm-gatekeepers please merge

@sarnex sarnex merged commit 9303f5a into sycl Apr 1, 2025
33 of 36 checks passed
KornevNikita added a commit to KornevNikita/llvm that referenced this pull request Apr 1, 2025
If assertions are disabled this warning is treated as an error: warning
C4101: 'e': unreferenced local variable
@KornevNikita KornevNikita deleted the fix-no-asserts branch April 1, 2025 17:25
KornevNikita added a commit that referenced this pull request May 28, 2025
If assertions are disabled this warning is treated as an error: warning
C4101: 'e': unreferenced local variable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants