[Beignet] [PATCH 7/7] backend: Turn on ASM dump.

[Song, Ruiling]

> --- a/backend/src/backend/gen_context.cpp
> +++ b/backend/src/backend/gen_context.cpp
> @@ -2299,6 +2299,14 @@ namespace gbe
>      if (OCL_OUTPUT_ASM)
>        outputAssembly(stdout, genKernel);
> 
> +    if (this->asmFileName) {
> +      FILE *asmDumpStream = fopen(this->asmFileName, "a");
> +      if (asmDumpStream) {
> +        outputAssembly(asmDumpStream, genKernel);
> +        fclose(asmDumpStream);
> +      }
> +    }
> +
>      return true;
>    }
> 
> diff --git a/backend/src/backend/program.cpp
> b/backend/src/backend/program.cpp index 6f64413..6d539a9 100644
> --- a/backend/src/backend/program.cpp
> +++ b/backend/src/backend/program.cpp
> @@ -840,6 +840,9 @@ namespace gbe {
>        }
>        #endif
> 
> +      FILE *asmDumpStream = fopen(dumpASMFileName.c_str(), "w");
> +      if (asmDumpStream)
> +        fclose(asmDumpStream);
> >>>>  is this piece of code useless?
This would clear the content in the dumpASMFileName.
As later " FILE *asmDumpStream = fopen(this->asmFileName, "a");" would append the cl kernel asm to the file one by one. Clearing is needed.

>        p = gbe_program_new_from_llvm(deviceID, NULL, out_module, llvm_ctx,
> dumpASMFileName.c_str(), stringSize,
>                                      err, errSize, optLevel);
>        if (err != NULL)
> --
> 1.9.1
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet