Skip to content

C++: Add expressions with type data to cpp/extraction-information#21544

Draft
paldepind wants to merge 1 commit intogithub:mainfrom
paldepind:cpp/extraction-information-expr-types
Draft

C++: Add expressions with type data to cpp/extraction-information#21544
paldepind wants to merge 1 commit intogithub:mainfrom
paldepind:cpp/extraction-information-expr-types

Conversation

@paldepind
Copy link
Contributor

@paldepind paldepind commented Mar 23, 2026

Follow up from #21512. Extends the query to also include data for expressions in the source with types. Like before the QL is more or less copy pasted from the similar queries for Rust/Java/C#.

Here's a comparison table for nmap similar to the one in the previous PR. The first row is the existing data, and the three below are from cpp/telemetry/extraction-information.

traced BMN BMN + deps inst
Expressions with a known type (existing data) 433,251 729,293 850,145
Percentage of expressions with known type 99.999 98.985 98.976
Number of expressions with known type 285,612 585,821 500,554
Number of expressions with unknown type 2 6,009 5,178

Again, the new numbers show more clearly the actual changes from BMN and dependency installation. Though at least for this project the call target data seemed more revealing. Two observations:

  • The increase in the number of expressions with a known type is relatively much greater than the increase in the number of calls with a target.
  • The percentage of expressions with a known type stays very high in both BMN cases. This seems to suggest that the reduction in expressions with a known type for dependency installation is because expressions are completely lost (and not just extracted without a type).

Perhaps others have some additional insight, but to me it seems that the resolved calls metric is more significant.

@github-actions github-actions bot added the C++ label Mar 23, 2026
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.

1 participant