[llvm-commits] [PATCH v2] Disable old-JIT and enable MCJIT tests on PowerPC

[Hal Finkel]

----- Original Message -----
> From: "Andrew Kaylor" <andrew.kaylor at intel.com>
> To: "Hal Finkel" <hfinkel at anl.gov>, "Ulrich Weigand" <Ulrich.Weigand at de.ibm.com>
> Cc: llvm-commits at cs.uiuc.edu
> Sent: Monday, November 19, 2012 2:08:08 PM
> Subject: RE: [llvm-commits] [PATCH v2] Disable old-JIT and enable MCJIT	tests on PowerPC
> 
> Probably so.  I believe the PPC support for MCJIT was in well before
> the branch, so this patch will make the tests report reasonable
> results.

Okay, then given that you are the code owner for MCJIT, I think that you should explicitly approve.

 -Hal

> 
> -Andy
> 
> -----Original Message-----
> From: Hal Finkel [mailto:hfinkel at anl.gov]
> Sent: Monday, November 19, 2012 12:04 PM
> To: Ulrich Weigand
> Cc: llvm-commits at cs.uiuc.edu; Kaylor, Andrew
> Subject: Re: [llvm-commits] [PATCH v2] Disable old-JIT and enable
> MCJIT tests on PowerPC
> 
> Andy, Ulrich,
> 
> Should this be merged into 3.2?
> 
>  -Hal
> 
> ----- Original Message -----
> > From: "Ulrich Weigand" <Ulrich.Weigand at de.ibm.com>
> > To: "Andrew Kaylor" <andrew.kaylor at intel.com>
> > Cc: llvm-commits at cs.uiuc.edu
> > Sent: Monday, November 19, 2012 11:58:36 AM
> > Subject: Re: [llvm-commits] [PATCH v2] Disable old-JIT and enable
> > MCJIT	tests on PowerPC
> > 
> > "Kaylor, Andrew" <andrew.kaylor at intel.com> wrote on 19.11.2012
> > 18:52:00:
> > 
> > > That looks great, and I agree that the directory level skipping
> > > is
> > > better.  There may be some problems with 32-bit PowerPC and the
> > > MCJIT tests, but I don't believe any of the buildbots use that
> > configuration.
> > 
> > Checked in as revision 168316.
> > 
> > Thanks,
> > Ulrich
> > 
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> > 
> 
> --
> Hal Finkel
> Postdoctoral Appointee
> Leadership Computing Facility
> Argonne National Laboratory
> 

-- 
Hal Finkel
Postdoctoral Appointee
Leadership Computing Facility
Argonne National Laboratory