<br><br><div class="gmail_quote">On Fri, Aug 17, 2012 at 11:41 AM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Er, my suggestion was to make these *static* variables, yes, even though they are in a header file, and then to document thoroughly that this header file is *not* for use within a library, but for use directly within the file defining main.</blockquote>
<div>As Manuel already noted, there was an idea to make definition of these variables explicit via a macro. And I didn't find a better way to implement that. If you think that a special "include-once-per-binary" header is still better, I can switch to this approach. </div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>The current patch is different from that, can you describe how? In general the comments weren't clear enough for me to be confident in how you expect this file to be used...</div>
</blockquote><div>Ok, I moved the usage example to the top and added some comments. Is it better?</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb">
<div class="h5"><div class="gmail_extra">
<br><br><div class="gmail_quote">On Fri, Aug 17, 2012 at 2:26 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

+using namespace llvm;<br>
<br>
Remove from header.<br>
<br>
Appart from that, lgtm, minus Chandler's approval of the general idea.<br>
<div><div><br>
On Thu, Aug 16, 2012 at 7:17 PM, Alexander Kornienko <<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>> wrote:<br>
> On Thu, Aug 16, 2012 at 7:04 PM, Manuel Klimek <<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>> wrote:<br>
>><br>
>> On Thu, Aug 16, 2012 at 5:45 PM, Alexander Kornienko <<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>><br>
>> wrote:<br>
>> > Now it's a bit uglier, but doesn't use CommandLine Library in an<br>
>> > unsupported<br>
>> > way.<br>
>> > Chandler, please take a look if it seems better to you.<br>
>><br>
>> After a short discussion off-list we came to the conclusion that<br>
>> tooling::CommonOptionsParser is a better name for the class, and<br>
>> especially makes the responsibilities clearer...<br>
><br>
> A new patch is attached.<br>
><br>
>><br>
>><br>
>> > BTW, we still have a number of alien options defined in some llvm<br>
>> > libraries<br>
>> > we link with. Should we try to avoid that by de-globalizing cl library<br>
>> > or is<br>
>> > someone working on its replacement/refactoring now?<br>
><br>
><br>
> --<br>
> Best regards,<br>
> Alexander Kornienko<br>
><br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Best regards,</div><div>Alexander Kornienko</div><br>