<div dir="ltr">Hi John,<div><br></div><div>first, I'd be eternally thankful if you submitted patches to Tooling via phabricator (<a href="http://llvm.org/docs/Phabricator.html">http://llvm.org/docs/Phabricator.html</a>) :)</div>
<div><br></div><div>Second, I'm not yet convinced this patch is the right direction. I'm trying to understand what you're doing:</div><div>1. when running a tool over code, I'd assume you don't need to change the include file arguments</div>
<div>2. after run modularize, if I understand you correctly, the header search paths are now not correct any more - this looks to me like it needs build system support, rather than trying to work around this with the include paths</div>
<div><br></div><div>Cheers,</div><div>/Manuel</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 12, 2013 at 6:16 AM, Thompson, John <span dir="ltr"><<a href="mailto:John_Thompson@playstation.sony.com" target="_blank">John_Thompson@playstation.sony.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class="MsoNormal">This patch adds an input file name parameter to ArgumentAdjuster’s Adjust function in Tooling, in order to allow support of changes to a compile’s arguments based on the file name.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">I’m in a testing phase for a new feature for modularize which allows you to add dependencies to the header files in the input header list.  In using modularize for a real-world set of headers, I found that some headers needed other headers
 to be included first, as opposed to including them themselves or relying on a master header to include the subset of headers in the right order.  With this patch, I can create an ArgumentAdjuster derivation that’s given a map with the dependencies keyed on
 the file name, and have the Adjust function look up the dependencies using the input file name and then add -include options to first include the depended-on headers.  This seemed easier than other changes or derivations that would need to be done.<u></u><u></u></p>

<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Thanks.<span class="HOEnZb"><font color="#888888"><u></u><u></u></font></span></p><span class="HOEnZb"><font color="#888888">
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">-John<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
</font></span></div>
</div>

<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>