Started 8 mo 4 days ago
Took 3 min 42 sec on green-dragon-15

Failed Build #293 (Feb 14, 2019 10:47:19 PM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 354098
  • http://llvm.org/svn/llvm-project/cfe/trunk : 354097
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 354092
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 346271
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 353955
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 354069
Changes
  1. X86: Replace isSafeToClobberEFLAGS implementation

    Also use modifiesRegister instead of looping over operands. (detail/ViewSVN)
    by arsenm
  2. Revert "Fix implementation of [temp.local]p4."

    This reverts commit 40bd10b770813bd1471d46f514545437516aa4ba.

    This seems to now emit an error when building the sanitizer tests:
    http://green.lab.llvm.org/green/job/clang-stage1-configure-RA/53965/consoleFull. (detail/ViewSVN)
    by thegameg
  3. Revert "[SystemZ] Do not emit VEXTEND or VROUND nodes without vector support."

    This reverts commit aa0b77d3395dc6ab91647138139c1a15a3aa088d.

    This fails to pass the machine verifier:
    http://green.lab.llvm.org/green/job/clang-stage1-cmake-RA-expensive/13579/ (detail/ViewSVN)
    by thegameg
  4. [lit][NFC] Cleanup copy&paste naming mistake (detail/ViewSVN)
    by yln
  5. [llvm-cxxfilt] Fix a comment typo. NFC. (detail/ViewSVN)
    by mattd
  6. [GISel][NFC]: Add methods to speed up insertion into GISelWorklist

    https://reviews.llvm.org/D58073

    Speed up insertion during the initial populating phase into the
    GISelWorkList by deferring repeatedly resizing the DenseMap.
    This results in ~10% improvement in the combiner passes, and
    ~3% speedup in the Legalizer.

    reviewed by: aemerson. (detail/ViewSVN)
    by aditya_nandakumar
  7. [libFuzzer] print new functions as they are discovered in the fork mode (detail/ViewSVN)
    by kcc
  8. Fix implementation of [temp.local]p4.

    When a template-name is looked up, we need to give injected-class-name
    declarations of class templates special treatment, as they denote a
    template rather than a type.

    Previously we achieved this by applying a filter to the lookup results
    after completing name lookup, but that is incorrect in various ways, not
    least of which is that it lost all information about access and how
    members were named, and the filtering caused us to generally lose
    all ambiguity errors between templates and non-templates.

    We now preserve the lookup results exactly, and the few places that need
    to map from a declaration found by name lookup into a declaration of a
    template do so explicitly. Deduplication of repeated lookup results of
    the same injected-class-name declaration is done by name lookup instead
    of after the fact. (detail/ViewSVN)
    by rsmith
  9. PR40642: Fix determination of whether the final statement of a statement
    expression is a discarded-value expression.

    Summary:
    We used to get this wrong in three ways:

    1) During parsing, an expression-statement followed by the }) ending a
       statement expression was always treated as producing the value of the
       statement expression. That's wrong for ({ if (1) expr; })
    2) During template instantiation, various kinds of statement (most
       statements not appearing directly in a compound-statement) were not
       treated as discarded-value expressions, resulting in missing volatile
       loads (etc).
    3) In all contexts, an expression-statement with attributes was not
       treated as producing the value of the statement expression, eg
       ({ [[attr]] expr; }).

    Also fix incorrect enforcement of OpenMP rule that directives can "only
    be placed in the program at a position where ignoring or deleting the
    directive would result in a program with correct syntax". In particular,
    a label (be it goto, case, or default) should not affect whether
    directives are permitted.

    Reviewers: aaron.ballman, rjmccall

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D57984 (detail/ViewSVN)
    by rsmith
  10. Fixed failure on Darwin due to r354064

    Summary:
    instrprof-darwin-exports.c test fails on Darwin due  to r354064.

    Updated clang list of exported symbols to fix the issue.

    Reviewers: vsk

    Reviewed By: vsk

    Subscribers: davidxl, efriedma

    Differential Revision: https://reviews.llvm.org/D58259 (detail/ViewSVN)
    by apazos
  11. [libFuzzer] fix the unit tests (detail/ViewSVN)
    by kcc
  12. [libFuzzer] when doing the merge, keep track of the coveraged edges, not just features (detail/ViewSVN)
    by kcc
  13. AMDGPU: Set ABI version to 1 for code object v3

    Differential Revision: https://reviews.llvm.org/D57811 (detail/ViewSVN)
    by kzhuravl
  14. [Driver][Darwin] Emit an error when using -pg on OS without support for it.

    Instead of letting a program fail at runtime, emit an error during
    compilation.

    rdar://problem/12206955

    Reviewers: dexonsmith, bob.wilson, steven_wu

    Reviewed By: steven_wu

    Subscribers: jkorous, cfe-commits

    Differential Revision: https://reviews.llvm.org/D57991 (detail/ViewSVN)
    by vsapsai
  15. [symbolizer] Avoid collecting symbols belonging to invalid sections.

    Summary:
    llvm-symbolizer would originally report symbols that belonged to an invalid object file section.
    Specifically the case where: `*Symbol.getSection() == ObjFile.section_end()`
    This patch prevents the Symbolizer from collecting symbols that belong to invalid sections.

    The test  (from PR40591) introduces a case where two symbols have address 0,
    one symbol is defined, 'foo', and the other is not defined, 'bar'.  This patch will cause
    the Symbolizer to keep 'foo' and ignore 'bar'.

    As a side note, the logic for adding symbols to the Symbolizer's store
    (`SymbolizableObjectFile::addSymbol`) replaces symbols with the
    same <address, size> pair.  At some point that logic should be revisited as in the
    aforementioned case, 'bar' was overwriting 'foo' in the Symbolizer's store,
    and 'foo' was forgotten.

    This fixes PR40591

    Reviewers: jhenderson, rupprecht

    Reviewed By: rupprecht

    Subscribers: hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D58146 (detail/ViewSVN)
    by mattd
  16. Revert "[INLINER] allow inlining of address taken blocks"

    This reverts commit 19e95fe61182945b7b68ad15348f144fb996633f. (detail/ViewSVN)
    by nickdesaulniers
  17. [INLINER] allow inlining of address taken blocks

    as long as their uses does not contain calls to functions that capture
    the argument (potentially allowing the blockaddress to "escape" the
    lifetime of the caller).

    TODO:
    - add more tests
    - fix crash in llvm::updateCGAndAnalysisManagerForFunctionPass when
      invoking Transforms/Inline/blockaddress.ll (detail/ViewSVN)
    by nickdesaulniers
  18. [x86] add tests for extractelement of FP; NFC (detail/ViewSVN)
    by spatel
  19. [libFuzzer] when doing the merge, keep track of the coveraged edges, not just features (detail/ViewSVN)
    by kcc
  20. [clang][FileManager] fillRealPathName even if we aren't opening the file

    The pathname wasn't previously filled when the getFile() method was called with openFile = false.
    We are caching FileEntry-s in ParsedAST::Includes in clangd and this caused the problem.

    This fixes an internal test failure in clangd - ClangdTests.GoToInclude.All

    rdar://47536127

    Differential Revision: https://reviews.llvm.org/D58213 (detail/ViewSVN)
    by Jan Korous
  21. [Sema] Fix-up a -Wfloat-conversion diagnostic

    We were warning on valid ObjC property reference exprs, and passing
    in the wrong arguments to DiagnoseFloatingImpCast (leading to a badly
    worded diagnostic).

    rdar://47644670

    Differential revision: https://reviews.llvm.org/D58145 (detail/ViewSVN)
    by epilk
  22. [lit] Remove --single-process option (use -j1 instead)

    Remove `--single-process` command line option. Use `-j1` instead.

    Also see commit: 96adb78b120b6aa9739eb714534dc8e819f7bc52 (detail/ViewSVN)
    by yln
  23. MC/ELF: Allow targets to set ABI version

    Tests are in the follow up change

    Differential Revision: https://reviews.llvm.org/D57810 (detail/ViewSVN)
    by kzhuravl
  24. GlobalISel: Add alignment to LegalityQuery MMOs

    This allows targets to specify the minimum alignment required for the
    load/store. (detail/ViewSVN)
    by arsenm
  25. Replace gcroot verifier tests

    These haven't been checking anything useful and have been testing the
    wrong failure reason for many years. Replace them with something which
    stresses what is actually implemented in the verifier now. (detail/ViewSVN)
    by arsenm
  26. [clang-tidy] Mention language version in test explicitly.

    "modernize-use-using" check is applicable only to C++11 and later. Spell it out
    to avoid relying on default language version.

    rdar://problem/47932196 (detail/ViewSVN)
    by vsapsai
  27. [lit] Set --single-process for single tests and --threads=1

    Summary:
    Automatically upgrade debugging experience (single process, no thread
    pool) when:
      1) we only run a single test
      2) user specifies `-j1`

    Details:
    Fix `--max-failures` in single process mode. Option did not have an
    effect in single process mode.

    Add display feedback for single process mode. Adapted test.

    Improve argument checking (require positive integers).

    `--single-process` is now essentially an alias for `-j1`. Should we
    remove it?

    Reviewers: rnk

    Differential Revision: https://reviews.llvm.org/D58249 (detail/ViewSVN)
    by yln
  28. AMDGPU/GlobalISel: Fix RegBankSelect for GEP.

    This is basically a pointer typed add, so shouldn't be any different.
    This was assuming everything was an SGPR, which is not true.

    Also cleanup legality for GEP. I don't seem to be seeing the problem
    the hack marking s64 as a legal pointer type the comment mentions. (detail/ViewSVN)
    by arsenm
  29. [AMDGPU] Ressociate 'add (add x, y), z' to use SALU

    Reassociate adds to collect scalar operands in a single
    instruction when possible. That will result in a scalar
    add followed by vector instead of two vector adds, thus
    better utilizing SALU.

    Differential Revision: https://reviews.llvm.org/D58220 (detail/ViewSVN)
    by rampitec
  30. AMDGPU/GlobalISel: Handle split for 64-bit VALU select (detail/ViewSVN)
    by arsenm

Started by upstream project Clang Stage 2: cmake, R -g Asan&UBSan, using Stage 1 RA, Phase 1 build number 6136
originally caused by:

This run spent:

  • 6.5 sec waiting;
  • 3 min 42 sec build duration;
  • 3 min 49 sec total from scheduled to completion.

Identified problems

Missing test results

The test result file Jenkins is looking for does not exist after the build.
Indication 1

CMake Error

This build failed because of an CMake configuration error. Below is a list of all errors in the build log:
Indication 2