[PATCH] D138415: add LLVMGetDINodeTag

NAKAMURA Takumi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 8 19:07:00 PST 2023


chapuni added a comment.

Warnings in Release build.



================
Comment at: llvm/tools/llvm-c-test/debuginfo.c:211
+  LLVMMetadataRef md = LLVMMDNodeInContext2(context, metas, 1);
+  uint16_t tag0 = LLVMGetDINodeTag(md);
+
----------------
Could you fold it into assert?


================
Comment at: llvm/tools/llvm-c-test/debuginfo.c:222
+      LLVMDIFlagObjcClassComplete, NULL, NULL, 0, 0, NULL, NULL, 0);
+  uint16_t tag1 = LLVMGetDINodeTag(decl);
+
----------------
ditto.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138415



More information about the llvm-commits mailing list