[llvm-commits] PATCH: pruning llvm w/ config flags --enable-target-oses=os1, os2 similar to --enable-target=arch1, arch2.

[Eric Christopher]

On Aug 29, 2011, at 4:58 PM, Jan Voung wrote:

> Ping.
> 
> - Is this approach acceptable? 
>   - I guess it is hard to test if this has any effect / depends on the compiler.

I'm not really wild about this approach, no. I'd prefer separating out the targets from each other and then conditionally compiling in support rather than relying on DCE and LTO to decide whether or not the code should be there. I'm open to other ideas/discussion on it though.

> - Is trimming the size of the llvm binaries (via config flags) useful to others?
>   - Other examples:
>     - make "include/llvm/Intrinsics.td" not #include intrinsincs for targets that were not enabled through "--enable-target"
>     - config flags to avoid building unused register allocators, other passes

This is quite useful in general, I'm just not a fan of the current method.

The cleanups that you have are definitely useful patches and I hope you'll submit those separately.

Thanks!

-eric