<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:johannesobermayr@gmx.de" title="Johannes Obermayr <johannesobermayr@gmx.de>"> <span class="fn">Johannes Obermayr</span></a>
</span> changed
<a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Build fail since r600g: Don't build llvm_wrapper.cpp when we aren't using LLVM"
href="https://bugs.freedesktop.org/show_bug.cgi?id=59877">bug 59877</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Attachment #73792 is obsolete</td>
<td>
</td>
<td>1
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Build fail since r600g: Don't build llvm_wrapper.cpp when we aren't using LLVM"
href="https://bugs.freedesktop.org/show_bug.cgi?id=59877#c8">Comment # 8</a>
on <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - Build fail since r600g: Don't build llvm_wrapper.cpp when we aren't using LLVM"
href="https://bugs.freedesktop.org/show_bug.cgi?id=59877">bug 59877</a>
from <span class="vcard"><a class="email" href="mailto:johannesobermayr@gmx.de" title="Johannes Obermayr <johannesobermayr@gmx.de>"> <span class="fn">Johannes Obermayr</span></a>
</span></b>
<pre>Created <span class=""><a href="attachment.cgi?id=73796" name="attach_73796" title="v2: Right attempt to fix it on current master">attachment 73796</a> <a href="attachment.cgi?id=73796&action=edit" title="v2: Right attempt to fix it on current master">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=59877&attachment=73796'>[review]</a>
v2: Right attempt to fix it on current master
It shouldn't fail because of:
if USE_R600_LLVM_COMPILER
if HAVE_GALLIUM_COMPUTE
[...]
endif
AM_CFLAGS += \
-I$(top_srcdir)/src/gallium/drivers/radeon/ \
$(LLVM_CFLAGS) \
-DR600_USE_LLVM
endif
So the compiler should find radeon_llvm.h on configure
--enable-r600-llvm-compiler without --enable-opencl.
I moved $(LLVM_CFLAGS). So the compiler should also find
$(LLVM_INCLUDEDIR)/llvm-c/Core.h.
On this version I also changed configure.ac to set USE_R600_LLVM_COMPILER also
for --enable-opencl without --enable-r600-llvm-compiler to reflect the
dependency (if) here.
@Tom: You as developer should know it is hard to fix things if you cannot
reproduce issues and people don't tell what happened ...
Because it has not been fixed the right way (maybe only my opinion) the
discussion should be where the bug was reported. (I hate jumping between
various places to get information for one topic/issue ...)</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>