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

Zhigang Gong zhigang.gong at linux.intel.com
Thu Aug 13 18:09:27 PDT 2015


Pushed, thanks.

On Thu, Aug 13, 2015 at 08:20:44AM +0000, Song, Ruiling wrote:
> 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
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list