[PATCH] D79573: [NFC][DebugInfo][DWARF] Add test for variables with a single location which doesn't span their entire scope.

Orlando Cazalet-Hyams via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 11 02:06:52 PDT 2020


Orlando marked an inline comment as done.
Orlando added a comment.

Thanks, landing shortly.



================
Comment at: llvm/test/DebugInfo/X86/location-range.mir:111
+  !32 = !DILocation(line: 9, column: 3, scope: !15)
+  !33 = !DILocation(line: 10, column: 1, scope: !15)
+  !34 = distinct !DISubprogram(name: "funtwo", scope: !3, file: !3, line: 13, type: !35, scopeLine: 13, flags: DIFlagAllCallsDescribed, spFlags: DISPFlagDefinition | DISPFlagOptimized, unit: !2, retainedNodes: !38)
----------------
aprantl wrote:
> you don't have to, but you can probably collapse all DILocations in the same scope to one location.
No problem, I did this before committing.


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

https://reviews.llvm.org/D79573





More information about the llvm-commits mailing list