<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Jan 31, 2018 at 2:08 PM Adrian Prantl 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">aprantl added inline comments.<br>
<br>
<br>
================<br>
Comment at: include/llvm/IR/DebugInfoMetadata.h:512<br>
   ChecksumKind CSKind;<br>
+  Optional<MDString *> Source;<br>
<br>
----------------<br>
It is probably better to make DIFile variable length so we don't have to pay for the extra pointer when we don't need it.<br></blockquote><div><br>Yeah, likely - any idea what the overhead is to variable length? (is there any overhead? I guess maybe not)<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D42765" rel="noreferrer" target="_blank">https://reviews.llvm.org/D42765</a><br>
<br>
<br>
<br>
</blockquote></div></div>