[PATCH] DebugInfo: Introduce the notion of "form classes"

Eric Christopher echristo at gmail.com
Tue Oct 22 12:55:09 PDT 2013



================
Comment at: include/llvm/DebugInfo/DWARFFormValue.h:65
@@ -47,6 +64,3 @@
 
-  uint64_t getReference(const DWARFUnit *U) const;
-  uint64_t getUnsigned() const { return Value.uval; }
-  int64_t getSigned() const { return Value.sval; }
-  const char *getAsCString(const DWARFUnit *U) const;
-  uint64_t getAsAddress(const DWARFUnit *U) const;
+  bool getAsReference(const DWARFUnit *U, uint64_t &Result) const;
+  bool getAsUnsignedConstant(uint64_t &Result) const;
----------------
Should be able to use Optional here?


http://llvm-reviews.chandlerc.com/D1961



More information about the llvm-commits mailing list