<div>Also, I missed this failing test in my last message:</div>
<div> </div>
<div>1>FAIL: Clang::CodeGenCXX/new.cpp (538 of 1739)<br>1>******************** TEST 'Clang::CodeGenCXX/new.cpp' FAILED ********************<br>1>Script:<br>1>--<br>1>C:/Tools/llvm/bin/Debug/clang-cc.exe C:\Tools\llvm\tools\clang\test\CodeGenCXX\new.cpp -emit-llvm -o - | FileCheck C:\Tools\llvm\tools\clang\test\CodeGenCXX\new.cpp<br>
1>--<br>1>Exit Code: 1<br>1>Command Output (stdout):<br>1>--<br>1>Command 0: "C:/Tools/llvm/bin/Debug/clang-cc.exe" "C:\Tools\llvm\tools\clang\test\CodeGenCXX\new.cpp" "-emit-llvm" "-o" "-"<br>
1>Command 0 Result: 1<br>1>Command 0 Output:<br>1>Command 0 Stderr:<br>1>C:\Tools\llvm\tools\clang\test\CodeGenCXX\new.cpp:8:7: error: 'operator new' takes type size_t ('unsigned int') as first parameter<br>
1>void* operator new(unsigned long, void*) throw();<br>1>      ^<br>1>1 diagnostic generated.<br>1>Command 1: "FileCheck" "C:\Tools\llvm\tools\clang\test\CodeGenCXX\new.cpp"<br>1>Command 1 Result: 1<br>
1>Command 1 Output:<br>1>Command 1 Stderr:<br>1>C:\Tools\llvm\tools\clang\test\CodeGenCXX\new.cpp:35:32: error: expected string not found in input<br>1> // CHECK: call void @_ZN1TC1Ev<br>1>                               ^<br>
1><stdin>:1:1: note: scanning from here<br>1>^<br>1>--<br>1>Command Output (stderr):<br>1>--<br>1>--<br><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>