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

review: fix(control-flow): Process constructor call in control flow graph #5738

Merged
merged 3 commits into from
Apr 20, 2024

Conversation

Mr-Pine
Copy link
Contributor

@Mr-Pine Mr-Pine commented Apr 5, 2024

Currently, a standalone constructor call doesn't appear in the control flow graph. With this PR it is processed like any other statement

Copy link
Collaborator

@MartinWitt MartinWitt left a comment

Choose a reason for hiding this comment

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

LGTM will merge

@MartinWitt
Copy link
Collaborator

@Mr-Pine there is a conflict.

@Mr-Pine Mr-Pine force-pushed the controlflow-constructor-call branch from 080f319 to de3aaf0 Compare April 10, 2024 20:29
@Mr-Pine Mr-Pine force-pushed the controlflow-constructor-call branch from de3aaf0 to 62f9f16 Compare April 10, 2024 20:31
@Mr-Pine
Copy link
Contributor Author

Mr-Pine commented Apr 10, 2024

Is resolved @MartinWitt

@I-Al-Istannen
Copy link
Collaborator

@MartinWitt ✨ ping ✨

@MartinWitt MartinWitt merged commit 2c30f68 into INRIA:master Apr 20, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants