Skip to content

C++: Make sure getInstructionTagId has a result for this related IPA branches#4560

Merged
dbartol merged 1 commit intogithub:mainfrom
MathiasVP:instruction-tag-for-this-addr-and-load
Oct 28, 2020
Merged

C++: Make sure getInstructionTagId has a result for this related IPA branches#4560
dbartol merged 1 commit intogithub:mainfrom
MathiasVP:instruction-tag-for-this-addr-and-load

Conversation

@MathiasVP
Copy link
Contributor

Fixes https://github.com/github/codeql-c-analysis-team/issues/176. That issue turned out to be a red herring for my dataflow-debugging purpose, but at least the fix was simple.

@MathiasVP MathiasVP added the C++ label Oct 27, 2020
@MathiasVP MathiasVP requested a review from a team as a code owner October 27, 2020 21:19
Copy link

@dbartol dbartol left a comment

Choose a reason for hiding this comment

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

LGTM

@dbartol dbartol merged commit 27ad7bc into github:main Oct 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants