[PATCH] DebugInfo: Introduce the notion of "form classes"
Alexey Samsonov
samsonov at google.com
Fri Oct 18 02:07:46 PDT 2013
Please take a look
================
Comment at: lib/DebugInfo/DWARFFormValue.cpp:483
@@ -482,3 +533,1 @@
-const uint8_t *DWARFFormValue::BlockData() const {
- if (!isInlinedCStr())
----------------
David Blaikie wrote:
> These 3 functions are just dead code? Please remove them separately if that's the case.
Done
================
Comment at: include/llvm/DebugInfo/DWARFFormValue.h:23
@@ -22,3 +22,3 @@
struct ValueType {
- ValueType() : data(NULL), IsDWOIndex(false) {
uval = 0;
----------------
David Blaikie wrote:
> Is this change separable? It looks like it could be. (if so, please commit or send for separate review)
Done
http://llvm-reviews.chandlerc.com/D1961
More information about the llvm-commits
mailing list