Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

Only check type resolutions for cycles where every callable is type parameterized #1411

Merged
merged 1 commit into from
Apr 14, 2022

Conversation

bettinaheim
Copy link
Contributor

@bettinaheim bettinaheim commented Apr 14, 2022

It would be more efficient to just store whether a node in the call graph has type parameters along with the node.

@swernli swernli enabled auto-merge (squash) April 14, 2022 05:39
Copy link
Contributor

@ScottCarda-MS ScottCarda-MS left a comment

Choose a reason for hiding this comment

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

This seems like a good improvement for efficiency. Could we wait to merge this in until I can chat with you all tomorrow, and get properly caught up?

@swernli swernli disabled auto-merge April 14, 2022 07:04
@swernli
Copy link
Contributor

swernli commented Apr 14, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@swernli
Copy link
Contributor

swernli commented Apr 14, 2022

This seems like a good improvement for efficiency. Could we wait to merge this in until I can chat with you all tomorrow, and get properly caught up?

Sure thing! Disabled the auto merge so we can sync up tomorrow.

@ScottCarda-MS
Copy link
Contributor

@swernli and I had a chat, and I'm all caught up. This looks ready to merge in. Thanks!

@swernli swernli merged commit a268b34 into main Apr 14, 2022
swernli added a commit to microsoft/qsharp-runtime that referenced this pull request May 3, 2022
Now that the fix from microsoft/qsharp-compiler#1411 has made it into the release, the QDK version can be updated to the April release instead of a beta version.
swernli added a commit to microsoft/qsharp-runtime that referenced this pull request May 15, 2022
Now that the fix from microsoft/qsharp-compiler#1411 has made it into the release, the QDK version can be updated to the April release instead of a beta version.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants