[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