[llvm-commits] [llvm] r164132 - in /llvm/trunk: include/llvm/Support/ lib/CodeGen/ lib/Target/ARM/MCTargetDesc/ lib/Target/MBlaze/MCTargetDesc/ lib/Target/Mips/MCTargetDesc/ lib/Target/PowerPC/MCTargetDesc/ lib/Target/X86/MCTargetDesc/ tools/llvm

[NAKAMURA Takumi]

2012/9/19 Roman Divacky <rdivacky at freebsd.org>:
> Author: rdivacky
> Date: Tue Sep 18 11:08:49 2012
> New Revision: 164132
>
> URL: http://llvm.org/viewvc/llvm-project?rev=164132&view=rev
> Log:
> When creating MCAsmBackend pass the CPU string as well. In X86AsmBackend
> store this and use it to not emit long nops when the CPU is geode which
> doesnt support them.
>
> Fixes PR11212.

> Modified: llvm/trunk/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp?rev=164132&r1=164131&r2=164132&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp (original)
> +++ llvm/trunk/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp Tue Sep 18 11:08:49 2012
> @@ -66,9 +66,10 @@
>  };
>
>  class X86AsmBackend : public MCAsmBackend {
> +  StringRef CPU;

Would it be safe with StringRef here? I suggest SmallString might be better.

...Takumi