[PATCH] D37078: [DebugInfo] Handle memory locations in LiveDebugValues

[Adrian Prantl via Phabricator via llvm-commits]

aprantl added a comment.

First off: Thanks!
Does this have have any performance impact? If have the time, it would be nice to just compare the wall clock time of building an asanified (that should generate lots of memory locations) llc or clang with and without the patch.



================
Comment at: llvm/lib/CodeGen/LiveDebugValues.cpp:124
+    uint64_t RegNo = 0;
+    const DIExpression *ExprOffset = nullptr;
 
----------------
I wouldn't call it offset if it's the entire expression


https://reviews.llvm.org/D37078