[PATCH] [lld] Fix feature detection in lld's lit.cfg

Shankar Easwaran shankare at codeaurora.org
Sun Feb 23 21:39:50 PST 2014


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.

Thoughts ?

On 2/23/2014 5:29 AM, Nico Rieck wrote:
> Reading from stdout a second time yields an empty string so all tests
> requiring assertions are never run.
>
> -Nico
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits


-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140223/e15a668d/attachment.html>


More information about the llvm-commits mailing list