Bug 138825 - [GTK] Mesa build fails over llvm-3.5
Summary: [GTK] Mesa build fails over llvm-3.5
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: ChangSeok Oh
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-11-18 01:51 PST by ChangSeok Oh
Modified: 2014-11-18 05:40 PST (History)
4 users (show)

See Also:


Attachments
Patch (2.73 KB, patch)
2014-11-18 01:56 PST, ChangSeok Oh
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description ChangSeok Oh 2014-11-18 01:51:00 PST
I saw a build break for Mesa while running update-weebkitgtk-libs on Ubuntu 14.10.

  CXX      gallivm/lp_bld_debug.lo
gallivm/lp_bld_debug.cpp:61:32: fatal error: llvm/ADT/OwningPtr.h: No such file or directory
 #include <llvm/ADT/OwningPtr.h>
                                ^
compilation terminated.
Makefile:1741: recipe for target 'gallivm/lp_bld_debug.lo' failed
make[3]: *** [gallivm/lp_bld_debug.lo] Error 1
make[3]: *** Waiting for unfinished jobs....
gallivm/lp_bld_tgsi_soa.c: In function 'build_gather':
gallivm/lp_bld_tgsi_soa.c:987:26: warning: 'temp_ptr' may be used uninitialized in this function [-Wmaybe-uninitialized]
             LLVMValueRef val = LLVMBuildLoad(builder, temp_ptr, "");
                          ^
gallivm/lp_bld_tgsi_soa.c:955:17: note: 'temp_ptr' was declared here
    LLVMValueRef temp_ptr;
                 ^
make[3]: Leaving directory '/home/changseok/Projects/WebKit/WebKitBuild/DependenciesGTK/Source/Mesa/src/gallium/auxiliary'
Makefile:1775: recipe for target 'all-recursive' failed
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory '/home/changseok/Projects/WebKit/WebKitBuild/DependenciesGTK/Source/Mesa/src/gallium/auxiliary'
Makefile:529: recipe for target 'all-recursive' failed
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory '/home/changseok/Projects/WebKit/WebKitBuild/DependenciesGTK/Source/Mesa/src'
Makefile:578: recipe for target 'all-recursive' failed
make: *** [all-recursive] Error 1
*** Error during phase build of mesa: ########## Error running make -j 8 *** [28/29]
*** module webkitgtk-testing-dependencies not built due to non buildable mesa *** [29/29]
*** the following modules were not built *** [29/29]
mesa webkitgtk-testing-dependencies
Running jhbuild-wrapper build failed.
Died at ./Tools/Scripts/update-webkitgtk-libs line 24.

After investiongation on this, I found a relevant fix from mesa upstream. http://cgit.freedesktop.org/mesa/mesa/commit/src/gallium/auxiliary/gallivm/lp_bld_debug.cpp?id=564821c917f4a9d5a0de2ee77b90b0cd85e3d3a6
I believe we need it.
Comment 1 ChangSeok Oh 2014-11-18 01:56:13 PST
Created attachment 241770 [details]
Patch
Comment 2 WebKit Commit Bot 2014-11-18 05:40:42 PST
Comment on attachment 241770 [details]
Patch

Clearing flags on attachment: 241770

Committed r176255: <http://trac.webkit.org/changeset/176255>
Comment 3 WebKit Commit Bot 2014-11-18 05:40:46 PST
All reviewed patches have been landed.  Closing bug.