On Mon, Jul 30, 2012 at 5:46 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank" class="cremed">aaron@aaronballman.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"><div class="im">On Mon, Jul 30, 2012 at 8:31 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com" class="cremed">chandlerc@google.com</a>> wrote:<br>

> Why different code from the phases::Compile code right below that handles<br>
> syntax only? I would expect these to be essentially identical.<br>
<br>
</div>I will admit I'm not too familiar with the driver architecture, but<br>
since this is applying to a header file, the phase comes in as<br>
precompile and not compile.  So I figured the special case was what<br>
made sense, instead of moving C headers to compile phases.<br></blockquote><div><br></div><div>Sorry, all I meant was to structure the code the same way. It would still be in the Precompile phase...</div><div><br></div><div>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
> Your test doesn't need '-OPTIONS1' or any suffix -- you only have one<br>
> compilation.<br>
><br>
> Also, make sure your subversion client submits that new test file with<br>
> native line endings rather than hard coded MS line endings.<br>
<br>
</div>Fair points.  If the fix is acceptable, I'll correct those on commit.<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div>