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

[RTGTest] Add RTGTest dialect #7841

Merged
merged 1 commit into from
Nov 20, 2024
Merged

[RTGTest] Add RTGTest dialect #7841

merged 1 commit into from
Nov 20, 2024

Conversation

maerhart
Copy link
Member

Add a dialect to test and demonstrate the RTG dialect, its passes and interfaces. This also adds a CMake flag called CIRCT_INCLUDE_TESTS analogous to MLIRs MLIR_INCLUDE_TESTS to not build and include this dialect in release builds.

@maerhart maerhart force-pushed the maerhart-rtgtest-dialect branch 2 times, most recently from 831f56d to 03ca06a Compare November 19, 2024 15:57
@maerhart maerhart added the RTG Involving the `rtg` dialect label Nov 19, 2024
Copy link
Contributor

@fabianschuiki fabianschuiki left a comment

Choose a reason for hiding this comment

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

LGTM!

Comment on lines +624 to +631
if (CIRCT_INCLUDE_TESTS)
add_definitions(-DCIRCT_INCLUDE_TESTS)
if (CIRCT_GTEST_AVAILABLE)
add_subdirectory(unittests)
endif()
add_subdirectory(test)
add_subdirectory(integration_test)
endif()
Copy link
Contributor

Choose a reason for hiding this comment

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

This is very neat 👍

@maerhart maerhart force-pushed the maerhart-rtg-context-interfaces branch from d67b2fd to f7acd43 Compare November 20, 2024 09:05
Base automatically changed from maerhart-rtg-context-interfaces to main November 20, 2024 09:25
Add a dialect to test and demonstrate the RTG dialect, its passes and interfaces.
This also adds a CMake flag called `CIRCT_INCLUDE_TESTS` analogous to MLIRs `MLIR_INCLUDE_TESTS` to not build and include this dialect in release builds.

Co-authored-by: Andrew Lenharth <andrew@lenharth.org>
@maerhart maerhart merged commit 65fd75a into main Nov 20, 2024
2 checks passed
@maerhart maerhart deleted the maerhart-rtgtest-dialect branch November 20, 2024 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RTG Involving the `rtg` dialect
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants