[PATCH] D85683: [AMDGPU] Add dedicated llvm-readobj test. NFC.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 10 17:03:52 PDT 2020


MaskRay added a comment.

I seem to recall that for test only change, the tag can be `[test]` or test instead of `NFC`



================
Comment at: llvm/test/tools/llvm-readobj/ELF/amdgpu-elf-headers.test:1
+# RUN: yaml2obj %s --docnum=1 -o %t
+# RUN: llvm-readobj -h %t | FileCheck %s --match-full-lines -DFILE=%t --check-prefixes=GCN,GFX600
----------------
You can use `yaml2obj -D FLAGS=` to avoid replicate so many YAML inputs.

See `file-header-machine-types.test` for an example.


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

https://reviews.llvm.org/D85683



More information about the llvm-commits mailing list