<div class="gmail_extra"><div class="gmail_quote">On Fri, Sep 14, 2012 at 1:54 PM, Bill Wendling <span dir="ltr"><<a href="mailto:wendling@apple.com" target="_blank" class="cremed">wendling@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I saw that too, but it was like that in the std::for_each's before it, so I didn't bother to pester Jan. :-)<br>
</blockquote><div><br></div><div>This is what I get for only looking at the touched lines. =]</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
-bw<br>
</font></span><div class="im HOEnZb"><br>
On Sep 14, 2012, at 1:51 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com" class="cremed">chandlerc@google.com</a>> wrote:<br>
<br>
> Glorious and wonderful with one nit:<br>
><br>
> +  if (!WithoutAliases)<br>
> +    std::for_each (M->alias_begin(), M->alias_end(),<br>
> +                DumpSymbolNameForGlobalValue);<br>
><br>
> No space between 'std::for_each' and the '('. Functions and if statements are different for LLVM's style...<br>
><br>
><br>
> On Fri, Sep 14, 2012 at 7:46 AM, Jan Sjodin <<a href="mailto:jan_sjodin@yahoo.com" class="cremed">jan_sjodin@yahoo.com</a>> wrote:<br>
> New patch with fixed indentation and test case (includes both with and without the new flag).<br>
><br>
> - Jan<br>
><br>
</div><div class="HOEnZb"><div class="h5">> From: Chandler Carruth <<a href="mailto:chandlerc@google.com" class="cremed">chandlerc@google.com</a>><br>
> To: "Villmow, Micah" <<a href="mailto:Micah.Villmow@amd.com" class="cremed">Micah.Villmow@amd.com</a>><br>
> Cc: Jan Sjodin <<a href="mailto:jan_sjodin@yahoo.com" class="cremed">jan_sjodin@yahoo.com</a>>; "<a href="mailto:llvm-commits@cs.uiuc.edu" class="cremed">llvm-commits@cs.uiuc.edu</a>" <<a href="mailto:llvm-commits@cs.uiuc.edu" class="cremed">llvm-commits@cs.uiuc.edu</a>><br>

> Sent: Thursday, September 13, 2012 5:34 PM<br>
><br>
> Subject: Re: [llvm-commits] Patch to add hidden flag to llvm-nm to not emit aliases.<br>
><br>
> On Thu, Sep 13, 2012 at 2:21 PM, Villmow, Micah <<a href="mailto:Micah.Villmow@amd.com" class="cremed">Micah.Villmow@amd.com</a>> wrote:<br>
> As far as I can tell, most basic tools have no test cases, are there examples?<br>
><br>
> test/Other/optimize-options.ll<br>
><br>
><br>
> From: Chandler Carruth [mailto:<a href="mailto:chandlerc@google.com" class="cremed">chandlerc@google.com</a>]<br>
> Sent: Thursday, September 13, 2012 2:17 PM<br>
> To: Villmow, Micah<br>
> Cc: Jan Sjodin; <a href="mailto:llvm-commits@cs.uiuc.edu" class="cremed">llvm-commits@cs.uiuc.edu</a><br>
> Subject: Re: [llvm-commits] Patch to add hidden flag to llvm-nm to not emit aliases.<br>
><br>
> On Thu, Sep 13, 2012 at 2:10 PM, Villmow, Micah <<a href="mailto:Micah.Villmow@amd.com" class="cremed">Micah.Villmow@amd.com</a>> wrote:<br>
> LGTM<br>
><br>
> No, this patch isn't ready to go in yet.<br>
><br>
> It has no test case updates, and it doesn't even indent things properly.<br>
><br>
><br>
> From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu" class="cremed">llvm-commits-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:llvm-commits-bounces@cs.uiuc.edu" class="cremed">llvm-commits-bounces@cs.uiuc.edu</a>] On Behalf Of Jan Sjodin<br>

> Sent: Thursday, September 13, 2012 1:41 PM<br>
> To: <a href="mailto:llvm-commits@cs.uiuc.edu" class="cremed">llvm-commits@cs.uiuc.edu</a><br>
> Subject: [llvm-commits] Patch to add hidden flag to llvm-nm to not emit aliases.<br>
><br>
> Hi, I posted in llvm-dev some time back about adding a -without-aliases flag that will not include aliases in the output. Here is the patch. Okay to commit?<br>
><br>
> - Jan<br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" class="cremed">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
><br>
><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" class="cremed">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
</div></div></blockquote></div><br></div>