krisb added inline comments.
================ Comment at: llvm/test/DebugInfo/Generic/split-dwarf-local-import3.ll:17-36 +; CHECK: DW_TAG_subprogram +; CHECK: DW_AT_name ("foo") +; CHECK: DW_TAG_imported_declaration +; CHECK: NULL + +; CHECK: DW_TAG_base_type +; CHECK: DW_AT_name ("int") ---------------- jmmartinez wrote: > I'd be tempted to match the offset of the abstract subprogram and of the > imported declaration too. > At least for me, it makes clear the intention of the test without running it. > > What do you think ? Good point, thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144004/new/ https://reviews.llvm.org/D144004 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits