[Mesa-dev] [PATCH 0/8] LLVM requirement for drivers using draw

Emil Velikov emil.l.velikov at gmail.com
Thu Jan 26 18:27:00 UTC 2017


Hi all,

Here's a few small fixes/functionality improvements when dealing with 
LLVM.

Most notably the series adds "LLVM" string [when applicable] to the .get_name()
callback for drivers that use draw.

Thus developer can respond accordingly - be that "rebuild with LLVM or 
enjoy the bad performance" or otherwise to reports.

We can go a step further and make both(?) configure and the callback 
produce more nagging message along the lines of "built w/o LLVM expect bad 
performance", if people prefer.

With this in mind we can drop the LLVM requirement, which some 
builders/distros explicitly patch out.

Let's be nice to each other and not force it onto them.

What do you guys think ?
Emil
 
Emil Velikov (8):
  virgl: remove unused draw_context.h include
  llvmpipe: use draw_get_option_use_llvm() instead of open coding it
  softpipe: set softpipe_screen::use_llvm when draw is build with LLVM
  softpipe: let .get_name() append LLVM if built with LLVM
  i915g: use draw_get_option_use_llvm() instead of open coding it
  i915g: let .get_name() append LLVM if built with LLVM
  r300: let .get_name() append LLVM if built with LLVM
  nouveau: let .get_name() append LLVM if built with LLVM

 src/gallium/drivers/i915/i915_screen.c       |  9 +++++++--
 src/gallium/drivers/llvmpipe/lp_screen.c     |  4 ++--
 src/gallium/drivers/nouveau/nouveau_screen.c |  7 ++++++-
 src/gallium/drivers/r300/r300_screen.c       | 10 +++++++++-
 src/gallium/drivers/softpipe/sp_screen.c     |  6 ++++++
 src/gallium/drivers/virgl/virgl_screen.c     |  1 -
 6 files changed, 30 insertions(+), 7 deletions(-)

-- 
2.11.0



More information about the mesa-dev mailing list