Skip to content

C++: Add semmle.code.cpp.dataflow.new#12163

Merged
jketema merged 3 commits into
github:mathiasvp/replace-ast-with-ir-use-usedataflowfrom
jketema:dataflow-new
Feb 13, 2023
Merged

C++: Add semmle.code.cpp.dataflow.new#12163
jketema merged 3 commits into
github:mathiasvp/replace-ast-with-ir-use-usedataflowfrom
jketema:dataflow-new

Conversation

@jketema
Copy link
Copy Markdown
Contributor

@jketema jketema commented Feb 13, 2023

No description provided.

@jketema jketema requested a review from a team as a code owner February 13, 2023 08:40
@github-actions github-actions Bot added the C++ label Feb 13, 2023
@jketema jketema added the no-change-note-required This PR does not need a change note label Feb 13, 2023
Copy link
Copy Markdown
Contributor

@MathiasVP MathiasVP left a comment

Choose a reason for hiding this comment

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

One comment, but otherwise LGTM!

Comment thread cpp/ql/lib/semmle/code/cpp/dataflow/new/DataFlow.qll Outdated
Copy link
Copy Markdown
Contributor

@MathiasVP MathiasVP left a comment

Choose a reason for hiding this comment

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

LGTM!

@jketema jketema merged commit 73d877e into github:mathiasvp/replace-ast-with-ir-use-usedataflow Feb 13, 2023
@jketema jketema deleted the dataflow-new branch February 13, 2023 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C++ no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants