From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 66847] compilation broken with llvm 3.3 Date: Fri, 12 Jul 2013 12:25:50 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0598927464==" Return-path: Received: from culpepper.freedesktop.org (unknown [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id F29EFE5D33 for ; Fri, 12 Jul 2013 05:25:50 -0700 (PDT) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============0598927464== Content-Type: multipart/alternative; boundary="1373631950.40fAd43b1.21795"; charset="us-ascii" --1373631950.40fAd43b1.21795 Date: Fri, 12 Jul 2013 12:25:50 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" https://bugs.freedesktop.org/show_bug.cgi?id=66847 --- Comment #5 from Andreas Boll --- (In reply to comment #4) > 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 http://cgit.freedesktop.org/~tstellar/llvm > 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 http://cgit.freedesktop.org/mesa/mesa/tree/configure.ac?h=9.1#n641 I think that should be enough. -- You are receiving this mail because: You are the assignee for the bug. --1373631950.40fAd43b1.21795 Date: Fri, 12 Jul 2013 12:25:50 +0000 MIME-Version: 1.0 Content-Type: text/html; charset="UTF-8"

Comment # 5 on bug 66847 from 
(In reply to comment #4)
> 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 http://cgit.freedesktop.org/~tstellar/llvm

> 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 http://cgit.freedesktop.org/mesa/mesa/tree/configure.ac?h=9.1#n641
I think that should be enough.

You are receiving this mail because:
  • You are the assignee for the bug.
--1373631950.40fAd43b1.21795-- --===============0598927464== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel --===============0598927464==--