<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 20, 2015 at 4:56 AM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Author: dexonsmith<br>
Date: Mon Jan 19 20:56:57 2015<br>
New Revision: 226541<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=226541&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=226541&view=rev</a><br>
Log:<br>
IR: Move MDNode clone() methods from ValueMapper to MDNode, NFC<br>
<br>
Now that the clone methods used by `MapMetadata()` don't do any<br>
remapping (and return a temporary), they make more sense as member<br>
functions on `MDNode` (and subclasses).<br>
<br></blockquote><div><br></div><div>This seems to break MSVC (2013) builds for me</div><div><br></div><div><div>FAIL: LLVM-Unit :: IR/IRTests.exe/MDLocationTest.getTemporary (19749 of 20220)</div><div>******************** TEST 'LLVM-Unit :: IR/IRTests.exe/MDLocationTest.getTemporary' FAILED ********************</div><div>Note: Google Test filter = MDLocationTest.getTemporary</div><div>[==========] Running 1 test from 1 test case.</div><div>[----------] Global test environment set-up.</div><div>[----------] 1 test from MDLocationTest</div><div>[ RUN      ] MDLocationTest.getTemporary</div><div>..\unittests\IR\MetadataTest.cpp(560): error: Value of: L->isTemporary()</div><div>  Actual: false</div><div>Expected: true</div><div>[  FAILED  ] MDLocationTest.getTemporary (47 ms)</div><div>[----------] 1 test from MDLocationTest (47 ms total)</div><div><br></div><div>[----------] Global test environment tear-down</div><div>[==========] 1 test from 1 test case ran. (47 ms total)</div><div>[  PASSED  ] 0 tests.</div><div>[  FAILED  ] 1 test, listed below:</div><div>[  FAILED  ] MDLocationTest.getTemporary</div><div><br></div><div> 1 FAILED TEST</div><div><br></div><div>********************</div><div>FAIL: LLVM-Unit :: IR/IRTests.exe/MDNodeTest.isTemporary (19760 of 20220)</div><div>******************** TEST 'LLVM-Unit :: IR/IRTests.exe/MDNodeTest.isTemporary' FAILED ********************</div><div>Note: Google Test filter = MDNodeTest.isTemporary</div><div>[==========] Running 1 test from 1 test case.</div><div>[----------] Global test environment set-up.</div><div>[----------] 1 test from MDNodeTest</div><div>[ RUN      ] MDNodeTest.isTemporary</div><div>..\unittests\IR\MetadataTest.cpp(332): error: Value of: T->isTemporary()</div><div>  Actual: false</div><div>Expected: true</div><div>[  FAILED  ] MDNodeTest.isTemporary (0 ms)</div><div>[----------] 1 test from MDNodeTest (0 ms total)</div><div><br></div><div>[----------] Global test environment tear-down</div><div>[==========] 1 test from 1 test case ran. (0 ms total)</div><div>[  PASSED  ] 0 tests.</div><div>[  FAILED  ] 1 test, listed below:</div><div>[  FAILED  ] MDNodeTest.isTemporary</div><div><br></div><div> 1 FAILED TEST</div><div><br></div><div>********************</div></div><div><div>FAIL: LLVM-Unit :: IR/IRTests.exe/MDNodeTest.replaceWithDistinct (19763 of 20220)</div><div>******************** TEST 'LLVM-Unit :: IR/IRTests.exe/MDNodeTest.replaceWithDistinct' FAILED ********************</div><div>Note: Google Test filter = MDNodeTest.replaceWithDistinct</div><div>[==========] Running 1 test from 1 test case.</div><div>[----------] Global test environment set-up.</div><div>[----------] 1 test from MDNodeTest</div><div>[ RUN      ] MDNodeTest.replaceWithDistinct</div><div>..\unittests\IR\MetadataTest.cpp(493): error: Value of: Temp->isTemporary()</div><div>  Actual: false</div><div>Expected: true</div><div>..\unittests\IR\MetadataTest.cpp(506): error: Value of: Temp->isTemporary()</div><div>  Actual: false</div><div>Expected: true</div><div>[  FAILED  ] MDNodeTest.replaceWithDistinct (0 ms)</div><div>[----------] 1 test from MDNodeTest (0 ms total)</div><div><br></div><div>[----------] Global test environment tear-down</div><div>[==========] 1 test from 1 test case ran. (0 ms total)</div><div>[  PASSED  ] 0 tests.</div><div>[  FAILED  ] 1 test, listed below:</div><div>[  FAILED  ] MDNodeTest.replaceWithDistinct</div><div><br></div><div> 1 FAILED TEST</div><div><br></div><div>********************</div><div>FAIL: LLVM-Unit :: IR/IRTests.exe/MDNodeTest.replaceWithUniqued (19764 of 20220)</div><div>******************** TEST 'LLVM-Unit :: IR/IRTests.exe/MDNodeTest.replaceWithUniqued' FAILED ********************</div><div>Note: Google Test filter = MDNodeTest.replaceWithUniqued</div><div>[==========] Running 1 test from 1 test case.</div><div>[----------] Global test environment set-up.</div><div>[----------] 1 test from MDNodeTest</div><div>[ RUN      ] MDNodeTest.replaceWithUniqued</div><div>..\unittests\IR\MetadataTest.cpp(447): error: Value of: Temp->isTemporary()</div><div>  Actual: false</div><div>Expected: true</div><div>..\unittests\IR\MetadataTest.cpp(459): error: Value of: Temp->isTemporary()</div><div>  Actual: false</div><div>Expected: true</div><div>..\unittests\IR\MetadataTest.cpp(471): error: Value of: Temp->isTemporary()</div><div>  Actual: false</div><div>Expected: true</div><div>[  FAILED  ] MDNodeTest.replaceWithUniqued (0 ms)</div><div>[----------] 1 test from MDNodeTest (0 ms total)</div><div><br></div><div>[----------] Global test environment tear-down</div><div>[==========] 1 test from 1 test case ran. (0 ms total)</div><div>[  PASSED  ] 0 tests.</div><div>[  FAILED  ] 1 test, listed below:</div><div>[  FAILED  ] MDNodeTest.replaceWithUniqued</div><div><br></div><div> 1 FAILED TEST</div><div><br></div><div>********************</div></div></div></div></div>