[llvm-commits] [patch] Add -mtriple to LTO

[Nick Lewycky]

I don't like how this works. I'd much rather expose getter+setter methods in
the libLTO API and let the non-library program do its own flags processing,
etc.

It'd be great if the apple linker folks could also comment on this.

@@ -405,12 +404,17 @@ bool
LTOCodeGenerator::generateAssemblyCode(formatted_raw_ostream& out,
 /// Optimize merged modules using various IPO passes
 void LTOCodeGenerator::setCodeGenDebugOptions(const char* options)
 {
+    std::vector<const char*>    codegenOptions;

Extra spaces

On 9 August 2010 11:05, Rafael Espindola <espindola at google.com> wrote:

> The attached patch adds -mtriple to LTO. It also moves setting the
> command line options earlier and moves _codegenOptions to the only
> method where it is used.
>
> Is it OK?
>
> Cheers,
> --
> Rafael Ávila de Espíndola
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20100809/6fb0c89d/attachment.html>