All right, I'll try.<div><br></div><div>-Alexei<br><br><div class="gmail_quote">On Tue, Apr 21, 2009 at 12:06 AM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div style="word-wrap:break-word"><div><div></div><div class="h5"><br><div><div>On Apr 20, 2009, at 4:51 PM, Alexei Svitkine wrote:</div>

<br><blockquote type="cite">Hi,<div><br></div><div>This is a patch for <a href="http://llvm.org/bugs/attachment.cgi?bugid=3979" target="_blank">http://llvm.org/bugs/attachment.cgi?bugid=3979</a>.</div><div><br></div><div>

It moves InitializePreprocessor() functionality to its own file under lib/Frontend from clang-cc.cpp.</div> <div><br></div><div>This allows users of the clang libraries to properly initialize all the expected macros without copying and pasting 500 lines of code from clang-cc.cpp.</div>

<div><br></div><div>Not included are changes to Makefiles and such to properly add the new InitializePreprocessor.cpp file to the build system, since I do not know what is needed there.</div> <div></div></blockquote></div>

<br></div></div><div>Hi Alexei,</div><div><br></div><div>This looks generally fine, except that we really prefer libfrontend to not use cl::opt at all.  We want all the command line option processing stuff to be in clang-cc, not in any lib directories.  Can you refactor the code to populate a class from the command line options and pass that in instead?</div>

<div><br></div><div>Thanks,</div><div><br></div><font color="#888888"><div>-Chris</div></font></div></blockquote></div><br></div>