[PATCH] D109806: DebugInfo: Move missing-abstract-variable.ll test from Generic to X86

Tom Stellard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 31 15:52:36 PDT 2022


tstellar added a comment.

In D109806#3018155 <https://reviews.llvm.org/D109806#3018155>, @jmorse wrote:

>> Also, the test passes for me when LLVM is configured to use the default triple of powerpc64le-unknown-linux-gnu, but fails when I configure the default triple to be ppc64le-redhat-linux-gnu.
>
> Hmmm. Possibly the longer term fix is for this test to
>  a) Not be in the 'Generic' directory, or
>  b) Become a MIR test instead of exercising instruction selection as part of the test too.
>
> (Adding an XFail in the meantime seems fine to me, especially as powerpc64 is already known to have this issue).

In the mean time, the test has been XFAIL'd on many more arches, so I just moved it to the X86 directory.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109806/new/

https://reviews.llvm.org/D109806



More information about the llvm-commits mailing list