<p dir="ltr">LGTM</p>
<div class="gmail_quote">On Dec 6, 2013 6:36 AM, "Diego Novillo" <<a href="mailto:dnovillo@google.com">dnovillo@google.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi chandlerc,<br>
<br>
GCC uses -fauto-profile to enable sample-based PGO. This patch<br>
adds it to Clang as an alias for -fprofile-sample-use.<br>
<br>
OK for trunk?<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2353" target="_blank">http://llvm-reviews.chandlerc.com/D2353</a><br>
<br>
Files:<br>
  include/clang/Driver/Options.td<br>
  test/Driver/clang_f_opts.c<br>
<br>
Index: include/clang/Driver/Options.td<br>
===================================================================<br>
--- include/clang/Driver/Options.td<br>
+++ include/clang/Driver/Options.td<br>
@@ -369,6 +369,8 @@<br>
 def fprofile_sample_use_EQ : Joined<["-"], "fprofile-sample-use=">,<br>
     Group<f_Group>, Flags<[DriverOption, CC1Option]>,<br>
     HelpText<"Enable sample-based profile guided optimizations">;<br>
+def fauto_profile_EQ : Joined<["-"], "fauto-profile=">,<br>
+    Alias<fprofile_sample_use_EQ>;<br>
<br>
 def fblocks : Flag<["-"], "fblocks">, Group<f_Group>, Flags<[CC1Option]>,<br>
   HelpText<"Enable the 'blocks' language feature">;<br>
Index: test/Driver/clang_f_opts.c<br>
===================================================================<br>
--- test/Driver/clang_f_opts.c<br>
+++ test/Driver/clang_f_opts.c<br>
@@ -54,6 +54,9 @@<br>
 // RUN: %clang -### -S -fprofile-sample-use=%S/Inputs/file.prof %s 2>&1 | FileCheck -check-prefix=CHECK-SAMPLE-PROFILE %s<br>
 // CHECK-SAMPLE-PROFILE: "-fprofile-sample-use={{.*}}/file.prof"<br>
<br>
+// RUN: %clang -### -S -fauto-profile=%S/Inputs/file.prof %s 2>&1 | FileCheck -check-prefix=CHECK-AUTO-PROFILE %s<br>
+// CHECK-AUTO-PROFILE: "-fprofile-sample-use={{.*}}/file.prof"<br>
+<br>
 // RUN: %clang -### -S -fvectorize %s 2>&1 | FileCheck -check-prefix=CHECK-VECTORIZE %s<br>
 // RUN: %clang -### -S -fno-vectorize -fvectorize %s 2>&1 | FileCheck -check-prefix=CHECK-VECTORIZE %s<br>
 // RUN: %clang -### -S -fno-vectorize %s 2>&1 | FileCheck -check-prefix=CHECK-NO-VECTORIZE %s<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div>