-
Notifications
You must be signed in to change notification settings - Fork 435
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
AndersenLCD implementation "incomplete": documentation and efficiency #6
Labels
Comments
Our default Andersen's analysis uses wave propagation solver "AndersenWaveDiff" which has been extensively tested using our micro-benchmarks and many standard benchmarks CPU2000/2006. AndersenLCD is very simple implementation of lazy cycle detection algorithm. The implementation may not be well tuned and tested to match the performance proposed in its paper. Feel free to improve it:) |
Closed
yuleisui
pushed a commit
that referenced
this issue
Jun 14, 2022
* Add SVFG read/write options Add << override for MRVer Co-authored-by: Wren_F <wrenfajardo123@gmail.com> * SVFG read and write functions Co-authored-by: Wren_F <wrenfajardo123@gmail.com> * Fix files to pass build * Fix files to pass build * Fix ctest error outOfBounds * outOfBounds Fix2 * Fix merge conflict * fix merge conflicts * fix merge conflict * fix merge conflicts * fix merge conflicts * Fix merge conflict * Revert "fix merge conflicts" This reverts commit 0628c40. * Fix build error * fix build error * Move read and write to SVFGReadWrite.cpp * Move read and write to SVFGReadWrite.cpp Co-authored-by: lukekoko <github@lukekoko.net> Co-authored-by: Wren_F <wrenfajardo123@gmail.com> Co-authored-by: root <root@Jeffrey-PC.localdomain>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The implementation of lazy cycle detection is described as 'incomplete' in Andersen.h,
is the implemented approach documented somewhere?
If not, could this be documented?
I suppose I'm unsure in what way what's implemented is "lazy cycle detection", is it?
The text was updated successfully, but these errors were encountered: