<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Thanks Eric,<div class=""><br class=""></div><div class="">I plan to resubmit only after I'll have a cc1 command line flag to enable/disable this instead of a compile-time flag as it is now. </div><div class="">I'll update the tests to either not rely on the named value or pass the flag to preserve them.</div><div class=""><br class=""></div><div class="">-- </div><div class="">Mehdi</div><div class=""><br class=""></div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Mar 11, 2016, at 5:52 PM, Eric Christopher <<a href="mailto:echristo@gmail.com" class="">echristo@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Hi Mehdi,<div class=""><br class=""></div><div class="">This broke a bunch of clang tests in a release build, a bot that shows the same symptoms is:</div><div class=""><br class=""></div><div class=""><a href="http://lab.llvm.org:8080/green/job/clang-stage2-cmake-RgSan_check/1337/consoleFull#15733454949ba4694-19c4-4d7e-bec5-911270d8a58c" target="_blank" class="">http://lab.llvm.org:8080/green/job/clang-stage2-cmake-RgSan_check/1337/consoleFull#15733454949ba4694-19c4-4d7e-bec5-911270d8a58c</a><br class=""></div><div class=""><br class=""></div><div class="">I've gone ahead and reverted this and the associated clang patches as I'm not quite sure what you'd like to do about the tests that are looking for names for parameters etc.</div><div class=""><br class=""></div><div class="">Thanks!</div><div dir="ltr" class=""><div class=""><br class=""></div><div class="">-eric</div></div><br class=""><div class="gmail_quote"><div dir="ltr" class="">On Fri, Mar 11, 2016 at 2:39 AM Michel Dänzer <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank" class="">llvm-commits@lists.llvm.org</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 11.03.2016 02:12, Mehdi Amini via llvm-commits wrote:<br class="">
> Should be fixed with r263127.<br class="">
<br class="">
It is, thanks!<br class="">
<br class="">
<br class="">
--<br class="">
Earthling Michel Dänzer | <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__www.amd.com&d=BQMFaQ&c=Hw-EJUFt2_D9PK5csBJ29kRV40HqSDXWTLPyZ6W8u84&r=XndYjVJuvcoEtO9BUlAZk8839TPlVRJeJXMNUFEz-qQ&m=X5rL6EBUtrqJECPG3Y_87XZIMxNquhocTP-iTnIY5xk&s=h_YV-TeRjelYwwZxZRP1plpGywojQ9euURI_bkHz9c0&e=" rel="noreferrer" target="_blank" class="">http://www.amd.com</a><br class="">
Libre software enthusiast | Mesa and X developer<br class="">
_______________________________________________<br class="">
llvm-commits mailing list<br class="">
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank" class="">llvm-commits@lists.llvm.org</a><br class="">
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank" class="">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br class="">
</blockquote></div></div>
</div></blockquote></div><br class=""></div></body></html>