[PATCH] Rewrite calculateDbgValueHistory to make it (hopefully) more transparent.

David Blaikie dblaikie at gmail.com
Fri May 2 15:32:33 PDT 2014


Given that this is a substantial rewrite/change to the algorithm, would it be worthwhile to make it even more readable to begin with. What I have in mind (but perhaps it doesn't make sense - this is just an off-the-cuff idea) is trying to break down the new algorithm into some smaller functions with meaningful names that would make it easier to read through the code.

(since I'm going to have to read through the new code and understand it from scratch, since it doesn't map simply to the old algorithm, even if they produce the same result)

http://reviews.llvm.org/D3597






More information about the llvm-commits mailing list