KEMBAR78
Switch the default to enable mlir by schweitzpgi · Pull Request #2366 · NVIDIA/cuda-quantum · GitHub
Skip to content

Conversation

@schweitzpgi
Copy link
Collaborator

@schweitzpgi schweitzpgi commented Nov 7, 2024

Update the tests and the driver to make --enable-mlir the default. Add a new option, --library-mode, to allow the user to select the old default of library mode.

@schweitzpgi schweitzpgi marked this pull request as draft November 7, 2024 20:27
@schweitzpgi schweitzpgi force-pushed the ch-enable.mlir.default.3 branch from 048c0f8 to fec3cc6 Compare November 13, 2024 17:30
Update the tests and the driver to make --enable-mlir the default. Add
a new option, --library-mode, to allow the user to select the old
default of library mode.

Add required command line argument to tests.

Repair the function definition as well.

The option is always ignored. Hardcode library-mode into
the add_nvqpp_test function instead.

Fix validation script to respect the compiler mode selected.

Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>
@schweitzpgi schweitzpgi force-pushed the ch-enable.mlir.default.3 branch from fec3cc6 to bdad400 Compare November 15, 2024 21:52
@schweitzpgi schweitzpgi marked this pull request as ready for review November 15, 2024 21:53
@schweitzpgi schweitzpgi added this to the release 0.9.1 milestone Nov 15, 2024
Copy link
Collaborator

@annagrin annagrin left a comment

Choose a reason for hiding this comment

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

LGTM!

@schweitzpgi schweitzpgi enabled auto-merge (squash) November 19, 2024 19:43
@schweitzpgi schweitzpgi disabled auto-merge November 19, 2024 19:43
@schweitzpgi schweitzpgi enabled auto-merge (squash) November 19, 2024 19:44
@schweitzpgi schweitzpgi merged commit f04463f into NVIDIA:main Nov 19, 2024
212 of 213 checks passed
@bettinaheim bettinaheim changed the title Switch the default to enable mlir. Switch the default to enable mlir Dec 10, 2024
@bettinaheim bettinaheim added breaking change Change breaks backwards compatibility release notes Changes need to be captured in the release notes labels Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change Change breaks backwards compatibility release notes Changes need to be captured in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants