[PowerPC, JIT] PPCJITInfo.cpp "This is not a 32bit PowerPC"

Hal Finkel hfinkel at anl.gov
Tue Jul 23 18:53:29 PDT 2013


----- Original Message -----
> Patch attached for real this time.

LGTM. Thanks!

 -Hal

> 
> >>  	A bunch of llvm tests in Analysis/Profiling and ExecutionEngine
> >>  started failing for me on powerpc-darwin8 last week with the
> >>  message:
> >>
> >>  This is not a 32bit PowerPC, you can't execute this!
> >
> > I meant to add a link to a failing log:
> > http://www.csl.cornell.edu/~fang/sw/llvm/logs/llvm-r186533-O2-powerpc-darwin8-g++-4.0.1-debug-check.log
> >
> >>  	It looks like r186252 modified the conditionals that surround
> >>  	the
> >>  callback function.  On darwinX, the system compiler (gcc-4.0.1)
> >>  defines
> >>  either __ppc__ or __ppc64__ for PowerPC.
> >>  	The attached patch accounts for these preprocessor definitions
> >>  	and
> >>  allows those recently failing tests to pass for me.
> >>
> >>  Ok to commit?
> >
> >
> 
> --
> David Fang
> http://www.csl.cornell.edu/~fang/
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory


More information about the llvm-commits mailing list