[llvm-commits] [PATCH, RFC] Medium code model support for 64-bit PowerPC
Hal Finkel
hfinkel at anl.gov
Tue Nov 27 10:03:47 PST 2012
----- Original Message -----
> From: "Will Schmidt" <will_schmidt at vnet.ibm.com>
> To: "Bill Schmidt" <wschmidt at linux.vnet.ibm.com>
> Cc: llvm-commits at cs.uiuc.edu
> Sent: Tuesday, November 27, 2012 10:36:30 AM
> Subject: Re: [llvm-commits] [PATCH, RFC] Medium code model support for 64-bit PowerPC
>
> On Mon, 2012-11-26 at 07:25 -0600, Bill Schmidt wrote:
> > Ping... Any further work needed before I commit this?
>
> I read through the latest patch, with your Uli inspired changes, and
> it
> looks good to me.
Agreed. Uli, if you're now happy with this then so am I.
-Hal
>
> I'll note that test-wise, this is quite thorough, with coverage for
> loading (and storing) of the assorted variable types, as well as
> verifying the relocation tags for ; tentatively defined variable,
> external variable, function address, static variable scoped to
> function,
> file-scope static variable, constant pool.
>
> Cosmetically ... is the value 0x3F4FD4920B498CF0 used in the
> testcases
> special in any way?
>
> Thanks,
> -Will
>
>
>
> > Thanks,
> > Bill
> >
> > On Fri, 2012-11-16 at 16:23 -0600, Bill Schmidt wrote:
> > > Here's a new revision of the patch. The changes are:
> > >
> > > (1) Add support for direct object file writing:
> > > - New relocation types in ELF.h.
> > > - New fixup kinds in PPCFixupKinds.h
> > > - Handle new fixups in PPCMCCodeEmitter.cpp, PPCAsmBackend.cpp,
> > > and
> > > PPCELFObjectWriter.cpp.
> > > - Add encoder method for LDrs in PPCInstr64Bit.td.
> > >
> > > (2) Small cleanups per private review from Roman Divacky:
> > > - Use GValue rather than GVal throughout, to more easily
> > > distinguish
> > > from GVar.
> > > - Use isa in preference to dyn_cast where appropriate in
> > > PPCISelDAGToDAG.cpp.
> > >
> > > I will be on vacation for the next week, but will occasionally
> > > monitor
> > > the list for any reviews. Thanks!
> > >
> > > Bill
> > >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> >
>
>
> _______________________________________________
> 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
More information about the llvm-commits
mailing list