On Sat, 2016-12-03 at 14:33 -0800, Vinson Lee wrote: > Hi, Tobias. > > <builddir>/bin/llvm-config --includedir returns <srcdir>/include. I am > not setting the "--with-llvm-prefix" configure option.
Have you completed mesa build with only the configure.ac fix? I'm not sure if using llvm without make install is supported, but I'd expect the srcdir headers to fail at some point. headers from llvm srcdir are incomplete; parts are generated by the build process. regards, Jan > > Cheers, > Vinson > > > On Sat, Dec 3, 2016 at 1:46 PM, Tobias Droste <[email protected]> wrote: > > Hi Vinson, > > > > > > > > why is LLVM_INCLUDEDIR not <builddir>/include? How does this happen? > > > > Do you set "--with-llvm-prefix=<builddir>" when building mesa? > > > > > > > > Tobias > > > > > > > > Am Samstag, 3. Dezember 2016, 11:53:53 CET schrieb Vinson Lee: > > > > > Hi, Tobias. > > > > > > I actually need this when using LLVM from a build tree and the header > > > files are not found. LLVM_INCLUDEDIR is <sourcedir>/include and the > > > header files are in <builddir>/include. > > > > > > Cheers, > > > Vinson > > > > > > On Sat, Dec 3, 2016 at 4:55 AM, Tobias Droste <[email protected]> wrote: > > > > Hi Vinson, > > > > > > > > this should not be needed and is obsolete as soon as this gets merged > > > > (Emil?): > > > > > > > > https://lists.freedesktop.org/archives/mesa-dev/2016-November/135840.html > > > > > > > > Did you actually need this or were you just seeing that the code is > > > > wrong? > > > > Because LLVM >=3.1 has the header files and this code should never > > > > execute! > > > > > > > > Tobias > > > > > > > > Am Samstag, 3. Dezember 2016, 01:20:42 CET schrieb Vinson Lee: > > > > > HAVE_LLVM variable included the patch version if the LLVM version had > > > > > a > > > > > patch version. > > > > > > > > > > For LLVM version '4.0.0', HAVE_LLVM would be '0x0400.0'. > > > > > > > > > > Fixes: 45574ab2e92f ("configure.ac: better detection of LLVM version") > > > > > Signed-off-by: Vinson Lee <[email protected]> > > > > > --- > > > > > > > > > > configure.ac | 2 +- > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > diff --git a/configure.ac b/configure.ac > > > > > index f62bc61e5025..3b8b32485ae1 100644 > > > > > --- a/configure.ac > > > > > +++ b/configure.ac > > > > > @@ -2195,7 +2195,7 @@ if test "x$enable_gallium_llvm" = xyes || test > > > > > "x$HAVE_RADEON_VULKAN" = xyes; th if test -n "${LLVM_VERSION_MAJOR}"; > > > > > then > > > > > > > > > > LLVM_VERSION_INT="${LLVM_VERSION_MAJOR}0${LLVM_VERSION_MINOR > > > > > }" > > > > > > > > > > else > > > > > > > > > > - LLVM_VERSION_INT=`echo $LLVM_VERSION | sed -e > > > > > 's/\([[0-9]]\)\.\([[0-9]]\)/\10\2/g'` + LLVM_VERSION_INT=`echo > > > > > $LLVM_VERSION | sed -e 's/\([[0-9]]\)\.\([[0-9]]\).*/\10\2/g'` fi > > > > > > > > > > LLVM_REQUIRED_VERSION_MAJOR="3" > > _______________________________________________ > mesa-dev mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev -- Jan Vesely <[email protected]>
signature.asc
Description: This is a digitally signed message part
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
