Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Will Wagner <will_wagner@carallon.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 0/4] llvm for mesa3d
Date: Fri, 16 Feb 2018 09:42:42 +0000	[thread overview]
Message-ID: <0d3785ce-032b-330c-d7ff-fb92bcc07f1d@carallon.com> (raw)
In-Reply-To: <20180216083606.6472-1-valentin.korenblit@smile.fr>

On 16/02/2018 08:36, Valentin Korenblit wrote:
> Hello all,
> 
> This series provides LLVM support for Mesa3d, enabling
> llvmpipe software rasterizer.
> 
> It must be considered that it does not provide a full LLVM
> installation for the host, only the necessary components to
> cross-compile LLVM for the target. It will install all target
> components as libLLVM.so.

Hi Valentin,

As it happens I've just been updating my patches for llvm as I added it 
to get radeonsi gallium working in mesa3d. My patches are much messier 
than your and hence I've never tried to upstream anything.

A couple of overall questions:

Have you only added host-llvm because of llvm-tblgen and llvm-config? 
llvm will try to build native versions of those for you but I had 
problems with llvm-tblgen and so had to go to a host package like you. 
However I only built those two tools with:

HOST_LLVM_MAKE_OPTS = llvm-tblgen llvm-config

It makes the build smaller and faster and it means you don't need to 
care about almost all of the host conf options.


My other question is about AMDGPU support, you seem to have mixed this 
up into a number of your patches not just number 3. What are you trying 
to do here, add an option to build amdgpu as well as the target arch? In 
my case I wanted to be able to build for only amdgpu and not the target 
arch as this is what radeonsi needs.

Regards
Will

-- 
------------------------------------------------------------------------
Will Wagner                                     will_wagner at carallon.com
Engineering Manager                      Office Tel: +44 (0)20 7471 9424
Carallon Ltd, International House, 7 High Street, Ealing, London  W5 5DB
------------------------------------------------------------------------

  parent reply	other threads:[~2018-02-16  9:42 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-16  8:36 [Buildroot] [PATCH 0/4] llvm for mesa3d Valentin Korenblit
2018-02-16  8:36 ` [Buildroot] [PATCH 1/4] package/llvm: new host package Valentin Korenblit
2018-02-16  8:36 ` [Buildroot] [PATCH 2/4] package/llvm: enable target variant Valentin Korenblit
2018-02-16  8:36 ` [Buildroot] [PATCH 3/4] package/llvm: add AMDGPU support Valentin Korenblit
2018-02-16  8:36 ` [Buildroot] [PATCH 4/4] package/mesa3d: enable llvm support Valentin Korenblit
2018-02-16  9:42 ` Will Wagner [this message]
2018-02-16 14:31   ` [Buildroot] [PATCH 0/4] llvm for mesa3d Valentin KORENBLIT
2018-02-17 23:00     ` nimaim
2018-02-19 12:38       ` Valentin KORENBLIT
2018-02-19 14:28         ` Nimai Mahajan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=0d3785ce-032b-330c-d7ff-fb92bcc07f1d@carallon.com \
    --to=will_wagner@carallon.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox