[llvm-commits] [llvm] r126426 - in /llvm/trunk/tools/lto: LTOCodeGenerator.cpp LTOCodeGenerator.h lto.cpp

[Owen Anderson]

On Feb 25, 2011, at 11:09 AM, Rafael Avila de Espindola wrote:

> On 11-02-25 01:35 PM, Owen Anderson wrote:
>> Should this apply to all targets, or only the same ones for which -integrated-as is the default?
> 
> Do you have a target for which you need LTO but MC is not working yet? This patch was intended as a stepping stone for requiring MC for finding definitions and uses in global asm, so if there is still a target with no MC that needs LTO, it is probably better to just revert this for now :-(

There are shipping LLVM compilers with LTO support that target ARM, but ARM MC is not 100% functional yet.

That said, this does make sense.  My point was just that it should be enabled by default on a per-target basis, just like clang's -integrated-as is.

--Owen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110225/4abae506/attachment.html>