On Mon, 2010-05-03 at 01:42 -0700, Török Edwin wrote:
> On 05/03/2010 11:36 AM, mesa3d-dev-requ...@lists.sourceforge.net wrote:
> > From: Francis Galiegue <fgalie...@gmail.com>
> > Subject: [Mesa3d-dev] LLVM and udis86 dependencies
> > To: mesa3d-dev@lists.sourceforge.net
> > Message-ID:
> >     <x2zac542ca01004280420u3fbcee55rcab4ffd573fb...@mail.gmail.com>
> > Content-Type: text/plain; charset=ISO-8859-1
> > 
> > In the current HEAD, in configure.ac:
> > 
> > ----
> > 1182-    [enable_gallium=yes])
> > 1183-if test "x$enable_gallium" = xyes; then
> > 1184-    SRC_DIRS="$SRC_DIRS gallium gallium/winsys gallium/targets"
> > 1185:    AC_CHECK_HEADER([udis86.h], [HAS_UDIS86="yes"],
> > 1186-                [HAS_UDIS86="no"])
> > 1187-    AC_PATH_PROG([LLVM_CONFIG], [llvm-config], [no])
> > 1188-fi
> > --
> > 1340-   LLVM_LIBS="`$LLVM_CONFIG --libs jit interpreter nativecodegen
> > bitwriter` -lstdc++"
> > 1341-
> > 1342-   if test "x$HAS_UDIS86" != xno; then
> > 1343:       LLVM_LIBS="$LLVM_LIBS -ludis86"
> > 1344-       DEFINES="$DEFINES -DHAVE_UDIS86"
> > 1345-   fi
> > 1346-   LLVM_LDFLAGS=`$LLVM_CONFIG --ldflags`
> > ----
> > 
> > This means basically that the udis86 dependency is "automagic" if you
> > elect to build with LLVM support.
> > 
> > I have a case here of a miscompiled udis86 (missing -fPIC, preventing
> > relocation) on a setup where LLVM was NOT compiled with udis86.
> > 
> > Would it be possible to make the udis86 dependency optional (ie,
> > --with-udis86 option to ./configure)?
> 
> LLVM 2.7 includes a disassembler of its own (libEnhancedDisassembly.so),
> could that one be used instead of udis86?

Yes, that would be nice. 

udis86 seems a bit unmaintained a the moment. I've sent a few patches
for some SSE4 opcodes to the maintainer and they weren't checked in yet.
I thought about bundling its source in mesa, but when I saw news of
LLVM's disassembler plans I thought it was better to wait.

But I don't have time to look into this myself. If you don't have time
either then a simple patch to make udis86 optional should be good enough
for now.

Jose


------------------------------------------------------------------------------
_______________________________________________
Mesa3d-dev mailing list
Mesa3d-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev

Reply via email to