<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Aug 29, 2017 at 11:09 AM Teresa Johnson 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">tejohnson added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D37266#855617" rel="noreferrer" target="_blank">https://reviews.llvm.org/D37266#855617</a>, @dblaikie wrote:<br>
<br>
> Is it worth having the DEBUG output if no aliases are ever imported?<br>
<br>
<br>
I think it is worthwhile emitting this since we also emit the functions/vars that are not imported.<br>
<br>
> Could potentially rewrite this whole loop as:<br>
><br>
>   assert(none_of(SrcModule.aliases(), [&](GlobalAlias &GA) { if (!GA.hasName()) return false; return ImportGUIDs.count(GA.getGUID()); }) && "Unexpected alias in import list");<br>
><br>
><br>
> Dunno if that comes out more legible, though.<br>
<br>
I was trying to keep it somewhat symmetric with the earlier code handling functions and variables. I'm hoping to start looking at importing these as a copy, which would mean this would go back to a loop anyway. I will change it if you feel strongly that this is better though.<br></blockquote><div><br>Nah, whatever suits you<br> </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/D37266" rel="noreferrer" target="_blank">https://reviews.llvm.org/D37266</a><br>
<br>
<br>
<br>
</blockquote></div></div>