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

Parser type conversions #750

Merged
merged 24 commits into from
Nov 25, 2024
Merged

Parser type conversions #750

merged 24 commits into from
Nov 25, 2024

Conversation

xlauko
Copy link
Member

@xlauko xlauko commented Nov 22, 2024

No description provided.

@xlauko xlauko added the parser label Nov 22, 2024
@xlauko xlauko requested a review from frabert November 22, 2024 11:37
@xlauko xlauko self-assigned this Nov 22, 2024
@xlauko xlauko force-pushed the xlauko/parser-type-conversion branch from 52f8dd7 to 94da22a Compare November 22, 2024 15:05
@xlauko xlauko marked this pull request as ready for review November 22, 2024 16:19
@xlauko xlauko force-pushed the xlauko/parser-type-conversion branch from 49ee89e to 6f04dc7 Compare November 25, 2024 08:53
Copy link
Contributor

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-format (v19.1.1) reports: 3 file(s) not formatted
  • include/vast/Conversion/TypeConverters/TypeConverter.hpp
  • include/vast/Dialect/Core/Interfaces/OperationInterfaces.hpp
  • lib/vast/Conversion/Parser/ToParser.cpp
clang-tidy (v19.1.1) reports: 1 concern(s)
  • include/vast/Dialect/Core/Interfaces/OperationInterfaces.hpp:16:10: error: [clang-diagnostic-error]

    'vast/Dialect/Core/Interfaces/OperationInterfaces.h.inc' file not found

       16 | #include "vast/Dialect/Core/Interfaces/OperationInterfaces.h.inc"
          |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Have any feedback or feature suggestions? Share it here.

@xlauko xlauko merged commit fd73384 into master Nov 25, 2024
8 checks passed
@xlauko xlauko deleted the xlauko/parser-type-conversion branch November 25, 2024 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants