Please use LLVM defines consistent with the surrounding code. Surrounding code uses LLVM_VERSION_MAJOR / LLVM_VERSION_MINOR.
This code should then be: #if LLVM_VERSION_MAJOR >= 7 … #endif Thank you! George > On Apr 1, 2018, at 3:05 PM, Vinson Lee <v...@freedesktop.org> wrote: > > On Sun, Apr 1, 2018 at 1:10 AM, Mike Lothian <m...@fireburn.co.uk> wrote: >> Include llvm/Transforms/Utils.h with the newest LLVM 7 >> >> Signed-of-by: Mike Lothian <m...@fireburn.co.uk> >> --- >> src/gallium/drivers/swr/rasterizer/jitter/jit_pch.hpp | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/src/gallium/drivers/swr/rasterizer/jitter/jit_pch.hpp >> b/src/gallium/drivers/swr/rasterizer/jitter/jit_pch.hpp >> index 031bced8a0..d726d4844e 100644 >> --- a/src/gallium/drivers/swr/rasterizer/jitter/jit_pch.hpp >> +++ b/src/gallium/drivers/swr/rasterizer/jitter/jit_pch.hpp >> @@ -67,6 +67,9 @@ using PassManager = llvm::legacy::PassManager; >> #include "llvm/Support/DynamicLibrary.h" >> #include "llvm/Transforms/IPO.h" >> #include "llvm/Transforms/Scalar.h" >> +#if HAVE_LLVM >= 0x0700 >> +#include <llvm/Transforms/Utils.h> >> +#endif >> #include "llvm/Support/Host.h" >> #include "llvm/Support/DynamicLibrary.h" >> >> -- >> 2.16.3 >> >> _______________________________________________ >> mesa-dev mailing list >> mesa-dev@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/mesa-dev > > > The added include uses <> delimiters whereas the other includes use "" > delimiters. > > Tested-by: Vinson Lee <v...@freedesktop.org> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
