<div dir="ltr">Sorry, I was thinking to review the test but didn't.<div><br></div><div>Is this test complete? It does invoke lld, but it didn't verify its output.</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jul 30, 2018 at 2:03 PM Andrew Kelley <<a href="mailto:superjoe30@gmail.com">superjoe30@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Ping Rui. Is there anything else that needs to be done on this patch?<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 17, 2018 at 6:58 AM, Carlo Kok via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Got it. <br>
<br>
Attached are both the testcase & the fix. <br>
<div><div class="m_-4529246432369178717h5"><br>
On Tue, Jul 17, 2018, at 12:06, Carlo Kok via llvm-dev wrote:<br>
> On Wed, Jul 11, 2018, at 16:45, Davide Italiano wrote:<br>
> > On Tue, Jul 10, 2018 at 10:12 PM Carlo Kok via llvm-dev<br>
> > <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
> > ><br>
> > > That sounds quite reasaonable; how does one usually go about doing that? a repro zip that hits both asserts?<br>
> > ><br>
> > <br>
> > You can take inspiration from anything in lld/test, but basically<br>
> > either an assembly source (or multiple) passed through llvm-mc and<br>
> > then lld, or a YAML file passed to yaml2obj (and then lld). From what<br>
> > I see the Mach-O backend prefers YAML-style testing, but I don't think<br>
> > that's a strict requirement.<br>
> <br>
> Doesn't look like the test system works on windows? <br>
> <br>
> <br>
> C:\p\llvm\llvm\tools\lld\test>python c:\p\llvm\llvm-bin32\relwithdebinfo<br>
> \bin\llvm-lit.py darwin\darwin-asserts-in-x86_64.ll<br>
> llvm-lit.py: C:/p/llvm/llvm\utils\lit\lit\TestingConfig.py:101: fatal: <br>
> unable to parse config file 'C:\\p\\llvm\\llvm\\tools\\lld\\test\<br>
> \<a href="http://lit.cfg.py" rel="noreferrer" target="_blank">lit.cfg.py</a>', traceback: Traceback (most recent call last):<br>
>   File "C:/p/llvm/llvm\utils\lit\lit\TestingConfig.py", line 88, in <br>
> load_from_path<br>
>     exec(compile(data, path, 'exec'), cfg_globals, None)<br>
>   File "C:\p\llvm\llvm\tools\lld\test\<a href="http://lit.cfg.py" rel="noreferrer" target="_blank">lit.cfg.py</a>", line 23, in <module><br>
>     config.test_format = lit.formats.ShTest(not <br>
> llvm_config.use_lit_shell)<br>
> AttributeError: 'NoneType' object has no attribute 'use_lit_shell'<br>
> <br>
</div></div><span>> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</span>> Email had 1 attachment:<br>
> + darwin-asserts-in-x86_64.ll<br>
>   5k (application/octet-stream)<br>
<br>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div><br></div></div></div>
</blockquote></div>