[all-commits] [llvm/llvm-project] f5f8d8: Don't error for zero-length arange entries

[Jonas Hahnfeld via All-commits]

  Branch: refs/heads/release/11.x
  Home:   https://github.com/llvm/llvm-project
  Commit: f5f8d86dc4c91ef492b919edf98335d4d09188a8
      https://github.com/llvm/llvm-project/commit/f5f8d86dc4c91ef492b919edf98335d4d09188a8
  Author: James Henderson <james.henderson at sony.com>
  Date:   2020-12-16 (Wed, 16 Dec 2020)

  Changed paths:
    M llvm/lib/DebugInfo/DWARF/DWARFDebugArangeSet.cpp
    M llvm/unittests/DebugInfo/DWARF/DWARFDebugArangeSetTest.cpp

  Log Message:
  -----------
  Don't error for zero-length arange entries

Although the DWARF specification states that .debug_aranges entries
can't have length zero, these can occur in the wild. There's no
particular reason to enforce this part of the spec, since functionally
they have no impact. The patch removes the error and introduces a new
warning for premature terminator entries which does not stop parsing.

This is a relanding of cb3a598c87db, adding the missing obj2yaml part
that was needed.

Fixes https://bugs.llvm.org/show_bug.cgi?id=46805. See also
https://reviews.llvm.org/D71932 which originally introduced the error.

Reviewed by: ikudrin, dblaikie, Higuoxing

Differential Revision: https://reviews.llvm.org/D85313


  Commit: d104e582838fd73d6ef565788f11617eccab87e2
      https://github.com/llvm/llvm-project/commit/d104e582838fd73d6ef565788f11617eccab87e2
  Author: Jonas Hahnfeld <hahnjo at hahnjo.de>
  Date:   2020-12-17 (Thu, 17 Dec 2020)

  Changed paths:
    M llvm/cmake/modules/AddLLVM.cmake
    M llvm/include/llvm/Support/CMakeLists.txt

  Log Message:
  -----------
  [CMake] Avoid __FakeVCSRevision.h with no git repository

Set the return variable to "" in find_first_existing_vc_file to
say that there is a repository, but no file to depend on. This works
transparently for all other callers that handle undefinedness and
equality to an empty string the same way.

Use the knowledge to avoid depending on __FakeVCSRevision.h if there
is no git repository at all (for example when building a release) as
there is no point in regenerating an empty VCSRevision.h.

Differential Revision: https://reviews.llvm.org/D92718

(cherry picked from commit 6e890ec7beb0874464a0af9f84e41a987f968b23)


Compare: https://github.com/llvm/llvm-project/compare/280e47ea0e83...d104e582838f