<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - compilation broken with llvm 3.3" href="https://bugs.freedesktop.org/show_bug.cgi?id=66847#c5">Comment # 5</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - compilation broken with llvm 3.3" href="https://bugs.freedesktop.org/show_bug.cgi?id=66847">bug 66847</a> from <a class="email" href="mailto:andreas.boll.dev@gmail.com" title="Andreas Boll <andreas.boll.dev@gmail.com>"> Andreas Boll</a> <pre>(In reply to <a href="show_bug.cgi?id=66847#c4">comment #4</a>) > You should also remove "or newer". Yes, will do. > Also "master" isn't the right branch > (which one is the right?). master is the right branch. See <a href="http://cgit.freedesktop.org/~tstellar/llvm">http://cgit.freedesktop.org/~tstellar/llvm</a> > While you are at it, checking for r600 target > isn't enough, because it also needs "host". > AFAIK the host target is needed for llvmpipe and draw-llvm too. So a more generic check would be required. But that's stuff for mesa master and should be in a separate commit anyway. > I have the feeling that we should disable r600-llvm support in 9.1 release > alltogether. I think it was just for testing anyway and likely unstable/lots > of bugs still. It also saves distros from compiling some experimental > version of llvm. For initial radeonsi support it's required. The r600g llvm backend isn't enabled by default and the help string says: Enable experimental LLVM backend for graphics shaders. See <a href="http://cgit.freedesktop.org/mesa/mesa/tree/configure.ac?h=9.1#n641">http://cgit.freedesktop.org/mesa/mesa/tree/configure.ac?h=9.1#n641</a> I think that should be enough.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>