<div dir="ltr">On Thu, Jul 4, 2013 at 10:08 AM, Nick Kledzik <span dir="ltr"><<a href="mailto:kledzik@apple.com" target="_blank">kledzik@apple.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im"><br>
On Jul 3, 2013, at 2:09 AM, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
> Author: ruiu<br>
> Date: Wed Jul  3 04:09:13 2013<br>
> New Revision: 185524<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=185524&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=185524&view=rev</a><br>
> Log:<br>
> Guard a test that fails on a Release build.<br>
><br>
> Modified:<br>
>    lld/trunk/test/layout-error-test.objtxt<br>
>    lld/trunk/test/lit.cfg<br>
><br>
> Modified: lld/trunk/test/layout-error-test.objtxt<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/test/layout-error-test.objtxt?rev=185524&r1=185523&r2=185524&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/test/layout-error-test.objtxt?rev=185524&r1=185523&r2=185524&view=diff</a><br>


> ==============================================================================<br>
> --- lld/trunk/test/layout-error-test.objtxt (original)<br>
> +++ lld/trunk/test/layout-error-test.objtxt Wed Jul  3 04:09:13 2013<br>
> @@ -1,3 +1,4 @@<br>
> +# REQUIRES: asserts<br>
</div>This still fails for me.  The REQUIRES tag is new to me.  Is “asserts” always the same thing as DEBUG() in source code?  I thought DEBUG and asserts were independent configure options.</blockquote><div><br></div><div>

I think you are right; "asserts" does not meani t's compiled with DEBUG. Looks like llc knows whether its compiled with DEBUG, so I'll add "debug" attribute and use it in REQUIRED tag.</div><div style>

<br></div><div style>REQUIRES tag seems to be used in many tests in LLVM (not LLD).</div><div style><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<span class=""><font color="#888888">
-Nick<br>
</font></span><div class=""><div class="h5"><br>
> # RUN: lld -core --add-pass layout -mllvm -debug-only=LayoutPass %s 2>&1 \<br>
> # RUN:   | FileCheck %s -check-prefix=CHECK<br>
><br>
><br>
> Modified: lld/trunk/test/lit.cfg<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/test/lit.cfg?rev=185524&r1=185523&r2=185524&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/test/lit.cfg?rev=185524&r1=185523&r2=185524&view=diff</a><br>


> ==============================================================================<br>
> --- lld/trunk/test/lit.cfg (original)<br>
> +++ lld/trunk/test/lit.cfg Wed Jul  3 04:09:13 2013<br>
> @@ -116,3 +116,16 @@ if lit.useValgrind:<br>
> # Shell execution<br>
> if platform.system() not in ['Windows'] or lit.getBashPath() != '':<br>
>     config.available_features.add('shell')<br>
> +<br>
> +# llc knows whether it is compiled with -DNDEBUG.<br>
> +import subprocess<br>
> +try:<br>
> +    llc_cmd = subprocess.Popen([os.path.join(llvm_tools_dir, 'llc'), '-version'],<br>
> +                           stdout = subprocess.PIPE)<br>
> +except OSError, why:<br>
> +    print "Could not find llc in " + llvm_tools_dir<br>
> +    exit(42)<br>
> +<br>
> +if re.search(r'with assertions', llc_cmd.stdout.read()):<br>
> +    config.available_features.add('asserts')<br>
> +llc_cmd.wait()<br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
</div></div></blockquote></div><br></div></div>