<div dir="ltr"><div><div><div>Davide,<br><br></div>This patch fixes the test failure. Please commit.<br><br></div>Thanks<br><br></div>Galina<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 7, 2017 at 11:26 AM, Davide Italiano <span dir="ltr"><<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think I understand what's going on. It's lit doing a substitution, i.e.<br>
<br>
    ToolSubst('%llc_dwarf', FindTool('llc'), extra_args=llc_args),<br>
<br>
<br>
llc_args = []<br>
<br>
# Similarly, have a macro to use llc with DWARF even when the host is win32.<br>
if re.search(r'win32', config.target_triple):<br>
    llc_args = [' -mtriple=' +<br>
                config.target_triple.replace('<wbr>-win32', '-mingw32')]<br>
<br>
<br>
Galina, can you please try the following patch? If that works, I'll commit.<br>
<br>
$ git diff<br>
diff --git a/test/DebugInfo/X86/nodebug.<wbr>ll b/test/DebugInfo/X86/nodebug.<wbr>ll<br>
index 67dd6549a4c..a5ad5e08c41 100644<br>
--- a/test/DebugInfo/X86/nodebug.<wbr>ll<br>
+++ b/test/DebugInfo/X86/nodebug.<wbr>ll<br>
<span class="">@@ -1,6 +1,6 @@<br>
 ; REQUIRES: object-emission<br>
<br>
</span>-; RUN: %llc_dwarf < %s -filetype=obj -mtriple=x86_64-apple-darwin |<br>
<span class="">llvm-dwarfdump -v - | FileCheck %s<br>
</span>+; RUN: llc < %s -filetype=obj -mtriple=x86_64-apple-darwin |<br>
<span class="im HOEnZb">llvm-dwarfdump -v - | FileCheck %s<br>
<br>
 ; Test that a nodebug function (a function not appearing in the debug info IR<br>
 ; metadata subprogram list) with DebugLocs on its IR doesn't cause crashes/does<br>
<br>
</span><div class="HOEnZb"><div class="h5">On Thu, Dec 7, 2017 at 11:19 AM, Davide Italiano <<a href="mailto:davide@freebsd.org">davide@freebsd.org</a>> wrote:<br>
> This is not related to my commit, something (presumably the driver?)<br>
> is passing a mingw triple?<br>
> I don't even know how to debug this because I don't have access to a<br>
> machine with that triple.<br>
> CC:ing Martin/Martell as they might now what's going on.<br>
><br>
> ******************** TEST 'LLVM :: DebugInfo/Generic/nodebug.ll'<br>
> FAILED ********************<br>
> Script:<br>
> --<br>
> C:\ps4-buildslave2\llvm-clang-<wbr>x86_64-expensive-checks-win\<wbr>build\bin\llc.EXE<br>
>  -mtriple=x86_64-pc-mingw32 <<br>
> C:\ps4-buildslave2\llvm-clang-<wbr>x86_64-expensive-checks-win\<wbr>llvm\test\DebugInfo\Generic\<wbr>nodebug.ll<br>
> -filetype=obj -mtriple=x86_64-apple-darwin |<br>
> C:\ps4-buildslave2\llvm-clang-<wbr>x86_64-expensive-checks-win\<wbr>build\bin\llvm-dwarfdump.EXE<br>
> -v - | C:\ps4-buildslave2\llvm-clang-<wbr>x86_64-expensive-checks-win\<wbr>build\bin\FileCheck.EXE<br>
> C:\ps4-buildslave2\llvm-clang-<wbr>x86_64-expensive-checks-win\<wbr>llvm\test\DebugInfo\Generic\<wbr>nodebug.ll<br>
> --<br>
> Exit Code: 2<br>
><br>
> Command Output (stdout):<br>
> --<br>
> $ "C:\ps4-buildslave2\llvm-<wbr>clang-x86_64-expensive-checks-<wbr>win\build\bin\llc.EXE"<br>
> "-mtriple=x86_64-pc-mingw32" "-filetype=obj"<br>
> "-mtriple=x86_64-apple-darwin"<br>
> # command stderr:<br>
> llc.EXE: for the -mtriple option: may only occur zero or one times!<br>
><br>
> On Thu, Dec 7, 2017 at 11:09 AM, Galina Kistanova <<a href="mailto:gkistanova@gmail.com">gkistanova@gmail.com</a>> wrote:<br>
>> Hello Davide,<br>
>><br>
>> One of your resent commits broke the test on the builder:<br>
>><br>
>> <a href="http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win/builds/6619" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/<wbr>builders/llvm-clang-x86_64-<wbr>expensive-checks-win/builds/<wbr>6619</a><br>
>><br>
>> . . .<br>
>> Failing Tests (2):<br>
>> . . .<br>
>>     LLVM :: DebugInfo/Generic/nodebug.ll<br>
>> . . .<br>
>><br>
>> Please have a look?<br>
>> The builder was already red and did not send notifications.<br>
>><br>
>> Thanks<br>
>><br>
>> Galina<br>
>><br>
>> On Wed, Dec 6, 2017 at 5:22 PM, Davide Italiano via llvm-commits<br>
>> <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
>>><br>
>>> Author: davide<br>
>>> Date: Wed Dec  6 17:22:10 2017<br>
>>> New Revision: 319997<br>
>>><br>
>>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=319997&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=319997&view=rev</a><br>
>>> Log:<br>
>>> [DebugInfo] Explicitly pass a triple to this test.<br>
>>><br>
>>> As we emit different linetables format on different operating<br>
>>> systems, this currently fails on linux. Speculative commit<br>
>>> to fix the bots.<br>
>>><br>
>>> Modified:<br>
>>>     llvm/trunk/test/DebugInfo/<wbr>Generic/nodebug.ll<br>
>>><br>
>>> Modified: llvm/trunk/test/DebugInfo/<wbr>Generic/nodebug.ll<br>
>>> URL:<br>
>>> <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/Generic/nodebug.ll?rev=319997&r1=319996&r2=319997&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/llvm/trunk/test/<wbr>DebugInfo/Generic/nodebug.ll?<wbr>rev=319997&r1=319996&r2=<wbr>319997&view=diff</a><br>
>>><br>
>>> ==============================<wbr>==============================<wbr>==================<br>
>>> --- llvm/trunk/test/DebugInfo/<wbr>Generic/nodebug.ll (original)<br>
>>> +++ llvm/trunk/test/DebugInfo/<wbr>Generic/nodebug.ll Wed Dec  6 17:22:10 2017<br>
>>> @@ -1,6 +1,6 @@<br>
>>>  ; REQUIRES: object-emission<br>
>>><br>
>>> -; RUN: %llc_dwarf < %s -filetype=obj | llvm-dwarfdump -v - | FileCheck %s<br>
>>> +; RUN: %llc_dwarf < %s -filetype=obj -mtriple=x86_64-apple-darwin |<br>
>>> llvm-dwarfdump -v - | FileCheck %s<br>
>>><br>
>>>  ; Test that a nodebug function (a function not appearing in the debug<br>
>>> info IR<br>
>>>  ; metadata subprogram list) with DebugLocs on its IR doesn't cause<br>
>>> crashes/does<br>
>>><br>
>>><br>
>>> ______________________________<wbr>_________________<br>
>>> llvm-commits mailing list<br>
>>> <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
>>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
>><br>
>><br>
><br>
><br>
><br>
> --<br>
> Davide<br>
><br>
> "There are no solved problems; there are only problems that are more<br>
> or less solved" -- Henri Poincare<br>
<br>
<br>
<br>
--<br>
Davide<br>
<br>
"There are no solved problems; there are only problems that are more<br>
or less solved" -- Henri Poincare<br>
</div></div></blockquote></div><br></div>