[PATCH] [Power] Add a new feature flag for support of the sync instruction

Hal Finkel hfinkel at anl.gov
Thu Sep 11 18:29:22 PDT 2014


----- Original Message -----
> From: "Bill Schmidt" <wschmidt at linux.vnet.ibm.com>
> To: morisset at google.com, jfb at chromium.org, hfinkel at anl.gov, wschmidt at linux.vnet.ibm.com
> Cc: alexr at leftfield.org, llvm-commits at cs.uiuc.edu
> Sent: Thursday, September 11, 2014 6:18:13 PM
> Subject: Re: [PATCH] [Power] Add a new feature flag for support of the sync instruction
> 
> I don't have a preference because I don't know the history and intent
> of these processor values.  I am just raising the question because
> the inconsistency needs to be understood.  Perhaps hfinkel has
> ideas.

I think we can take "ppc" to represent a commodity ppc32 core, and I'd not restrict it to a common subset with early Book E cores. Printing "sync 0" on "ppc", regardless of what is done for Book E generally, is fine.

 -Hal

> 
> http://reviews.llvm.org/D5316
> 
> 
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory



More information about the llvm-commits mailing list