[Mesa-dev] [PATCH v3] clover: Introduce CLOVER_EXTRA_COMPILER_OPTIONS

[Vedran Miletić]

On 09/07/2016 04:14 PM, Jan Vesely wrote:
> On Wed, 2016-09-07 at 13:43 +0200, Vedran Miletić wrote:
>> Options specified via the CLOVER_EXTRA_COMPILER_OPTIONS shell
>> variable
>> are appended to the compiler and linker options specified by the
>> OpenCL
>> program (if any).
>>
>> v2:
>>  * rename to CLOVER_EXTRA_COMPILER_OPTIONS
>>  * use debug_get_option
>>  * append to linker options as well
> 
> is this safe? sets of recognized compiler and linker options are
> distinct, so there's a risk of getting
> CL_INVALID_COMPILER_OPTION/CL_INVALID_LINKER_OPTION if the env var
> includes options that is permitted by one and not the other.
> 
> since this is a debug mechanism I'm OK if it just produces warning.
> 
> Jan
> 

Indeed. We could use CLOVER_EXTRA_LINKER_OPTIONS then. v4 incoming.

Regards,
Vedran

-- 
Vedran Miletić
vedran.miletic.net