[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-mc/

[John McCall]

On Sep 18, 2012, at 9:08 AM, Roman Divacky wrote:
> 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.

It appears you need to update clang as well:

llvm[2]: Compiling cc1as_main.cpp for Debug+Asserts build
/Volumes/Data/llvm/tools/clang/tools/driver/cc1as_main.cpp:332:54: error: too
      few arguments to function call, expected 2, have 1
      MAB = TheTarget->createMCAsmBackend(Opts.Triple);
            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~            ^
/Volumes/Data/llvm/include/llvm/Support/TargetRegistry.h:357:5: note: 
      'createMCAsmBackend' declared here
    MCAsmBackend *createMCAsmBackend(StringRef Triple, StringRef CPU) const {
    ^
/Volumes/Data/llvm/tools/clang/tools/driver/cc1as_main.cpp:346:66: error: too
      few arguments to function call, expected 2, have 1
    MCAsmBackend *MAB = TheTarget->createMCAsmBackend(Opts.Triple);
                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~            ^
/Volumes/Data/llvm/include/llvm/Support/TargetRegistry.h:357:5: note: 
      'createMCAsmBackend' declared here
    MCAsmBackend *createMCAsmBackend(StringRef Triple, StringRef CPU) const {
    ^
2 errors generated.

John.