[Mesa-dev] [PATCHv2 3/4] configure.ac: Separate HAVE_LLVM defines for gallium and radv

Dave Airlie airlied at gmail.com
Thu Feb 2 00:00:44 UTC 2017


On 2 February 2017 at 09:58, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> On 1 February 2017 at 22:19, Dave Airlie <airlied at gmail.com> wrote:
>> Just make radv depend on enable-gallium-llvm.
>>
>> I really don't want to make the #defines into the spaghetti this introduces.
>>
> Not a huge fan of it either.
>
> We need these for 17.0 and flipping things in a feature freeze period
> is iffy. For 17.1.x we'll remove this and a lot more nasty code by
> superseding --enable-gallium-llvm in favour of --enable-llvm.

Why do we need it, just make radv not build if enable-gallium-llvm
isn't specified.

Dave.


More information about the mesa-dev mailing list