[llvm-commits] [PATCH] Add placeholder for eight-byte mandatory traceback table on PPC64

Hal Finkel hfinkel at anl.gov
Wed Aug 29 10:15:13 PDT 2012


Bill,

Why did you change the wording in the comment from "Eventually we
should add" to "Eventually it would be potentially useful to fill in"?
What's the rationale for the softer wording?

 -Hal

On Wed, 29 Aug 2012 11:53:06 -0500
"William J. Schmidt" <wschmidt at linux.vnet.ibm.com> wrote:

> After offline discussion it became clear we need to reserve space for
> the mandatory traceback fields, though leaving them as zero is
> appropriate for now.  This small patch adds the extra eight bytes and
> updates the associated test case.
> 
> Please commit if OK.
> 
> Thanks!
> Bill
> 



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



More information about the llvm-commits mailing list