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

NAKAMURA Takumi geek4civic at gmail.com
Mon Feb 24 00:18:05 PST 2014


Yes, FYI, also llvm-config is available there.

$ bin/llvm-config --assertion-mode
ON

2014-02-24 14:39 GMT+09:00 Shankar Easwaran <shankare at codeaurora.org>:
> 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
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>



More information about the llvm-commits mailing list