[PATCH] D45324: [debug_loc] Fix typo in DWARFExpression constructor

Pavel Labath via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 6 01:54:02 PDT 2018


labath added inline comments.


================
Comment at: lib/DebugInfo/DWARF/DWARFContext.cpp:689
   LocDWO.reset(new DWARFDebugLocDWO());
-  LocDWO->parse(LocData);
+  // assume all compile units have the same address byte size
+  if (getNumCompileUnits()) {
----------------
aprantl wrote:
> LLVM always prefers full sentences:
> ```// Assume all compile units have the same address byte size.```
> 
I thought you might say that. :)
I will fix also the place where I copied this from.


Repository:
  rL LLVM

https://reviews.llvm.org/D45324





More information about the llvm-commits mailing list