Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Performance issue with VRegAnalysis.modified and VRegAnalysis.depends_on #10

Closed
dmitryya opened this issue Feb 17, 2023 · 0 comments
Closed
Labels
bug Something isn't working

Comments

@dmitryya
Copy link
Contributor

Processing time for 'tests/other/sha512sum.bc' is terribly long(I didn't have the patience to wait until the end).
Some digging revealed that bottleneck are VRegAnalysis.modified and VRegAnalysis.depends_on for vuln detection rules null_dereference and format_string.
VRegAnalysis.modified is a known bottleneck because it is too generic, it needs to be narrowed.
VRegAnalysis.depends_on requires more investigation.

@dmitryya dmitryya added the bug Something isn't working label Feb 17, 2023
dmitryya added a commit that referenced this issue Apr 14, 2023
The usage of `VRegAnalysis.modified` and `VRegAnalysis.modified_at`
can result in poor performance due to generate excessive records,
particularly when processing large files.

Remove them and use DataFlowGraph instead.

Partially addressed the #10 issue.
dmitryya added a commit that referenced this issue Apr 14, 2023
The usage of `VRegAnalysis.modified` and `VRegAnalysis.modified_at`
can result in poor performance due to generate excessive records,
particularly when processing large files.

Remove them and use DataFlowGraph instead.

Partially addressed the #10 issue.
dmitryya added a commit that referenced this issue Apr 14, 2023
The usage of `VRegAnalysis.depends_on` can result in poor performance due to
generate excessive records, particularly when processing large files.

Remove them and use DataFlowGraph instead.

Fixes #10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant