[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