[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