[PATCH] D105613: [Debug-Info] [llvm-dwarfdump] Don't try to dump location list for attributes that don't have the loclist class.

[Esme Yi via Phabricator via llvm-commits]

Esme added inline comments.


================
Comment at: llvm/test/tools/llvm-dwarfdump/ELF/formclass3.s:14
+
+# RUN: llvm-mc %s -filetype obj -triple powerpc64le-unknown-unknown -o %t.o
+# RUN: llvm-dwarfdump -debug-info %t.o | FileCheck %s -check-prefix=AT-COUNT
----------------
probinson wrote:
> If you simplify the source to have no executable instructions, then I *think* you can remove the `-triple` which will allow this test to be run in all environments.  (A number of bots do not build all targets, so it's preferable to have tests be as generic as possible.)
Some targets, like AIX, have not yet supported llvm-mc. So I think we should still keep the `-triple`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105613