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

[Villmow, Micah]

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