LGTM Thanks! Ruiling
> -----Original Message----- > From: Beignet [mailto:[email protected]] On Behalf Of > Yang Rong > Sent: Monday, August 1, 2016 2:48 PM > To: [email protected] > Cc: Yang, Rong R <[email protected]> > Subject: [Beignet] [PATCH 1/2] FindLLVM: allow LLVM/Clang 3.8 and reorder > the llvm-config priority. > > Beignet support 3 released llvm version, now is 3.6, 3.7, 3.8. For > LLVM 3.4, 3.5, beignet may still support them, but full test don't cover > them. > > Signed-off-by: Yang Rong <[email protected]> > --- > CMake/FindLLVM.cmake | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/CMake/FindLLVM.cmake b/CMake/FindLLVM.cmake > index 83c8934..6129909 100644 > --- a/CMake/FindLLVM.cmake > +++ b/CMake/FindLLVM.cmake > @@ -8,12 +8,12 @@ > # LLVM_FOUND - True if llvm found. > if (LLVM_INSTALL_DIR) > find_program(LLVM_CONFIG_EXECUTABLE > - NAMES llvm-config-35 llvm-config-3.5 llvm-config-36 > llvm-config-3.6 > llvm-config-37 llvm-config-3.7 llvm-config-33 llvm-config-3.3 llvm-config-34 > llvm-config-3.4 llvm-config > + NAMES llvm-config-37 llvm-config-3.7 llvm-config-36 > llvm-config-3.6 > llvm-config-38 llvm-config-3.8 llvm-config llvm-config-35 llvm-config-3.5 > llvm- > config-34 llvm-config-3.4 > DOC "llvm-config executable" > PATHS ${LLVM_INSTALL_DIR} NO_DEFAULT_PATH) > else (LLVM_INSTALL_DIR) > find_program(LLVM_CONFIG_EXECUTABLE > - NAMES llvm-config-35 llvm-config-3.5 llvm-config-36 > llvm-config-3.6 > llvm-config-37 llvm-config-3.7 llvm-config-33 llvm-config-3.3 llvm-config-34 > llvm-config-3.4 llvm-config > + NAMES llvm-config-37 llvm-config-3.7 llvm-config-36 > llvm-config-3.6 > llvm-config-38 llvm-config-3.8 llvm-config llvm-config-35 llvm-config-3.5 > llvm- > config-34 llvm-config-3.4 > DOC "llvm-config executable") > endif (LLVM_INSTALL_DIR) > > -- > 2.1.4 > > _______________________________________________ > Beignet mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/beignet _______________________________________________ Beignet mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/beignet
