[PATCH] D27170: Abstract almost all DwarfDebug out of the classes in DIE.cpp.

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 1 10:39:34 PST 2016


aprantl accepted this revision.
aprantl added a comment.
This revision is now accepted and ready to land.

Looks good to me now. Thanks!



================
Comment at: include/llvm/CodeGen/DIE.h:598
 
-  /// Offset - Offset in debug info section.
+  /// Offset - Dwarf unit relative offset.
   ///
----------------
Not super important, but we would generally spell this like this nowadays:

```
  /// Dwarf unit relative offset.
  unsigned Offset;
```


https://reviews.llvm.org/D27170





More information about the llvm-commits mailing list