<div dir="ltr">On Thu, Aug 22, 2013 at 10:35 PM, Kim Gräsman <span dir="ltr"><<a href="mailto:kim.grasman@gmail.com" target="_blank">kim.grasman@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey Manuel,<br>
<div class="im"><br>
On Thu, Aug 22, 2013 at 11:38 AM, Manuel Klimek <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br>
>><br>
>> Ah, and then build two different binaries out of it? One stand-alone<br>
>> tool and one Clang plugin dylib? Or can they be combined into the same<br>
>> binary? That would be my preference, of course, but I can't quite see<br>
>> the big picture for all the details at this point.<br>
><br>
> Yes, you'd have one binary and one dylib (which you'd give to clang as<br>
> plugin during the normal build).<br>
> The cool thing about that solution is that you can run it next to the build<br>
> on every build - you don't need a special tool run.<br>
<br>
</div>Interesting input, I'll give it a try.<br>
<br>
Though I think IWYU was purposefully designed to NOT run together with<br>
a normal build because<br>
<br>
1) you couldn't re-run it without an intervening ``make clean``<br>
2) it would be take at least as long as building the software and if<br>
you're not IWYU-clean, that's not something you want to do<br>
unnecessarily :-)<br>
<br>
I wasn't around when the current strategy was set, but I think it was<br>
directed by these two goals and the absence of the Tooling library at<br>
the time.<br></blockquote><div><br></div><div>I understood it in a way that you wanted to run IWYU as part of a make-build, and that was why you wanted to not run it as a standalone tool with a compilation db. I'm confused now :)</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Sincere thanks for the good information, though, I have more to work from!<br>
<span class="HOEnZb"><font color="#888888"><br>
- Kim<br>
</font></span></blockquote></div><br></div></div>