<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 29, 2018 at 3:52 PM David Blaikie via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">dblaikie added inline comments.<br>
<br>
<br>
================<br>
Comment at: llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFListTable.h:165<br>
+  /// True if this list is located in a split-DWARF (dwo or dwp) file.<br>
+  bool isDWO;<br>
   /// This string is displayed as a heading before the list is dumped<br>
----------------<br>
echristo wrote:<br>
> dblaikie wrote:<br>
> > echristo wrote:<br>
> > > Trying to figure out a way around isDWO as a boolean argument and data member would be great.<br>
> > Avoiding it being a boolean, or avoiding having the same name for an argument and data member at the same time?<br>
> The former. I'd like to avoid the boolean existing if we can avoid it, and definitely as an argument to the constructor as well. <br>
Ah, OK. In the "boolean arguments are hard to read" (eg: replace them with an enum) or the "this is extra state/information that this part of the code might, hopefully, not need to know"?<br></blockquote><div><br></div><div>Yep. Those. :) </div></div></div>