[all-commits] [llvm/llvm-project] e5c3b9: DWARFVerifier: Test failures to parse DW_AT_locations

[David Blaikie via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e5c3b9711677fc21a3b288ecba4205b3478f890e
      https://github.com/llvm/llvm-project/commit/e5c3b9711677fc21a3b288ecba4205b3478f890e
  Author: David Blaikie <dblaikie at gmail.com>
  Date:   2021-12-16 (Thu, 16 Dec 2021)

  Changed paths:
    M llvm/test/tools/llvm-dwarfdump/X86/verify_split_cu.s

  Log Message:
  -----------
  DWARFVerifier: Test failures to parse DW_AT_locations


  Commit: 4f60a42878b0d46bc2cc84d8f0d316cac2c60c9d
      https://github.com/llvm/llvm-project/commit/4f60a42878b0d46bc2cc84d8f0d316cac2c60c9d
  Author: David Blaikie <dblaikie at gmail.com>
  Date:   2021-12-16 (Thu, 16 Dec 2021)

  Changed paths:
    M llvm/include/llvm/DebugInfo/DWARF/DWARFDebugLoc.h
    M llvm/lib/DebugInfo/DWARF/DWARFDebugLoc.cpp
    M llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp
    M llvm/test/tools/llvm-dwarfdump/X86/verify_split_cu.s
    M llvm/unittests/DebugInfo/DWARF/DWARFDieTest.cpp

  Log Message:
  -----------
  DWARFVerifier: Skip resolution failures for locations in dwo files

When reading location lists in dwo files the addresses cannot be
resolved, but that's not a problem.

Long term this probably should be fixed with a different API that
exposes location expressions without the need to resolve the address
ranges, since that's all the verifier (in its current state) requires.
(though the verifier should probably also eventually verify the address
ranges in location lists are a subset of the enclosing scope's address
range)


Compare: https://github.com/llvm/llvm-project/compare/054cdb34a20d...4f60a42878b0