<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 4, 2016 at 8:55 AM Amjad Aboud <<a href="mailto:amjad.aboud@intel.com">amjad.aboud@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aaboud added a comment.<br>
<br>
> Side note: Is there any reason we're creating DIMacroFiles that have no macros in them? Or are they unnecessary?<br>
<br>
<br>
<br>
<br>
1. GCC does that too.<br>
2. It is implementation details, because we create the DIMacroFile before we parse its content, thus at that point we do not know if it will contain macros or not.<br></blockquote><div><br></div><div>Presumably we know when we've finished the DIMacroFile (so we stop putting macros in that file), so perhaps we could defer adding the DIMacroFile to its parent until that point - and if we find that there's nothing in it at that point, just not to add it at all.<br><br>Anyway *shrug* just an optimization for some time, if you're interested or you find this has memory problems, etc.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="https://reviews.llvm.org/D23056" rel="noreferrer" target="_blank">https://reviews.llvm.org/D23056</a><br>
<br>
<br>
<br>
</blockquote></div></div>