Tom Stellard <[email protected]> writes:

> From: Tom Stellard <[email protected]>
>
> This way users can see compiler warnings.
> ---
>  src/gallium/state_trackers/clover/llvm/invocation.cpp | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp 
> b/src/gallium/state_trackers/clover/llvm/invocation.cpp
> index e6cd415..290c31a 100644
> --- a/src/gallium/state_trackers/clover/llvm/invocation.cpp
> +++ b/src/gallium/state_trackers/clover/llvm/invocation.cpp
> @@ -206,6 +206,9 @@ namespace {
>        if (!c.ExecuteAction(act))
>           throw build_error(log);
>  
> +      // Display any warnings
> +      std::cerr << log;
> +
>        // Get address spaces map to be able to find kernel argument address 
> space
>        memcpy(address_spaces, c.getTarget().getAddressSpaceMap(), 
>                                                          
> sizeof(address_spaces));

Isn't this supposed to be part of the application's policy to decide how
where and when to print compilation errors and warnings?  Maybe wrap it
between #ifdef DEBUG/#endif or use gallium's debug_printf so it's only
printed out in debug builds?

Attachment: pgppX3nK33zIY.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to