[llvm-commits] [llvm] r115391 - in /llvm/trunk: CMakeLists.txt lib/Target/MSP430/InstPrinter/CMakeLists.txt

Óscar Fuentes ofv at wanadoo.es
Fri Oct 1 17:50:27 PDT 2010


Jim Grosbach <grosbach at apple.com> writes:

> Update CMake files for recent AsmPrinter->InstPrinter changes. Can someone who
> is more familiar with CMake please review?

[snip]

>        "${LLVM_ENUM_ASM_PRINTERS}LLVM_ASM_PRINTER(${t})\n")
> +  if( EXISTS ${LLVM_MAIN_SRC_DIR}/lib/Target/${t}/InstPrinter/CMakeLists.txt )
> +    add_subdirectory(lib/Target/${t}/InstPrinter)
> +    set(LLVM_ENUM_ASM_PRINTERS 
> +      "${LLVM_ENUM_ASM_PRINTERS}LLVM_ASM_PRINTER(${t})\n")

As long as there is a good reason for placing that library on
MSP430/InstPrinter directory and keep calling it LLVMMSP430AsmPrinter,
it's okay.




More information about the llvm-commits mailing list