vsk added a comment.

This lgtm with one nit, and pending approval from others.


================
Comment at: include/clang/Driver/Options.td:482
@@ +481,3 @@
+def fpgo_train_EQ : Joined<["-"], "fpgo-train=">,
+    Group<f_Group>, Flags<[DriverOption]>, MetaVarName<"<file>">,
+    HelpText<"Generate instrumented code to collect execution counts into 
<file> (overridden by LLVM_PROFILE_FILE env var)">;
----------------
Would something like `<options>` be a better MetaVarName?


Repository:
  rL LLVM

http://reviews.llvm.org/D21823



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to