[PATCH] Review: Add Cmake Options to disable target components (AsmParser, InstPrinter, Disassembler)

Aditya Nandakumar aditya_nandakumar at apple.com
Mon Apr 14 11:26:44 PDT 2014


Hi Tom

I just modified R600 target as an example for demonstration to go with the main CMAKE changes. The changes required for the other targets would be similar.

Aditya
On Apr 13, 2014, at 8:08 PM, Tom Stellard <tom at stellard.net> wrote:

> On Sat, Apr 12, 2014 at 02:56:02PM -0700, Aditya Nandakumar wrote:
>> Hi
>> 
>> I have attached a patch which accepts CMAKE options to disable target specific components. Also attached is a patch for changes required for the R600 target. This can save megabytes in released binaries
>> Eg usage :
>> cmake -DLLVM_TARGETS_TO_BUILD="R600" -G "Ninja" -DCMAKE_BUILD_TYPE=Release ../llvm/ -DLLVM_TARGETS_DISABLE_INST_PRINTER=“R600”
>> llvm-build and llvm-lit has been updated as well.
>> 
>> Additional options to Cmake are 
>> -DLLVM_TARGETS_DISABLE_DISASSEMBLER=“…” and 
>> -DLLVM_TARGETS_DISABLE_PARSER=“…”
>> 
>> I request you to kindly review this change.
>> 
>> 
> 
> Why does R600 need to be changed and the other targets don't?  Is the
> R600 backend doing something wrong?
> 
> -Tom
> 
> 
>> 
>> 
>> Thanks
>> Aditya Nandakumar
> 
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140414/c2b6ef29/attachment.html>


More information about the llvm-commits mailing list