[llvm-commits] [PATCH 1/10] PowerPC64 definitions for ELF.h

Kaylor, Andrew andrew.kaylor at intel.com
Thu Sep 20 15:18:02 PDT 2012


Looks good to me.

If you want to add a test case, I think test/Object/objdump-file-header.test is the place to do it.

-Andy

-----Original Message-----
From: llvm-commits-bounces at cs.uiuc.edu [mailto:llvm-commits-bounces at cs.uiuc.edu] On Behalf Of Adhemerval Zanella
Sent: Thursday, September 20, 2012 7:46 AM
To: llvm-commits at cs.uiuc.edu
Subject: Re: [llvm-commits] [PATCH 1/10] PowerPC64 definitions for ELF.h

On 09/17/2012 05:18 PM, Adhemerval Zanella wrote:
> After the inputs from my preiovus batch of patches for PPC64 MCJIT 
> support, I decided to split up them in more patches and also added more tests.
>
> This is the first patch of a set of 10: a trivial one intended to add 
> the
> PowerPC64 definition on ELF.h.
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Ping. Anything blocking this trivial patch in particular? If so, what kind of testcases would be required to do so?


--
Adhemerval Zanella Netto
  Software Engineer
  Linux Technology Center Brazil
  Toolchain / GLIBC on Power Architecture
  azanella at linux.vnet.ibm.com / azanella at br.ibm.com
  +55 61 8642-9890

_______________________________________________
llvm-commits mailing list
llvm-commits at cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list