[llvm-commits] [llvm] r135760 - in /llvm/trunk: cmake/modules/LLVMLibDeps.cmake lib/Support/CMakeLists.txt lib/Support/TargetRegistry.cpp lib/Target/CMakeLists.txt lib/Target/TargetRegistry.cpp

Chandler Carruth chandlerc at gmail.com
Fri Aug 5 22:40:09 PDT 2011


On Mon, Aug 1, 2011 at 3:40 PM, Evan Cheng <evan.cheng at apple.com> wrote:

> I am not comfortable with a few weeks. Is it not possible to just move
> TargetRegistry.cpp back to Support?


Putting this into Support violates a lot of layering principles. I care a
lot about layering and other invariants, thats how we keep building and
using LLVM effectively.

Anyways, I understand the need for an expedient solution. I think I have a
reasonable compromise -- we could move the registry into MC. The only
targets that don't actually need MC are the CBackend and the CppBackend.
That seems likely to be the ongoing trend. It would allow MC clients to not
pull in any part of Target.

Best of all, that's where the other two pieces of the registry already live:
the Reloc and CodeModel classes.

I'm trying this out now to ensure it actually works, and does in fact shrink
llvm-mc back down to size.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110805/ad75cd0c/attachment.html>


More information about the llvm-commits mailing list