<br><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 29, 2018, 4:08 PM Wolfgang Pieb 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">wolfgangp 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>
dblaikie wrote:<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>
Yeah, it looks like it's always available from the context. If all goes well with this patch I'll try to do this as a follow-on if that's OK with you. <br></blockquote></div><div>Absolutely! Thanks!</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div>