Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Split callsite and return site in a CFBBNode #1227

Merged
merged 4 commits into from
Oct 17, 2023

Conversation

jumormt
Copy link
Contributor

@jumormt jumormt commented Oct 16, 2023

  • Split callsite and return site in a CFBBNode
  • WTO is updated based on the new structure of CFBBG

@codecov
Copy link

codecov bot commented Oct 16, 2023

Codecov Report

Merging #1227 (8fc5113) into master (73cd22e) will not change coverage.
Report is 3 commits behind head on master.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1227   +/-   ##
=======================================
  Coverage   64.50%   64.50%           
=======================================
  Files         223      223           
  Lines       23667    23667           
=======================================
  Hits        15266    15266           
  Misses       8401     8401           
Files Coverage Δ
svf/lib/SVFIR/SVFValue.cpp 74.00% <0.00%> (ø)

@@ -493,6 +498,9 @@ class CFBasicBlockGraph : public GenericCFBasicBlockGTy
class CFBasicBlockGBuilder
{

public:
typedef Map<const SVFBasicBlock*, std::vector<CFBasicBlockNode*>> SVFBBToCFBBNodes;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Since the input of CFBasicBlock is ICFG, we need to remove "SVFBasicBlock" from CFBasicBlockG. Clean up code and remove unnecessary data structures of SVF.

@yuleisui yuleisui merged commit c3bcbc7 into SVF-tools:master Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants