[cfe-commits] r169885 - in /cfe/trunk: include/clang/Driver/Options.td lib/Driver/Tools.cpp test/Driver/clang_f_opts.c

[Benjamin Kramer]

On 11.12.2012, at 18:12, Chad Rosier <mcrosier at apple.com> wrote:

> Author: mcrosier
> Date: Tue Dec 11 11:12:28 2012
> New Revision: 169885
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=169885&view=rev
> Log:
> [driver] Add the -fvectorize flag to enable the loop vectorization passes.
> rdar://12839978 

Should we add -ftree-vectorize as an alias for this flag? It's GCC's way to enable the vectorizer.

- Ben
> 
> Modified:
>    cfe/trunk/include/clang/Driver/Options.td
>    cfe/trunk/lib/Driver/Tools.cpp
>    cfe/trunk/test/Driver/clang_f_opts.c
> 
> Modified: cfe/trunk/include/clang/Driver/Options.td
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Options.td?rev=169885&r1=169884&r2=169885&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/Driver/Options.td (original)
> +++ cfe/trunk/include/clang/Driver/Options.td Tue Dec 11 11:12:28 2012
> @@ -650,6 +650,9 @@
> def ftemplate_backtrace_limit_EQ : Joined<["-"], "ftemplate-backtrace-limit=">,
>                                    Group<f_Group>;
> def ftest_coverage : Flag<["-"], "ftest-coverage">, Group<f_Group>;
> +def fvectorize : Flag<["-"], "fvectorize">, Group<f_Group>,
> +  HelpText<"Enable the loop vectorization passes">;
> +def fno_vectorize : Flag<["-"], "fno-vectorize">, Group<f_Group>;
> def Wlarge_by_value_copy_def : Flag<["-"], "Wlarge-by-value-copy">,
>   HelpText<"Warn if a function definition returns or accepts an object larger "
>            "in bytes than a given value">, Flags<[HelpHidden]>;
> 
> Modified: cfe/trunk/lib/Driver/Tools.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=169885&r1=169884&r2=169885&view=diff
> ==============================================================================
> --- cfe/trunk/lib/Driver/Tools.cpp (original)
> +++ cfe/trunk/lib/Driver/Tools.cpp Tue Dec 11 11:12:28 2012
> @@ -2928,6 +2928,13 @@
>                    false))
>     CmdArgs.push_back("-fasm-blocks");
> 
> +  // -fno-vectorize is default.
> +  if (Args.hasFlag(options::OPT_fvectorize,
> +                   options::OPT_fno_vectorize, false)) {
> +    CmdArgs.push_back("-backend-option");
> +    CmdArgs.push_back("-vectorize-loops");
> +  }
> +
>   if (Arg *A = Args.getLastArg(options::OPT_fshow_overloads_EQ))
>     A->render(Args, CmdArgs);
> 
> 
> Modified: cfe/trunk/test/Driver/clang_f_opts.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/clang_f_opts.c?rev=169885&r1=169884&r2=169885&view=diff
> ==============================================================================
> --- cfe/trunk/test/Driver/clang_f_opts.c (original)
> +++ cfe/trunk/test/Driver/clang_f_opts.c Tue Dec 11 11:12:28 2012
> @@ -38,3 +38,12 @@
> 
> // RUN: %clang -fms-extensions -fenable-experimental-ms-inline-asm %s -### 2>&1 | FileCheck -check-prefix=CHECK-OPTIONS3 %s
> // CHECK-OPTIONS3: -fenable-experimental-ms-inline-asm
> +
> +// RUN: %clang -### -S -fvectorize %s 2>&1 | FileCheck -check-prefix=CHECK-VECTORIZE %s
> +// RUN: %clang -### -S -fno-vectorize -fvectorize %s 2>&1 | FileCheck -check-prefix=CHECK-VECTORIZE %s
> +// RUN: %clang -### -S -fno-vectorize %s 2>&1 | FileCheck -check-prefix=CHECK-NO-VECTORIZE %s
> +// RUN: %clang -### -S -fvectorize -fno-vectorize %s 2>&1 | FileCheck -check-prefix=CHECK-NO-VECTORIZE %s
> +// CHECK-VECTORIZE: "-backend-option"
> +// CHECK-VECTORIZE: "-vectorize-loops"
> +// CHECK-NO-VECTORIZE-NOT: "-backend-option"
> +// CHECK-NO-VECTORIZE-NOT: "-vectorize-loops"
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits