[Mesa-dev] [PATCH mesa] freedreno: gallium driver for adreno

Marek Olšák maraeo at gmail.com
Tue Feb 19 04:23:43 PST 2013


On Mon, Feb 18, 2013 at 7:58 PM, Rob Clark <robdclark at gmail.com> wrote:
> On Mon, Feb 18, 2013 at 12:47 PM, Matt Turner <mattst88 at gmail.com> wrote:
>> On Sun, Feb 17, 2013 at 11:33 AM, Rob Clark <robdclark at gmail.com> wrote:
>>>
>>> diff --git a/src/gallium/drivers/freedreno/Makefile.am b/src/gallium/drivers/freedreno/Makefile.am
>>> new file mode 100644
>>> index 0000000..10abdfb
>>> --- /dev/null
>>> +++ b/src/gallium/drivers/freedreno/Makefile.am
>>> @@ -0,0 +1,35 @@
>>> +include $(top_srcdir)/src/gallium/Automake.inc
>>> +
>>> +noinst_LTLIBRARIES = libfreedreno.la
>>> +
>>> +AM_CFLAGS = \
>>> +       -Werror -Wno-packed-bitfield-compat \
>>> +       -I$(top_srcdir)/src/gallium/include \ <--
>>> +       -I$(top_srcdir)/src/gallium/auxiliary \ <--
>>> +       -I$(top_srcdir)/src/gallium/drivers \
>>> +       -I$(top_srcdir)/include \ <--
>>> +       $(FREEDRENO_CFLAGS) \
>>> +       $(DEFINES) \ <--
>>> +       $(PIC_FLAGS) \
>>> +       $(VISIBILITY_CFLAGS)
>>
>> The <-- mark things that are provided by the GALLIUM_CFLAGS variable
>> in Automake.inc that you've already included. PIC_FLAGS is now dead.
>> Distributions don't like -Werror being hardcoded into upstream's
>> CFLAGS.
>
> Hmm, is there a better way to get -Werror for just freedreno when I am
> building myself?  I do find that it is pretty useful to let the
> compiler help me catch problems rather than debugging them the hard
> way ;-)

You can set the CFLAGS and CXXFLAGS environment variables before
configuring Mesa.

Marek


More information about the mesa-dev mailing list