Thanks! Corrected in r<span style="line-height:19.799999237060547px">204588.</span><br><br><div>On Sun Mar 23 2014 at 11:49:51 AM, Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Mar 23, 2014 at 10:47 AM, David Majnemer <span dir="ltr"><<a href="mailto:david.majnemer@gmail.com" target="_blank">david.majnemer@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: majnemer<br>
Date: Sun Mar 23 12:47:22 2014<br>
New Revision: 204564<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=204564&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=204564&view=rev</a><br>
Log:<br>
Driver: Map /Gy to -ffunction-sections<br>
<br>
/Gy is equivalent to -ffunction-sections.<br>
/Gy- is equivalent to -fno-function-sections.<br>
<br>
Currently, LLVM doesn't do anything interesting with -ffunction-sections<br>
under WinCOFF.<br>
<br>
Modified:<br>
    cfe/trunk/include/clang/Driver/CLCompatOptions.td<br>
    cfe/trunk/test/Driver/cl-options.c<br>
<br>
Modified: cfe/trunk/include/clang/Driver/CLCompatOptions.td<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/CLCompatOptions.td?rev=204564&r1=204563&r2=204564&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/CLCompatOptions.td?rev=204564&r1=204563&r2=204564&view=diff</a><br>


==============================================================================<br>
--- cfe/trunk/include/clang/Driver/CLCompatOptions.td (original)<br>
+++ cfe/trunk/include/clang/Driver/CLCompatOptions.td Sun Mar 23 12:47:22 2014<br>
@@ -62,6 +62,10 @@ def _SLASH_GR : CLFlag<"GR">, HelpText<"<br>
 def _SLASH_GR_ : CLFlag<"GR-">, HelpText<"Disable RTTI">, Alias<fno_rtti>;<br>
 def _SLASH_GF_ : CLFlag<"GF-">, HelpText<"Disable string pooling">,<br>
   Alias<fwritable_strings>;<br>
+def _SLASH_Gy : CLFlag<"Gy">, HelpText<"Put each function in it's own section">,<br>
+  Alias<ffunction_sections>;<br>
+def _SLASH_Gy_ : CLFlag<"Gy-">, HelpText<"Don't put each function in it's own section">,<br>
+  Alias<fno_function_sections>;<br></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Typo: "it's" should be "its"</div></div>
</div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

 def _SLASH_help : CLFlag<"help">, Alias<help>,<br>
   HelpText<"Display available options">;<br>
 def _SLASH_HELP : CLFlag<"HELP">, Alias<help>;<br>
@@ -225,8 +229,6 @@ def _SLASH_GS : CLFlag<"GS">;<br>
 def _SLASH_Gs : CLJoined<"Gs">;<br>
 def _SLASH_GT : CLFlag<"GT">;<br>
 def _SLASH_GX : CLFlag<"GX">;<br>
-def _SLASH_Gy : CLFlag<"Gy">;<br>
-def _SLASH_Gy_ : CLFlag<"Gy-">;<br>
 def _SLASH_Gz : CLFlag<"Gz">;<br>
 def _SLASH_GZ : CLFlag<"GZ">;<br>
 def _SLASH_H : CLFlag<"H">;<br>
<br>
Modified: cfe/trunk/test/Driver/cl-options.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cl-options.c?rev=204564&r1=204563&r2=204564&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cl-options.c?rev=204564&r1=204563&r2=204564&view=diff</a><br>


==============================================================================<br>
--- cfe/trunk/test/Driver/cl-options.c (original)<br>
+++ cfe/trunk/test/Driver/cl-options.c Sun Mar 23 12:47:22 2014<br>
@@ -24,6 +24,12 @@<br>
 // RUN: %clang_cl /GR- -### -- %s 2>&1 | FileCheck -check-prefix=GR_ %s<br>
 // GR_: -fno-rtti<br>
<br>
+// RUN: %clang_cl /Gy -### -- %s 2>&1 | FileCheck -check-prefix=Gy %s<br>
+// Gy: -ffunction-sections<br>
+<br>
+// RUN: %clang_cl /Gy /Gy- -### -- %s 2>&1 | FileCheck -check-prefix=Gy_ %s<br>
+// Gy_-NOT: -ffunction-sections<br>
+<br>
 // RUN: %clang_cl /Imyincludedir -### -- %s 2>&1 | FileCheck -check-prefix=SLASH_I %s<br>
 // RUN: %clang_cl /I myincludedir -### -- %s 2>&1 | FileCheck -check-prefix=SLASH_I %s<br>
 // SLASH_I: "-I" "myincludedir"<br>
@@ -198,8 +204,6 @@<br>
 // RUN:     /Gs1000 \<br>
 // RUN:     /GT \<br>
 // RUN:     /GX \<br>
-// RUN:     /Gy \<br>
-// RUN:     /Gy- \<br>
 // RUN:     /Gz \<br>
 // RUN:     /GZ \<br>
 // RUN:     /H \<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div></div></div></blockquote>