[llvm] [DirectX][objdump] Add support for printing signatures (PR #152531)
James Henderson via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 7 23:47:14 PDT 2025
================
@@ -0,0 +1,161 @@
+# RUN: yaml2obj %s -o %t
+# RUN: llvm-objdump -p %t | FileCheck %s
----------------
jh7370 wrote:
A few nits/suggestions:
1) I'd add a blank line between the RUN and YAML lines.
2) I'd add a brief comment at the start of the test describing the feature being tested.
3) I'd add `--match-full-lines` and `--strict-whitespace` to the FileCheck command, to ensure the formatting matches the expected.
https://github.com/llvm/llvm-project/pull/152531
More information about the llvm-commits
mailing list