[Beignet] [PATCH V2] fix issue when build against llvm3.3

Song, Ruiling ruiling.song at intel.com
Thu Aug 13 01:20:44 PDT 2015


LGTM

> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> Guo Yejun
> Sent: Thursday, August 13, 2015 3:35 AM
> To: beignet at lists.freedesktop.org
> Cc: Guo, Yejun
> Subject: [Beignet] [PATCH V2] fix issue when build against llvm3.3
> 
> llvm 3.3 has a different constructure of llvm::raw_fd_ostream
> 
> V2: refine the code
> Signed-off-by: Guo Yejun <yejun.guo at intel.com>
> ---
>  backend/src/backend/program.cpp | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/backend/src/backend/program.cpp
> b/backend/src/backend/program.cpp index 9caf1ac..4c001f4 100644
> --- a/backend/src/backend/program.cpp
> +++ b/backend/src/backend/program.cpp
> @@ -817,7 +817,13 @@ namespace gbe {
>        if (!dumpLLVMFileName.empty()) {
>          std::string err;
>          llvm::raw_fd_ostream ostream (dumpLLVMFileName.c_str(),
> -                                      err, llvm::sys::fs::F_RW);
> +                                      err,
> +        #if LLVM_VERSION_MINOR == 3
> +                                      0
> +        #else
> +                                      llvm::sys::fs::F_RW
> +        #endif
> +                                      );
>          if (err.empty()) {
>            out_module->print(ostream, 0);
>          } //Otherwise, you'll have to make do without the dump.
> --
> 1.9.1
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list