[PATCH] D66328: [DebugInfo] Add debug location to dynamic atexit destructor
Adrian Prantl via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 16 09:11:44 PDT 2019
aprantl requested changes to this revision.
aprantl added inline comments.
This revision now requires changes to proceed.
Comment at: test/CodeGen/debug-info-no-location.cpp:4
+// CHECK-NEXT: ret void, !dbg !51
+// CHECK: !48 = distinct !DISubprogram(name: "`dynamic atexit destructor for 'f'", scope: !3, file: !3, line: 16, type: !49, scopeLine: 16, spFlags: DISPFlagLocalToUnit | DISPFlagDefinition, unit: !27, retainedNodes: !29)
+// CHECK: !51 = !DILocation(line: 16, scope: !48)
Please don't hardcode metadata numbers in the tests, this is bound to break almost immediately.
It seems like this function is compiler-generated and thus should be marked as artificial?
CHANGES SINCE LAST ACTION
More information about the cfe-commits