[Mesa-dev] [Bug 59877] Build fail since r600g: Don't build llvm_wrapper.cpp when we aren't using LLVM

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=59877

Johannes Obermayr <johannesobermayr at gmx.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #73792|0                           |1
        is obsolete|                            |

--- Comment #8 from Johannes Obermayr <johannesobermayr at gmx.de> ---
Created attachment 73796
  --> https://bugs.freedesktop.org/attachment.cgi?id=73796&action=edit
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 ...)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20130128/1dd654f8/attachment.html>