[llvm-commits] [llvm] r72700 - in /llvm/trunk/tools/lto: LTOModule.cpp LTOModule.h

[Duncan Sands]

Hi Nick,

> <rdar://problem/6927148> libLTO needs to handle i386 magic objc class symbols
> Parse __OBJC data structures and synthesize magic .objc_ symbols.
> Also, alter mangler so that objc method names are readable.

I appreciate that you're trying to solve a tricky problem, but do we
really want to push this kind of objc junk into LLVM?  Can't it be fixed
in the objc front-end instead, or alternatively by moving this logic
the llvm parts of llvm-gcc, for example?  Also, I don't see how having
this in LTOModule can be correct.  What if someone links .bc files
together by hand using llvm-link?  Surely this should either be in the
generic llvm linker code, or in the code generators?

Ciao,

Duncan.