[Mesa-dev] [PATCH 2/7] clover: Dump the OpenCL C code earlier

Tom Stellard tom at stellard.net
Fri Dec 19 13:08:31 PST 2014


On Sun, Dec 14, 2014 at 11:31:22AM +0100, EdB wrote:
> ---
>  src/gallium/state_trackers/clover/llvm/invocation.cpp | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp b/src/gallium/state_trackers/clover/llvm/invocation.cpp
> index 10dbe08..510e195 100644
> --- a/src/gallium/state_trackers/clover/llvm/invocation.cpp
> +++ b/src/gallium/state_trackers/clover/llvm/invocation.cpp
> @@ -709,6 +709,9 @@ clover::compile_program_llvm(const compat::string &source,
>     llvm_ctx.setDiagnosticHandler(diagnostic_handler, &r_log);
>  #endif
>  
> +   if (debug_flags & DBG_CLC)
> +      debug_log(source, ".cl");
> +

Why do we need to do this?  Could you add an explanation to the commit message?

>     // The input file name must have the .cl extension in order for the
>     // CompilerInvocation class to recognize it as an OpenCL source file.
>     llvm::Module *mod = compile_llvm(llvm_ctx, source, headers, "input.cl",
> @@ -719,9 +722,6 @@ clover::compile_program_llvm(const compat::string &source,
>  
>     optimize(mod, optimization_level, kernels);
>  
> -   if (debug_flags & DBG_CLC)
> -      debug_log(source, ".cl");
> -
>     if (debug_flags & DBG_LLVM) {
>        std::string log;
>        llvm::raw_string_ostream s_log(log);
> -- 
> 2.2.0
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list