<div>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.</div><div><br></div><div>It'd be great if the apple linker folks could also comment on this.</div>

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

<div> {</div><div>+    std::vector<const char*>    codegenOptions;</div><div><br></div><div>Extra spaces</div><div><br></div><div class="gmail_quote">On 9 August 2010 11:05, Rafael Espindola <span dir="ltr"><<a href="mailto:espindola@google.com">espindola@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">The attached patch adds -mtriple to LTO. It also moves setting the<br>
command line options earlier and moves _codegenOptions to the only<br>
method where it is used.<br>
<br>
Is it OK?<br>
<br>
Cheers,<br>
<font color="#888888">--<br>
Rafael Ávila de Espíndola<br>
</font></blockquote></div><br>