[PATCH] D91043: [DebugInfo] Fix convert-loclist.ll

Luís Marques via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 24 15:06:52 PST 2020


luismarques added a comment.

In D91043#2406312 <https://reviews.llvm.org/D91043#2406312>, @probinson wrote:

> I think the first question is rhetorical, but it ensures a DWARF target.  The second question is the key:  What makes this test special, for @luismarques environment?  If you build with just RISC-V as the target, this entire directory should become unsupported.  If you have both X86 and RISC-V, then these tests do run, and apparently (mostly) run fine.  Why didn't this one?

I think the answer is perhaps that you need *all* of:

1. a non-x86 machine;
2. a build that includes the (non-native) x86 target;
3. a test that is x86-specific, but doesn't specify the x86 target triple nor `REQUIRES` x86.

Perhaps this test and my build setup was a rare occurrence that matched all 3 requirements?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91043



More information about the llvm-commits mailing list