Bug 43861 - Build error in Mesa with llvm >= r145623
Summary: Build error in Mesa with llvm >= r145623
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: Drivers/Gallium/r600 (show other bugs)
Version: git
Hardware: All Linux (All)
: highest blocker
Assignee: Default DRI bug account
QA Contact:
URL: https://build.opensuse.org/package/ra...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-15 08:29 UTC by Johannes Obermayr
Modified: 2013-08-19 13:35 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Johannes Obermayr 2011-12-15 08:29:06 UTC
r600_shader.c: At top level:
r600_shader.c:190:44: error: array size missing in 'r600_shader_tgsi_instruction'
r600_shader.c:190:76: error: array size missing in 'eg_shader_tgsi_instruction'
r600_shader.c:190:106: error: array size missing in 'cm_shader_tgsi_instruction'
Comment 1 Alex Deucher 2011-12-15 08:49:21 UTC
Builds fine here.
Comment 2 Johannes Obermayr 2011-12-17 14:31:27 UTC
I assume it is because 'llvm-config --{c,cpp,cxx}flags' added '-pedantic' after change to llvm-config-2:
http://llvm.org/viewvc/llvm-project?view=rev&revision=145623

r145618 works
r145619 unrelated (test commit)
r145620 and r145621 not LLVM tree
r145622 more or less unrelated before merge
r145623 build error after merge
Comment 3 Michel Dänzer 2011-12-19 07:21:05 UTC
I'd still argue it's not our bug, strictly speaking: llvm-config should only expose compiler flags absolutely necessary for building against LLVM, such as -I<...> -D<...>, certainly not such generic ones as -pedantic. We've run into similar issues before with other flags.
Comment 4 Laurent carlier 2013-08-19 13:35:39 UTC
Probably fixed long time ago


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.