I think it would be better to put stuff like this only into the renderer string of drivers which use LLVM. The majority of drivers don't care about the LLVM version.
Marek On Mon, May 23, 2016 at 9:21 AM, Giuseppe Bilotta <[email protected]> wrote: > Code generation (kernel compilation) may sometimes hit LLVM-specific > bugs. Adding the used LLVM version to the version string may make bug > triaging easier. (This was inspired by a similar patch recently > proposed for pocl.) > > Signed-off-by: Giuseppe Bilotta <[email protected]> > --- > configure.ac | 3 ++- > src/gallium/state_trackers/clover/Makefile.am | 1 + > src/mesa/main/version_string.h | 2 +- > 3 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/configure.ac b/configure.ac > index 6eee2bc..400dd9e 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -2122,7 +2122,7 @@ if test "x$enable_gallium_llvm" = xyes; then > LLVM_COMPONENTS="${LLVM_COMPONENTS} all-targets ipo linker > instrumentation" > LLVM_COMPONENTS="${LLVM_COMPONENTS} irreader option objcarcopts > profiledata" > fi > - DEFINES="${DEFINES} -DHAVE_LLVM=0x0$LLVM_VERSION_INT > -DMESA_LLVM_VERSION_PATCH=$LLVM_VERSION_PATCH" > + DEFINES="${DEFINES} -DHAVE_LLVM=0x0$LLVM_VERSION_INT > -DMESA_LLVM_VERSION_PATCH=$LLVM_VERSION_PATCH > '-DMESA_LLVM_VERSION_STRING=\"$LLVM_VERSION_MAJOR.$LLVM_VERSION_MINOR.$LLVM_VERSION_PATCH\"'" > MESA_LLVM=1 > > dnl Check for Clang internal headers > @@ -2137,6 +2137,7 @@ if test "x$enable_gallium_llvm" = xyes; then > else > MESA_LLVM=0 > LLVM_VERSION_INT=0 > + DEFINES="${DEFINES} '-DMESA_LLVM_VERSION_STRING=\"\"'" > fi > else > MESA_LLVM=0 > diff --git a/src/gallium/state_trackers/clover/Makefile.am > b/src/gallium/state_trackers/clover/Makefile.am > index 4c9d7d9..f6f3d7f 100644 > --- a/src/gallium/state_trackers/clover/Makefile.am > +++ b/src/gallium/state_trackers/clover/Makefile.am > @@ -50,6 +50,7 @@ libclllvm_la_SOURCES = $(LLVM_SOURCES) > > libclover_la_CXXFLAGS = \ > -std=c++11 \ > + $(DEFINES) \ > $(VISIBILITY_CXXFLAGS) > > libclover_la_LIBADD = \ > diff --git a/src/mesa/main/version_string.h b/src/mesa/main/version_string.h > index 970cc8b..b175293 100644 > --- a/src/mesa/main/version_string.h > +++ b/src/mesa/main/version_string.h > @@ -30,5 +30,5 @@ > #define MESA_GIT_SHA1_SUFFIX "" > #endif > > -#define MESA_VERSION_SUFFIX " Mesa " PACKAGE_VERSION MESA_GIT_SHA1_SUFFIX > +#define MESA_VERSION_SUFFIX " Mesa " PACKAGE_VERSION MESA_GIT_SHA1_SUFFIX > MESA_LLVM_VERSION_STRING > > -- > 2.8.1.372.g9612035 > > _______________________________________________ > mesa-dev mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
