scott.linder added inline comments.
================
Comment at: clang/test/CodeGenHIP/debug-info-address-class.hip:8
+
+// CHECK-DAG: ![[FILEVAR0:[0-9]+]] = distinct !DIGlobalVariable(name:
"FileVar0", scope: !{{[0-9]+}}, file: !{{[0-9]+}}, line: {{[0-9]+}}, type:
!{{[0-9]+}}, isLocal: false, isDefinition: true)
+// CHECK-DAG: !DIGlobalVariableExpression(var: ![[FILEVAR0]], expr:
!DIExpression())
----------------
aprantl wrote:
> They are more convenient, but having very many CHECK_DAGs is also really slow
> — would it be feasible to reorder them and use CHECKs? Perhaps by running
> clang/FileCheck twice with different sets of CHECK lines?
The test is pretty short, so I just re-ordered the checks to match how they
appear in the output (and used more descriptive names to make it easier to
follow).
This does mean the test relies on the order these things are traversed. Some
bits are maybe a bit surprising, like how the metadata for the `__shared__`
auto variable comes before the argument, but I don't imagine it is liable to
change often/accidentally.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88976/new/
https://reviews.llvm.org/D88976
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
