<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">This patch LGTM, but I think going
forward we should stop using llc as a way to detect whether
assertions are enabled, the output from llc which is used to
figure out if assertions are enabled or not, should be contained
in lld as well.<br>
<br>
Thoughts ?<br>
<br>
On 2/23/2014 5:29 AM, Nico Rieck wrote:<br>
</div>
<blockquote cite="mid:5309DB8E.30409@gmail.com" type="cite">
<pre wrap="">Reading from stdout a second time yields an empty string so all tests
requiring assertions are never run.
-Nico
</pre>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
llvm-commits mailing list
<a class="moz-txt-link-abbreviated" href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>
<a class="moz-txt-link-freetext" href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a>
</pre>
</blockquote>
<br>
<br>
<pre class="moz-signature" cols="72">--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation</pre>
</body>
</html>