Re: [Mesa-dev] [PATCH 4/4] swr: Fix include for createPromoteMemoryToRegisterPass
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 wrote: > > On Sun, Apr 1, 2018 at 1:10 AM, Mike Lothian wrote: >> Include llvm/Transforms/Utils.h with the newest LLVM 7 >> >> Signed-of-by: Mike Lothian >> --- >> 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 >> +#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 ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Re: [Mesa-dev] [PATCH 4/4] swr: Fix include for createPromoteMemoryToRegisterPass
On Sun, Apr 1, 2018 at 1:10 AM, Mike Lothian wrote: > Include llvm/Transforms/Utils.h with the newest LLVM 7 > > Signed-of-by: Mike Lothian > --- > 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 > +#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 ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
[Mesa-dev] [PATCH 4/4] swr: Fix include for createPromoteMemoryToRegisterPass
Include llvm/Transforms/Utils.h with the newest LLVM 7 Signed-of-by: Mike Lothian --- 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 +#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