+ariel who wrote the original patch<br><br><div class="gmail_quote">On Tue Sep 02 2014 at 1:41:37 AM Justin Bogner <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Steffen Prohaska pointed out on cfe-dev that a condition is exactly<br>
backwards in an error_code check in Replaceent::<u></u>setFromSourceLocation.<br>
<br>
The obvious fix of reversing the condition causes about 30 tests to<br>
fail, so it looks to me like we actually rely on the path *not* being<br>
made absolute. Furthermore, this has been broken like this for at least<br>
a year, and before that it didn't try making the path absolute.<br>
<br>
The attached patch simplifies the code such that it doesn't bother<br>
calling make_absolute. This preserves the current behaviour, passes all<br>
tests, and avoids the case where make_absolute fails and we use the<br>
result anyway.<br></blockquote><div><br></div><div>The original patch where this was introduced is:</div><div><a href="http://reviews.llvm.org/rL191776">http://reviews.llvm.org/rL191776</a><br></div><div>Apparently the test in test/Tooling/clang-check-rel-path.cpp is incorrect (?)</div>
<div>Ariel, did you actually see this test break at least once?</div><div><br></div><div>Cheers,<br></div><div>/Manuel</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
______________________________<u></u>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/cfe-commits</a><br>
</blockquote></div>