[llvm-commits] [PATCH]: dont emit long nops on AMD Geode

[Roman Divacky]

How do I get it from X86AsmBackend::writeNopData() ? I didnt find
any way...

On Wed, Sep 12, 2012 at 05:34:04PM +0000, Villmow, Micah wrote:
> This information stored in the subtarget already, can't you retrieve it from target.getSubtarget()?
> 
> > -----Original Message-----
> > From: llvm-commits-bounces at cs.uiuc.edu [mailto:llvm-commits-
> > bounces at cs.uiuc.edu] On Behalf Of Roman Divacky
> > Sent: Wednesday, September 12, 2012 10:09 AM
> > To: llvm-commits at cs.uiuc.edu
> > Subject: [llvm-commits] [PATCH]: dont emit long nops on AMD Geode
> > 
> > Hi,
> > 
> > it happens to be so that we unconditionaly emit long nops. Even on CPUs
> > that dont support those. See PR11212 for details.
> > 
> > The attached patch passes CPU option (StringRef) to MCAsmBackend so
> > that X86AsmBackend can use it and not emit long nops when the cpu is
> > "geode".
> > 
> > Thus fixing it. The patch is a little messy because I have to pass the
> > new argument everywhere.
> > 
> > It needs trivial clang adjustment too.
> > 
> > What do you think? Roman