Re: [Mesa-dev] [PATCH 2/2] ac/nir_to_llvm: add frexp support

2018-03-20 Thread Marek Olšák
I think we should set READNONE on the intrinsic calls. Other than that: Reviewed-by: Marek Olšák Marek On Mon, Mar 19, 2018 at 10:09 PM, Timothy Arceri wrote: > Fixes CTS tests: > KHR-GL40.gpu_shader_fp64.builtin.frexp_double > KHR-GL40.gpu_shader_fp64.builtin.frexp_dvec2 > KHR-GL40.gpu_shad

[Mesa-dev] [PATCH 2/2] ac/nir_to_llvm: add frexp support

2018-03-19 Thread Timothy Arceri
Fixes CTS tests: KHR-GL40.gpu_shader_fp64.builtin.frexp_double KHR-GL40.gpu_shader_fp64.builtin.frexp_dvec2 KHR-GL40.gpu_shader_fp64.builtin.frexp_dvec3 KHR-GL40.gpu_shader_fp64.builtin.frexp_dvec4 And piglit test: tests/spec/arb_gpu_shader_fp64/execution/built-in-functions/fs-frexp-dvec4.shader_t