<div>Here's a patch which fixes the assertion failure in running the tests on Windows, basically matching what is done for the include directive put in the Predefines.</div>
<div> </div>
<div>Should I check it in?</div>
<div> </div>
<div>-John<br><br></div>
<div class="gmail_quote">On Wed, Nov 11, 2009 at 12:31 PM, John Thompson <span dir="ltr"><<a href="mailto:john.thompson.jtsoftware@gmail.com">john.thompson.jtsoftware@gmail.com</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote">
<div>30 tests are failing on Windows, 27 newly since Monday.</div>
<div> </div>
<div>25 of them are due to an assert at line 172 in PCHReader.cpp.</div>
<div> </div>
<div>The problem is that the file path appended to the Predefines has been converted to use double-'\', whereas the path in OriginalFileName used single '/'.</div>
<div> </div>
<div>I'm not sure which side to change.</div>
<div> </div>
<div>The other is in CodeGenCXX/new-operator-phi.cpp:</div>
<div> </div>
<div>1>  Line 4: 'operator new' takes type size_t ('unsigned int') as first parameter<br></div>
<div>I'm not sure what you want to do here, whether to change it to use size_t and to include stddef.h, or put in a platform conditional.<br clear="all"></div>
<div>-John</div>
<div><br>-- <br>John Thompson<br><a href="mailto:John.Thompson.JTSoftware@gmail.com" target="_blank">John.Thompson.JTSoftware@gmail.com</a><br><br></div></blockquote></div><br><br clear="all"><br>-- <br>John Thompson<br>
<a href="mailto:John.Thompson.JTSoftware@gmail.com">John.Thompson.JTSoftware@gmail.com</a><br><br>