Started 2 mo 26 days ago
Took 9 hr 31 min on green-dragon-13

Success Build #4179 (Jul 23, 2019 12:03:10 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 366781
  • http://llvm.org/svn/llvm-project/cfe/trunk : 366761
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 366738
  • http://llvm.org/svn/llvm-project/lld/trunk : 366780
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 364589
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 366776
  • http://llvm.org/svn/llvm-project/test-suite/trunk : 366290
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 366698
  • http://llvm.org/svn/llvm-project/polly/trunk : 366697
Changes
  1. [AMDGPU][NFC] Simplify test file for amdgcn intrinsics

    Summary: Remove unchecked attribute in the call site and use FileCheck String Substitution for `convergent` check.

    Reviewers: nhaehnle

    Reviewed By: nhaehnle

    Subscribers: kzhuravl, jvesely, wdng, nhaehnle, yaxunl, dstuttard, tpr, t-tye, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64901 (detail/ViewSVN)
    by uenoku
  2. [COFF] Unbreak sorting of mingw comdat .tls sections after SVN r363457

    Code built for mingw with -fdata-sections will store each TLS variable
    in a comdat section, named .tls$$<varname>. Normal TLS variables are
    stored in sections named .tls$ with a trailing dollar, which are
    sorted after a starter marker (in a later linked object file) in a
    section named ".tls" (with no dollar suffix), before an ending marker
    in a section named ".tls$ZZZ".

    The mingw comdat section suffix stripping introduced in SVN r363457
    broke sorting of such tls sections, ending up sorting the stripped
    .tls$$<varname> sections (stripped to ".tls") before the start marker
    in the section named ".tls".

    We could add exceptions to the section name suffix stripping for
    .tls (and .CRT, where suffixes always should be honored), but the
    more conservative option is probably the reverse; to only apply the
    stripping for the normal sections where sorting shouldn't have any
    effect.

    Differential Revision: https://reviews.llvm.org/D65018 (detail/ViewSVN)
    by mstorsjo
  3. [DAGCombiner] Make ShrinkLoadReplaceStoreWithStore return an SDValue instead of an SDNode*. NFCI

    The function was calling getNode() on an SDValue to return and the
    caller turned the result back into a SDValue. So just return the
    original SDValue to avoid this. (detail/ViewSVN)
    by ctopper
  4. [DAGCombiner] Use SDNode::isOperandOf to simplify some code. NFCI (detail/ViewSVN)
    by ctopper
  5. [LLVM-C] Improve Bindings to The Internalize Pass

    Summary: Adds a binding to the internalize pass that allows the caller to pass a function pointer that acts as the visibility-preservation predicate.  Previously, one could only pass an unsigned value (not LLVMBool?) that directed the pass to consider "main" or not.

    Reviewers: whitequark, deadalnix, harlanhaskins

    Reviewed By: whitequark, harlanhaskins

    Subscribers: kren1, hiraditya, llvm-commits, harlanhaskins

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D62456 (detail/ViewSVN)
    by codafi
  6. Implement most of P1612R1: Relocate endian.  Moves the std::endian functionality from 'type-traits' to 'bit'. No other change. The reason that this is 'partial' is that P1621 also recommends a feature-test macro, but I don't have the value for that one yet. In a month or so, I'll add that (detail/ViewSVN)
    by marshall
  7. [PowerPC] Replace float load/store pair with integer load/store pair when it's only used in load/store

    Replace float load/store pair with integer load/store pair when it's only used in load/store,
    because float load/store instructions cost more cycles then integer load/store.

    A typical scenario is when there is a call with more than 13 float arguments passing, we need pass them by stack.
    So we need a load/store pair to do such memory operation if the variable is global variable.

    Differential Revision: https://reviews.llvm.org/D64195 (detail/ViewSVN)
    by wuzish
  8. Inline function call into assert to fix unused variable warning. (detail/ViewSVN)
    by rtrieu
  9. Move variable out from debug only section.

    MFI is no longer just needed for an assert.  Move it out of the debug only
    section to allow non-assert builds to be able to find it. (detail/ViewSVN)
    by rtrieu
  10. [llvm-lipo] Implement -info

    Prints architecture type of all input files.

    Patch by Anusha Basana <anusha.basana@gmail.com>

    Differential Revision: https://reviews.llvm.org/D64668 (detail/ViewSVN)
    by smeenai
  11. Fixing build error from commit 95cbc3d

    [Attributor] Liveness analysis.

    Liveness analysis abstract attribute used to indicate which BasicBlocks are dead and can therefore be ignored.
    Right now we are only looking at noreturn calls.

    Reviewers: jdoerfert, uenoku

    Subscribers: hiraditya, llvm-commits

    Differential Revision: https://reviews.llvm.org/D64162 (detail/ViewSVN)
    by sstefan
  12. [Statepoints] Fix a bug in statepoint lowering for functions w/no-realign-stack

    We were silently using the ABI alignment for all of the stores generated for deopt and gc values.  We'd gotten the alignment of the stack slot itself properly reduced (via MachineFrameInfo's clamping), but having the MMO on the store incorrect was enough for us to generate an aligned store to a unaligned location.

    The simplest fix would have been to just pass the alignment to the helper function, but once we do that, the helper function doesn't really help.  So, inline it and directly call the MMO version of DAG.getStore with a properly constructed MMO.

    Note that there's a separate performance possibility here.  Even if we *can* realign stacks, we probably don't *want to* if all of the stores are in slowpaths.  But that's a later patch, if at all.  :) (detail/ViewSVN)
    by reames
  13. Fix pointer width in test from r366754. (detail/ViewSVN)
    by pcc
  14. gn build: Wrap two comments to 80 columns (detail/ViewSVN)
    by nico
  15. [DWARF] Add more error handling to debug line parser.

    This patch exnteds the error handling in the debug line parser to get
    rid of the existing MD5 assertion. I want to reuse the debug line parser
    from LLVM in LLDB where we cannot crash on invalid input.

    Differential revision: https://reviews.llvm.org/D64544 (detail/ViewSVN)
    by Jonas Devlieghere
  16. [NFC][clang] Refactor getCompilationPhases()+Types.def step 1.

    Moves list of phases into Types.def table: Currently Types.def contains a
    table of strings that are used to assemble a list of compilation phases to be
    setup in the clang driver's jobs pipeline. This change makes it so that the table
    itself contains the list of phases. A subsequent patch will remove the strings.

    Differential Revision: https://reviews.llvm.org/D64098 (detail/ViewSVN)
    by zer0
  17. [Statepoints] Add a test which shows a miscompile with no-realign-stacks (detail/ViewSVN)
    by reames
  18. Revert "Fixing build error from commit 9285295."

    This reverts commit 95cbc3da8871f43c1ce2b2926afaedcd826202b1. (detail/ViewSVN)
    by sstefan
  19. llvm-objcopy/test: add REQUIRES: shell for use of umask

    (follow-up to r365162) (detail/ViewSVN)
    by dblaikie
  20. Analysis: Don't look through aliases when simplifying GEPs.

    It is not safe in general to replace an alias in a GEP with its aliasee
    if the alias can be replaced with another definition (i.e. via strong/weak
    resolution (linkonce_odr) or via symbol interposition (default visibility
    in ELF)) while the aliasee cannot. An example of how this can go wrong is
    in the included test case.

    I was concerned that this might be a load-bearing misoptimization (it's
    possible for us to use aliases to share vtables between base and derived
    classes, and on Windows, vtable symbols will always be aliases in RTTI
    mode, so this change could theoretically inhibit trivial devirtualization
    in some cases), so I built Chromium for Linux and Windows with and without
    this change. The file sizes of the resulting binaries were identical, so it
    doesn't look like this is going to be a problem.

    Differential Revision: https://reviews.llvm.org/D65118 (detail/ViewSVN)
    by pcc
  21. Fixing build error from commit 9285295.

    [Attributor] Liveness analysis.

    Liveness analysis abstract attribute used to indicate which BasicBlocks are dead and can therefore be ignored.
    Right now we are only looking at noreturn calls.

    Reviewers: jdoerfert, uenoku

    Subscribers: hiraditya, llvm-commits

    Differential revision: https://reviews.llvm.org/D64162 (detail/ViewSVN)
    by sstefan
  22. [NFC][PatternMatch] Refactor code into a proper "matcher for any integral constant"

    Having it as a proper matcher is better for reusability elsewhere
    (in a follow-up patch.) (detail/ViewSVN)
    by lebedevri
  23. [InstSimplify][NFC] Tests for skipping 'div-by-0' checks before @llvm.umul.with.overflow

    These may remain after @llvm.umul.with.overflow was canonicalized
    from the code that was originally doing the check via division. (detail/ViewSVN)
    by lebedevri
  24. [SimplifyCFG][NFC] Test that we fail to flatten CFG in JPEG "sign" value extend pattern

    This comes up in JPEG decoding, see e.g.
    Figure F.12 – Extending the sign bit of a decoded value in V
    of ITU T.81 (JPEG specification). (detail/ViewSVN)
    by lebedevri
  25. [SimplifyCFG][NFC] Test that we fail to flatten CFG after forming @llvm.umul.with.overflow

    Even if we formed @llvm.umul.with.overflow, we are still stuck
    with that guard against div-by-zero, which is no longer needed,
    because we didn't flatten the CFG. (detail/ViewSVN)
    by lebedevri
  26. [InstCombine][NFC] Tests for canonicalization of unsigned multiply overflow check (detail/ViewSVN)
    by lebedevri
  27. [NFC][PhaseOrdering] Add tests showcasing the problems of unsigned multiply overflow check

    While we can form the @llvm.mul.with.overflow easily,
    we are still left with that check that was guarding against div-by-0.
    And in the second case we won't even flatten the CFG. (detail/ViewSVN)
    by lebedevri
  28. [IndVarSimplify][NFC] Autogenerate check lines in loop_evaluate_1.ll

    Being affected by upcoming patch. (detail/ViewSVN)
    by lebedevri
  29. [Driver] Set the default win32-macho debug format to DWARF

    rdar://53267670

    Differential Revision: https://reviews.llvm.org/D65116 (detail/ViewSVN)
    by Vedant Kumar
  30. AMDGPU: Don't use SDNodeXForm for DS offset output

    The xform has no real valuewhen it's using out of a complex pattern
    output. The complex pattern was already creating TargetConstants with
    i16, so this was just unnecessary machinery.

    This allows global isel to import the simple cases once the complex
    pattern is implemented. (detail/ViewSVN)
    by arsenm

Started by upstream project SVN: Clang Stage 1: cmake, RA, using system compiler build number 58175
originally caused by:

Started by upstream project SVN: Clang Stage 1: cmake, RA, using system compiler build number 58176
originally caused by:

Started by upstream project SVN: Clang Stage 1: cmake, RA, using system compiler build number 58179
originally caused by:

Started by upstream project SVN: Clang Stage 1: cmake, RA, using system compiler build number 58178
originally caused by:

Started by upstream project SVN: Clang Stage 1: cmake, RA, using system compiler build number 58180
originally caused by:

Started by upstream project SVN: Clang Stage 1: cmake, RA, using system compiler build number 58181
originally caused by:

This run spent:

  • 8 hr 54 min waiting;
  • 9 hr 31 min build duration;
  • 18 hr total from scheduled to completion.
LLVM/Clang Warnings: 2 warnings.