Summary: | [clover] build failure in llvm/codegen/native.cpp:129:52 | ||
---|---|---|---|
Product: | Mesa | Reporter: | Iaroslav Andrusyak <pontostroy> |
Component: | Other | Assignee: | mesa-dev |
Status: | RESOLVED FIXED | QA Contact: | mesa-dev |
Severity: | normal | ||
Priority: | medium | ||
Version: | git | ||
Hardware: | All | ||
OS: | Linux (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Attachments: | clover-llvm-codegen-native.cpp.patch |
Description
Iaroslav Andrusyak
2016-07-21 11:21:10 UTC
I'm back from vacation and can second this with LLVM 3.9.0svn and current (today) LLVM 4.0.0svn my post was titled: clover: compilation error with former LLVM 3.9.0svn and current LLVM 4.0.0svn This looks like an old compiler (not fully c++11 capable). what version are you using? c++ (SUSE Linux) 4.8.3 openSUSE 13.2 system version (worked for ages) Iaroslav is on openSUSE Leap 42.1+ (4.8.5) or updated to 5.x, I think. (In reply to Dieter Nützel from comment #3) > c++ (SUSE Linux) 4.8.3 > > openSUSE 13.2 system version (worked for ages) > > Iaroslav is on openSUSE Leap 42.1+ (4.8.5) or updated to 5.x, I think. Clover was significantly rewritten few weeks ago. I'd guess this error does not happen before 600ac51448f0dc62808e962a206d11e1b179a32f It works OK with gcc-4.9.3. The question is whether to bump clover reqs or avoid using the feature. Seems like a GCC bug... You may be able to work around the issue by using the old-fashioned constructor call syntax with parentheses instead of braces to initialize the "fos" variable. (In reply to Francisco Jerez from comment #5) > Seems like a GCC bug... You may be able to work around the issue by using > the old-fashioned constructor call syntax with parentheses instead of braces > to initialize the "fos" variable. This one worked (see attachment). OpenGL renderer string: Gallium 0.4 on AMD TURKS (DRM 2.43.0 / 4.6.4-6.g684e9e1-default, LLVM 4.0.0) OpenGL core profile version string: 3.3 (Core Profile) Mesa 12.1.0-devel (git-e7b2ce5) opencl-example / run_tests.sh Passed 71 passes, 0 fails Created attachment 125305 [details] [review] clover-llvm-codegen-native.cpp.patch (In reply to Dieter Nützel from comment #6) > (In reply to Francisco Jerez from comment #5) > > Seems like a GCC bug... You may be able to work around the issue by using > > the old-fashioned constructor call syntax with parentheses instead of braces > > to initialize the "fos" variable. > > This one worked (see attachment). > > OpenGL renderer string: Gallium 0.4 on AMD TURKS (DRM 2.43.0 / > 4.6.4-6.g684e9e1-default, LLVM 4.0.0) > OpenGL core profile version string: 3.3 (Core Profile) Mesa 12.1.0-devel > (git-e7b2ce5) > > opencl-example / run_tests.sh > > Passed > 71 passes, 0 fails Seems reasonable, would you mind sending the patch for review to the mailing list? (feel free to add me to the CC list) REOPENED until commit lands in Mesa git. Should be fixed in master now. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.