[Mesa-dev] [PATCH 5/5] clover: Add environment variables for dumping kernel code v2

[Kai Wasserbäch]

Tom Stellard wrote on 09.10.2014 17:07:
> There are two debug variables:
> 
> CLOVER_DEBUG which you can set to any combination of llvm,clc,asm
> (separated by commas) to dump llvm IR, OpenCL C, and native assembly.
> 
> CLOVER_DEBUG_FILE which you can set to a file name for dumping output
> instead of stderr.  If you set this variable, the output will be split
> into three separate files with different suffixes: .cl for OpenCL C,
> .ll for LLVM IR, and .asm for native assembly.  Note that when data
> is written, it is always appended to the files.
> 
> v2:
>   - Code cleanups
>   - Add CLOVER_DEBUG_FILE environment variable for dumping to a file.
> ---
>  .../state_trackers/clover/llvm/invocation.cpp      | 92 +++++++++++++++++++---
>  1 file changed, 82 insertions(+), 10 deletions(-)
> 
> diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp b/src/gallium/state_trackers/clover/llvm/invocation.cpp
> index 6349769..b7edcac 100644
> --- a/src/gallium/state_trackers/clover/llvm/invocation.cpp
> +++ b/src/gallium/state_trackers/clover/llvm/invocation.cpp
> @@ -61,6 +61,8 @@
>  #include <llvm/Support/TargetRegistry.h>
>  #include <llvm/Transforms/IPO.h>
>  #include <llvm/Transforms/IPO/PassManagerBuilder.h>
> +#include "llvm/Transforms/Utils/Cloning.h"

Shouldn't this be: #include <llvm/Transforms/Utils/Cloning.h>

> +

Is the additional blank line required/intended?


Cheers,
Kai

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 648 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20141009/425a8ab5/attachment.sig>