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

Benjamin Kramer benny.kra at gmail.com
Tue Sep 18 04:34:28 PDT 2012


On 12.09.2012, at 19:08, Roman Divacky <rdivacky at FreeBSD.org> wrote:

> 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.

I don't see another way to get the subtarget into the AsmBackend, patch LGTM.

- Ben



More information about the llvm-commits mailing list