LGTM, pushed. Beignet support llvm from 3.5 to 3.8 now, I will add llvm3.8 and also remove some legacy version, thanks for your patch.
> -----Original Message----- > From: Beignet [mailto:[email protected]] On Behalf Of > Rebecca N. Palmer > Sent: Monday, April 25, 2016 5:41 > To: [email protected] > Subject: [Beignet] [PATCH] FindLLVM: allow LLVM/Clang 3.7 > > As beignet now works with LLVM/Clang 3.7, accept this version when > searching for llvm-config. > > Signed-off-by: Rebecca Palmer <[email protected]> > > --- 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-33 llvm-config-3.3 llvm-config-34 llvm-config-3.4 llvm-config > + 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 > 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-33 llvm-config-3.3 llvm-config-34 llvm-config-3.4 llvm-config > + 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 > DOC "llvm-config executable") endif (LLVM_INSTALL_DIR) > > > _______________________________________________ > Beignet mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/beignet _______________________________________________ Beignet mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/beignet
