Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

-Werror: deprecated enum -> enum conversion #5783

Open
Asti0s opened this issue Sep 4, 2024 · 1 comment
Open

-Werror: deprecated enum -> enum conversion #5783

Asti0s opened this issue Sep 4, 2024 · 1 comment
Assignees
Labels
component:diff Issues related to spirv-diff

Comments

@Asti0s
Copy link

Asti0s commented Sep 4, 2024

Hello, i'm making this issue to signal that spirv tools cant actually compile using werror due to a deprecated enum conversion:

SPIRV-Tools/tools/diff/diff.cpp: In function ‘std::unique_ptrspvtools::opt::IRContext {anonymous}::load_module(const char*)’:
SPIRV-Tools/tools/diff/diff.cpp:90:54: error: bitwise operation between different enumeration types ‘spvtools::SpirvTools::’ and ‘spv_text_to_binary_options_t’ is deprecated [-Werror=deprecated-enum-enum-conversion]
90 | spvtools::SpirvTools::kDefaultAssembleOption |
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
91 | SPV_TEXT_TO_BINARY_OPTION_PRESERVE_NUMERIC_IDS);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

@s-perron
Copy link
Collaborator

@ShabbyX The file mentioned is in spirv-diff. Can you take a look?

@s-perron s-perron added the component:diff Issues related to spirv-diff label Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:diff Issues related to spirv-diff
Projects
None yet
Development

No branches or pull requests

3 participants