Closed JonasAlaif closed 5 months ago
Responding to the PR: this looks great! I will pull this into the main crate
ah, whoops, my bad!! will enable that in the future, can't believe I missed that lol!
@Gopiandcode I've made some changes, let me know if everything still works for you or if I broke something. The main change is that I keep around the InstIdx
for much longer instead of using RawNodeIndex
(the missing link is that one can go from the former to the latter using the InstIdx as IndexesInstGraph
trait). I also switched to petgraph::Dfs
which requires some not very nice EdgeFiltered
and Reversed
adaptors but still nicer than doing the dfs manually imo
Ooh, yep, this looks good! Lemme try it out!
Awesome, tried it out! Works great!
Duplicate of https://github.com/viperproject/axiom-profiler-2/pull/45, I just don't have permissions to push to your fork. @Gopiandcode could you enable this in the future :)