Avoid visiting an instruction more than once by using a map.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
was there a quadratic behavior before ? It seems linear before and after the patch -- but just cut the computation by a factor of 2 . Do you need an option to control the max chain length?
Comment Actions
I think at least something more than linear. The downstream nodes got visited many, many times (eg. in the test, imagine how many paths there are from %v301 to %v298, to %v296, then all the way down to %v0.)
We could use a max length option in the future but for now it seems fine.