Hi Bas, Pardon for joining so late.
On 18 July 2018 at 13:01, Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> wrote: > radv always needs it, so just check the header instead. Also > do not declare the function if the variable is not set, so we > get a nice compile error instead of failing to open a device > at runtime. > AFAICT the ENABLE_SHADER_CACHE define is set on all POSIX platforms. Back in the early days we had --enable-shader-cache which was removed with d1efa09d342bff3e5def2978a0bef748d74f9c82 Apart from the reasons listed, we do not want to control this at configure time since it will produce varying driver behaviour. Hence multiple unhappy users, as they hit the codepath w/o cache. Skimming through - I would imagine you're building with meson, which has the explicit shader cache toggle. I'm assuming Dylan had it based on early autconf builds? We can drop that one and effectively revert this and the follow-up Android fix. What do you think? Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
