Skip to content

Conversation

enm-intel
Copy link
Contributor

@enm-intel enm-intel commented Sep 2, 2025

Related-To: VLCLJ-2555

Several hundred type-related warnings appear when compiling the Level-Zero CTS public & embargo code. This leads to important warnings being overlooked because they are buried with the hundreds of type mismatch warnings.

This task will remove these warnings by applying the appropriate type conversions. See comments in the related Jira task for details of some of the fixes.

This PR only fixes those warnings related to mismatched type warnings during comparisons (i.e., "-Wsign-compare" and "-Wnarrowing"). A future PR will fix other warnings (e.g., "-Wunused-variable", "-Wparentheses", "-Wdangling-else", etc.) with the ultimate goal to remove all warnings ("-Wall -Wextra -Wpedantic").

Also, this PR is for the warnings in the public Level-Zero CTS repository. A similar PR will be submitted for the embargo CTS repository components once this PR has been merged.

KonstantyMisiak and others added 4 commits August 29, 2025 19:27
Signed-off-by: Misiak, Konstanty <[email protected]>
Signed-off-by: Eric Mortensen <[email protected]>
Related-To: VLCLJ-2555

Signed-off-by: Eric Mortensen <[email protected]>
Related-To: VLCLJ-2555

Signed-off-by: Eric Mortensen <[email protected]>
Related-To: VLCLJ-2555

Signed-off-by: Eric Mortensen <[email protected]>
marekkozl
marekkozl previously approved these changes Sep 3, 2025
Copy link
Contributor

@marekkozl marekkozl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, before merge, please squash commits and make proper commit message.
It will affect git history for many files.

Related-To: VLCLJ-2555

Signed-off-by: Eric Mortensen <[email protected]>
Related-To: VLCLJ-2555

Signed-off-by: Eric Mortensen <[email protected]>
Related-To: VLCLJ-2555

Signed-off-by: Eric Mortensen <[email protected]>
Related-To: VLCLJ-2555

Signed-off-by: Eric Mortensen <[email protected]>
Related-To: VLCLJ-2555

Signed-off-by: Eric Mortensen <[email protected]>
Related-To: VLCLJ-2555

Signed-off-by: Eric Mortensen <[email protected]>
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.

3 participants