<div style="font-family: arial, helvetica, sans-serif"><font size="2"><div class="gmail_quote">On Thu, Jun 14, 2012 at 1:05 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thu, Jun 14, 2012 at 11:08 AM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>

> The driver support for rewrite includes LGTM, commit away.<br>
<br>
</div>r158463<br>
<div class="im"><br>
> Regarding the crash reporting, that looks too easy! ;]<br>
<br>
</div>It does, doesn't it...<br>
<div class="im"><br>
> Does anything need to be changed in the reproduction script to make it use the include-rewritten<br>
> file instead of a fully preprocessed file?<br>
<br>
</div>Ah, I hadn't actually tried the repro script file - I tend to just run<br>
the crashers myself (but of course then I don't get the extra flags,<br>
etc, from the repro).<br>
<br>
The script is generated based on the file name reported as the output<br>
file by the compilation command, as I understand/read it - so it's not<br>
referring to a non-existent file (& the file name remains the same in<br>
any case).<br>
<div class="im"><br>
>  Do we need to change the filename?<br></div></blockquote><div><br></div><div>I should have said filename *extension*. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">It seems to work fine (reckon it'd be appropriate to have a test case</div>
for this that actually executes the crash report script file? that<br>
seems a little nasty, but with some value). The -cc1 command includes<br>
an explicit -x so I don't think there's any file extension based<br>
language detection (so it doesn't try to skip preprocessing, for<br>
example).</blockquote><div><br></div><div>Wild. I was thinking the '.ii' extension might cause it to do the wrong thing. I think we should still fix the extension to be the same as the original file.</div></div></font></div>