[llvm] r319997 - [DebugInfo] Explicitly pass a triple to this test.

[Yvan Roux via llvm-commits]

Hi Davide,

On 7 December 2017 at 02:22, Davide Italiano via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: davide
> Date: Wed Dec  6 17:22:10 2017
> New Revision: 319997
>
> URL: http://llvm.org/viewvc/llvm-project?rev=319997&view=rev
> Log:
> [DebugInfo] Explicitly pass a triple to this test.
>
> As we emit different linetables format on different operating
> systems, this currently fails on linux. Speculative commit
> to fix the bots.
>
> Modified:
>     llvm/trunk/test/DebugInfo/Generic/nodebug.ll
>
> Modified: llvm/trunk/test/DebugInfo/Generic/nodebug.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/Generic/nodebug.ll?rev=319997&r1=319996&r2=319997&view=diff
> ==============================================================================
> --- llvm/trunk/test/DebugInfo/Generic/nodebug.ll (original)
> +++ llvm/trunk/test/DebugInfo/Generic/nodebug.ll Wed Dec  6 17:22:10 2017
> @@ -1,6 +1,6 @@
>  ; REQUIRES: object-emission
>
> -; RUN: %llc_dwarf < %s -filetype=obj | llvm-dwarfdump -v - | FileCheck %s
> +; RUN: %llc_dwarf < %s -filetype=obj -mtriple=x86_64-apple-darwin | llvm-dwarfdump -v - | FileCheck %s

This ARM and AArch64 bots are still broken with this as they don't
have x86_64-apple-darwin backend support.  I still don't know well the
testsuite syntax (I'm new in the community) but I d'on't think
restricting this test to x86_64 darwin target is the right fix for the
previous regression, since it will is will remove a test which did
work on other targets, it should be better to check the expected value
on each targets.

Thanks
Yvan

>  ; Test that a nodebug function (a function not appearing in the debug info IR
>  ; metadata subprogram list) with DebugLocs on its IR doesn't cause crashes/does
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits