Started 16 days ago
Took 1 hr 16 min on green-dragon-19

Success Build rL:362929 - C:362924 - #57394 (Jun 10, 2019 5:15:35 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 362929
  • http://llvm.org/svn/llvm-project/cfe/trunk : 362924
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 362859
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 362745
  • http://llvm.org/svn/llvm-project/zorg/trunk : 362851
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 362866
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 362811
Changes
  1. [yaml2obj] - Do not assert when .dynsym is specified explicitly, but .dynstr is not present.

    We have a code in buildSectionIndex() that adds implicit sections:

    // Add special sections after input sections, if necessary.
    for (StringRef Name : implicitSectionNames())
      if (SN2I.addName(Name, SecNo)) {
        // Account for this section, since it wasn't in the Doc
        ++SecNo;
        DotShStrtab.add(Name);
      }

    The problem arises when .dynsym is specified explicitly and no
    DynamicSymbols is used. In that case, we do not add
    .dynstr implicitly and will assert later when will try to set Link
    for .dynsym.

    Seems, in this case, reasonable behavior is to allow Link field to be zero.
    This is what this patch does.

    Differential revision: https://reviews.llvm.org/D63001 (detail/ViewSVN)
    by grimar

Started by an SCM change (2 times)

This run spent:

  • 37 min waiting;
  • 1 hr 16 min build duration;
  • 1 hr 53 min total from scheduled to completion.
LLVM/Clang Warnings: 1 warning.
    Test Result (no failures)