Skip to content

Conversation

@adityasharad
Copy link
Collaborator

This should be merged together with the corresponding internal PR. The tests on this PR can be ignored.

pavgust and others added 3 commits September 24, 2018 18:18
We currently erroneously keep mentions of class instantiations, which
can lead to bad performance on template-heavy code bases. We never
want to link those anyway, so we can simply suppress them.
C++ definitions: Ignore type mentions of class inatantiations.
@adityasharad adityasharad added depends on internal PR This PR should only be merged in sync with an internal Semmle PR Mergeback labels Sep 25, 2018
@adityasharad adityasharad requested review from jbj and pavgust September 25, 2018 13:34
@adityasharad adityasharad added the WIP This is a work-in-progress, do not merge yet! label Sep 25, 2018
@adityasharad adityasharad removed the WIP This is a work-in-progress, do not merge yet! label Sep 25, 2018
@nickrolfe nickrolfe merged commit 57f3ac8 into github:master Sep 26, 2018
aibaars pushed a commit that referenced this pull request Oct 14, 2021
smowton pushed a commit to smowton/codeql that referenced this pull request Feb 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

depends on internal PR This PR should only be merged in sync with an internal Semmle PR Mergeback

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants