[Mesa-dev] Please test the automake-gallium branch

[Matt Turner]

On Tue, Sep 25, 2012 at 10:57 AM, Török Edwin
<edwin+ml-mesadev at etorok.net> wrote:
> On 09/25/2012 12:52 AM, Matt Turner wrote:
>>
>> Please help me find problems now before it goes into master. Here's a
>> branch. It's easy to test. If you don't test and I commit this in a
>> week and break your stuff, I'm sorry, but you didn't test.
>>
>> http://cgit.freedesktop.org/~mattst88/mesa/log/?h=automake-gallium
>
> 1. Noticed this warning:
> ./configure: line 22447: LLVM_INCLUDEDIR: command not found
>
> This $() in configure.ac should probably be just a $:
>    if grep '\<IntrinsicsR600.td\>' $(LLVM_INCLUDEDIR)/llvm/Intrinsics.td >/dev/null 2>&1; then

Ah, I have this fixed locally and forgot to rebase it into the commit
that adds that code.

> 2. i965 fails to link:
>   CCLD     i965_dri.la
> gcc: error: ../../../../../src/mesa/libdricore/.libs/libdricore9.1.0.so: No such file or directory
>
> Appears to be a typo in src/mesa/Makefile.am: s/DRICORE_SUBDIR/DRICORE_SUBDIRS/
>     SUBDIRS = x86 x86-64 . $(DRICORE_SUBDIR) drivers

Thanks. This was an late addition. I'll fix that.

> 3. Next r600g fails to build:
> R600ISelLowering.cpp: In member function 'virtual llvm::SDValue llvm::R600TargetLowering::LowerOperation(llvm::SDValue, llvm::SelectionDAG&) const':
> R600ISelLowering.cpp:363:10: error: 'r600_read_ngroups_x' was not declared in this scope
> R600ISelLowering.cpp:365:10: error: 'r600_read_ngroups_y' was not declared in this scope
> R600ISelLowering.cpp:367:10: error: 'r600_read_ngroups_z' was not declared in this scope
> R600ISelLowering.cpp:369:10: error: 'r600_read_global_size_x' was not declared in this scope
> R600ISelLowering.cpp:371:10: error: 'r600_read_global_size_y' was not declared in this scope
> R600ISelLowering.cpp:373:10: error: 'r600_read_global_size_z' was not declared in this scope
> R600ISelLowering.cpp:375:10: error: 'r600_read_local_size_x' was not declared in this scope
> R600ISelLowering.cpp:377:10: error: 'r600_read_local_size_y' was not declared in this scope
> R600ISelLowering.cpp:379:10: error: 'r600_read_local_size_z' was not declared in this scope
> R600ISelLowering.cpp:382:10: error: 'r600_read_tgid_x' was not declared in this scope
> R600ISelLowering.cpp:385:10: error: 'r600_read_tgid_y' was not declared in this scope
> R600ISelLowering.cpp:388:10: error: 'r600_read_tgid_z' was not declared in this scope
> R600ISelLowering.cpp:391:10: error: 'r600_read_tidig_x' was not declared in this scope
> R600ISelLowering.cpp:394:10: error: 'r600_read_tidig_y' was not declared in this scope
> R600ISelLowering.cpp:397:10: error: 'r600_read_tidig_z' was not declared in this scope

Not sure I've seen this. Maybe related to recent changes. I'll check
it out either way.

Thanks for testing!
Matt