Bug 43861

Summary:	Build error in Mesa with llvm >= r145623
Product:	Mesa	Reporter:	Johannes Obermayr <johannesobermayr>
Component:	Drivers/Gallium/r600	Assignee:	Default DRI bug account <dri-devel>
Status:	RESOLVED FIXED	QA Contact:
Severity:	blocker
Priority:	highest	CC:	ojab
Version:	git
Hardware:	All
OS:	Linux (All)
URL:	https://build.opensuse.org/package/rawlog?arch=i586&package=Mesa&project=home%3Ajobermayr&repository=openSUSE_12.1
Whiteboard:
i915 platform:		i915 features:

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.