Started 2 mo 6 days ago
Took 17 hr on green-dragon-08

Failed Build #5472 (Sep 11, 2019 11:24:41 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 371640
  • http://llvm.org/svn/llvm-project/cfe/trunk : 371642
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 371628
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 364589
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 371639
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 371504
Changes
  1. [MS] Consder constexpr globals to be inline, as in C++17

    Summary:
    Microsoft seems to do this regardless of the language mode, so we must
    also do it in order to be ABI compatible.

    Fixes PR36125

    Reviewers: thakis

    Subscribers: cfe-commits

    Differential Revision: https://reviews.llvm.org/D47956 (detail/ViewSVN)
    by rnk
  2. LiveIntervals: Split live intervals on multiple dead defs

    If there are multiple dead defs of the same virtual register, these
    are required to be split into multiple virtual registers with separate
    live intervals to avoid a verifier error. (detail/ViewSVN)
    by arsenm
  3. Consolidate swap, swap_ranges, and iter_swap in <type_traits>.

    NFC. Thanks to @Quuxplusone (Arthur O'Dwyer) for this change. (detail/ViewSVN)
    by zoecarver
  4. [libc++] Add a CMake cache for Apple-specific configuration options (detail/ViewSVN)
    by Louis Dionne
  5. [Clang][Bundler] Replace std::vector by SmallVector [NFC]

    Differential Revision: https://reviews.llvm.org/D67413 (detail/ViewSVN)
    by sdmitriev
  6. gn build: Merge r371635 (detail/ViewSVN)
    by nico
  7. Reland "clang-misexpect: Profile Guided Validation of Performance Annotations in LLVM"

    This patch contains the basic functionality for reporting potentially
    incorrect usage of __builtin_expect() by comparing the developer's
    annotation against a collected PGO profile. A more detailed proposal and
    discussion appears on the CFE-dev mailing list
    (http://lists.llvm.org/pipermail/cfe-dev/2019-July/062971.html) and a
    prototype of the initial frontend changes appear here in D65300

    We revised the work in D65300 by moving the misexpect check into the
    LLVM backend, and adding support for IR and sampling based profiles, in
    addition to frontend instrumentation.

    We add new misexpect metadata tags to those instructions directly
    influenced by the llvm.expect intrinsic (branch, switch, and select)
    when lowering the intrinsics. The misexpect metadata contains
    information about the expected target of the intrinsic so that we can
    check against the correct PGO counter when emitting diagnostics, and the
    compiler's values for the LikelyBranchWeight and UnlikelyBranchWeight.
    We use these branch weight values to determine when to emit the
    diagnostic to the user.

    A future patch should address the comment at the top of
    LowerExpectIntrisic.cpp to hoist the LikelyBranchWeight and
    UnlikelyBranchWeight values into a shared space that can be accessed
    outside of the LowerExpectIntrinsic pass. Once that is done, the
    misexpect metadata can be updated to be smaller.

    In the long term, it is possible to reconstruct portions of the
    misexpect metadata from the existing profile data. However, we have
    avoided this to keep the code simple, and because some kind of metadata
    tag will be required to identify which branch/switch/select instructions
    are influenced by the use of llvm.expect

    Patch By: paulkirth
    Differential Revision: https://reviews.llvm.org/D66324 (detail/ViewSVN)
    by phosek
  8. Revert [InstCombine] Use SimplifyFMulInst to simplify multiply in fma.

    This introduces additional rounding error in some cases. See D67434.

    This reverts r371518 (git commit 18a1f0818b659cee13865b4fad2648d85984a4ed) (detail/ViewSVN)
    by fhahn
  9. [Clang][Bundler] Fix for a potential memory leak [NFC]

    Bundler leaks memory if it is called with -type=o but given input isn't an object file (though it has to have a known binary type like IR, archive, etc...). Memory leak is happening when binary object returned by the createBinary(...) call cannot be casted to an ObjectFile type. In this case returned BinaryOrErr object releases ownership of the binary, but no one is taking it (see line 626).

    Differential Revision: https://reviews.llvm.org/D67416 (detail/ViewSVN)
    by sdmitriev
  10. [OPENMP] Update the diagnosis message for canonical loop form, by Chi
    Chun Chen.

    The previous patch (https://reviews.llvm.org/D54441) support the
    relational-op != very well for openmp canonical loop form, however,
    it didn't update the diagnosis message. So this patch is simply
    update the diagnosis message by adding !=, update the test
    related to it, and update the section number for canonical loop
    form for OpenMP 5.0 in comment.

    Differential Revision: https://reviews.llvm.org/D66559 (detail/ViewSVN)
    by abataev
  11. [InstSimplify] Pass SimplifyQuery into simplifyUnsignedRangeCheck() and use it for isKnownNonZero()

    This was actually the original intention in D67332,
    but i messed up and forgot about it.
    This patch was originally part of D67411, but precommitting this. (detail/ViewSVN)
    by lebedevri
  12. [NFC][InstSimplify] Add extra test for D67411 with @llvm.assume (detail/ViewSVN)
    by lebedevri
  13. [scudo][standalone] Android related improvements

    Summary:
    This changes a few things to improve memory footprint and performances
    on Android, and fixes a test compilation error:
    - add `stdlib.h` to `wrappers_c_test.cc` to address
      https://bugs.llvm.org/show_bug.cgi?id=42810
    - change Android size class maps, based on benchmarks, to improve
      performances and lower the Svelte memory footprint. Also change the
      32-bit region size for said configuration
    - change the `reallocate` logic to reallocate in place for sizes larger
      than the original chunk size, when they still fit in the same block.
      This addresses patterns from `memory_replay` dumps like the following:
    ```
    202: realloc 0xb48fd000 0xb4930650 12352
    202: realloc 0xb48fd000 0xb48fd000 12420
    202: realloc 0xb48fd000 0xb48fd000 12492
    202: realloc 0xb48fd000 0xb48fd000 12564
    202: realloc 0xb48fd000 0xb48fd000 12636
    202: realloc 0xb48fd000 0xb48fd000 12708
    202: realloc 0xb48fd000 0xb48fd000 12780
    202: realloc 0xb48fd000 0xb48fd000 12852
    202: realloc 0xb48fd000 0xb48fd000 12924
    202: realloc 0xb48fd000 0xb48fd000 12996
    202: realloc 0xb48fd000 0xb48fd000 13068
    202: realloc 0xb48fd000 0xb48fd000 13140
    202: realloc 0xb48fd000 0xb48fd000 13212
    202: realloc 0xb48fd000 0xb48fd000 13284
    202: realloc 0xb48fd000 0xb48fd000 13356
    202: realloc 0xb48fd000 0xb48fd000 13428
    202: realloc 0xb48fd000 0xb48fd000 13500
    202: realloc 0xb48fd000 0xb48fd000 13572
    202: realloc 0xb48fd000 0xb48fd000 13644
    202: realloc 0xb48fd000 0xb48fd000 13716
    202: realloc 0xb48fd000 0xb48fd000 13788
    ...
    ```
      In this situation we were deallocating the old chunk, and
      allocating a new one for every single one of those, but now we can
      keep the same chunk (we just updated the header), which saves some
      heap operations.

    Reviewers: hctim, morehouse, vitalybuka, eugenis, cferris, rengolin

    Reviewed By: morehouse

    Subscribers: srhines, delcypher, #sanitizers, llvm-commits

    Tags: #llvm, #sanitizers

    Differential Revision: https://reviews.llvm.org/D67293 (detail/ViewSVN)
    by cryptoad
  14. [OPENMP]Updated status page, NFC. (detail/ViewSVN)
    by abataev
  15. gn build: Merge r371562 (detail/ViewSVN)
    by nico
  16. LLVM: Optimization Pass: Remove conflicting attribute, if any, before
    adding new read attribute to an argument
    Summary: Update optimization pass to prevent adding read-attribute to an
    argument without removing its conflicting attribute.

    A read attribute, based on the result of the attribute deduction
    process, might be added to an argument. The attribute might be in
    conflict with other read/write attribute currently associated with the
    argument. To ensure the compatibility of attributes, conflicting
    attribute, if any, must be removed before a new one is added.

    The following snippet shows the current behavior of the compiler, where
    the compilation process is aborted due to incompatible attributes.

    $ cat x.ll
    ; ModuleID = 'x.bc'

    %_type_of_d-ccc = type <{ i8*, i8, i8, i8, i8 }>

    @d-ccc = internal global %_type_of_d-ccc <{ i8* null, i8 1, i8 13, i8 0,
    i8 -127 }>, align 8

    define void @foo(i32* writeonly %.aaa) {
    foo_entry:
      %_param_.aaa = alloca i32*, align 8
      store i32* %.aaa, i32** %_param_.aaa, align 8
      store i8 0, i8* getelementptr inbounds (%_type_of_d-ccc,
    %_type_of_d-ccc* @d-ccc, i32 0, i32 3)
      ret void
    }

    $ opt -O3 x.ll
    Attributes 'readnone and writeonly' are incompatible!
    void (i32*)* @foo
    in function foo
    LLVM ERROR: Broken function found, compilation aborted!
    The purpose of this changeset is to fix the above error. This fix is
    based on a suggestion from Johannes @jdoerfert (many thanks!!!)
    Authored By: anhtuyen
    Reviewer: nicholas, rnk, chandlerc, jdoerfert
    Reviewed By: rnk
    Subscribers: hiraditya, jdoerfert, llvm-commits, anhtuyen, LLVM
    Tag: LLVM
    Differential Revision: https://reviews.llvm.org/D58694 (detail/ViewSVN)
    by whitneyt
  17. [ConstProp] add tests for fma that produce NaN; NFC (detail/ViewSVN)
    by spatel
  18. [libFuzzer] Make -merge=1 to reuse coverage information from the control file.

    Summary:
    This change allows to perform corpus merging in two steps. This is useful when
    the user wants to address the following two points simultaneously:

    1) Get trustworthy incremental stats for the coverage and corpus size changes
        when adding new corpus units.
    2) Make sure the shorter units will be preferred when two or more units give the
        same unique signal (equivalent to the `REDUCE` logic).

    This solution was brainstormed together with @kcc, hopefully it looks good to
    the other people too. The proposed use case scenario:

    1) We have a `fuzz_target` binary and `existing_corpus` directory.
    2) We do fuzzing and write new units into the `new_corpus` directory.
    3) We want to merge the new corpus into the existing corpus and satisfy the
        points mentioned above.
    4) We create an empty directory `merged_corpus` and run the first merge step:

        `
        ./fuzz_target -merge=1 -merge_control_file=MCF ./merged_corpus ./existing_corpus
        `

        this provides the initial stats for `existing_corpus`, e.g. from the output:

        `
        MERGE-OUTER: 3 new files with 11 new features added; 11 new coverage edges
        `

    5) We recreate `merged_corpus` directory and run the second merge step:

        `
        ./fuzz_target -merge=1 -merge_control_file=MCF ./merged_corpus ./existing_corpus ./new_corpus
        `

        this provides the final stats for the merged corpus, e.g. from the output:

        `
        MERGE-OUTER: 6 new files with 14 new features added; 14 new coverage edges
        `

    Alternative solutions to this approach are:

    A) Store precise coverage information for every unit (not only unique signal).
    B) Execute the same two steps without reusing the control file.

    Either of these would be suboptimal as it would impose an extra disk or CPU load
    respectively, which is bad given the quadratic complexity in the worst case.

    Tested on Linux, Mac, Windows.

    Reviewers: morehouse, metzman, hctim, kcc

    Reviewed By: morehouse

    Subscribers: JDevlieghere, delcypher, mgrang, #sanitizers, llvm-commits, kcc

    Tags: #llvm, #sanitizers

    Differential Revision: https://reviews.llvm.org/D66107 (detail/ViewSVN)
    by dor1s
  19. [ConstProp] move test file from InstSimplify; NFC

    These are constant folding tests; there is no code
    directly in InstSimplify for this. (detail/ViewSVN)
    by spatel
  20. [InstSimplify] regenerate test CHECKs; NFC (detail/ViewSVN)
    by spatel
  21. [Alignment][NFC] use llvm::Align for AsmPrinter::EmitAlignment

    Summary:
    This is patch is part of a series to introduce an Alignment type.
    See this thread for context: http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html
    See this patch for the introduction of the type: https://reviews.llvm.org/D64790

    Reviewers: courbet

    Subscribers: dschuff, sdardis, nemanjai, hiraditya, kbarton, jrtc27, MaskRay, atanasyan, jsji, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D67443 (detail/ViewSVN)
    by gchatelet
  22. [LangRef] add link for fma intrinsic (detail/ViewSVN)
    by spatel
  23. [LangRef] fix punctuation; NFC (detail/ViewSVN)
    by spatel
  24. gn build: add include_dir that's necessary after r371564 (detail/ViewSVN)
    by nico
  25. [InstCombine] fold sign-bit compares of srem

    (srem X, pow2C) sgt/slt 0 can be reduced using bit hacks by masking
    off the sign bit and the module (low) bits:
    https://rise4fun.com/Alive/jSO
    A '2' divisor allows slightly more folding:
    https://rise4fun.com/Alive/tDBM

    Any chance to remove an 'srem' use is probably worthwhile, but this is limited
    to the one-use improvement case because doing more may expose other missing
    folds. That means it does nothing for PR21929 yet:
    https://bugs.llvm.org/show_bug.cgi?id=21929

    Differential Revision: https://reviews.llvm.org/D67334 (detail/ViewSVN)
    by spatel
  26. [Alignment] Use llvm::Align in MachineFunction and TargetLowering - fixes mir parsing

    Summary:
    This catches malformed mir files which specify alignment as log2 instead of pow2.
    See https://reviews.llvm.org/D65945 for reference,

    This is patch is part of a series to introduce an Alignment type.
    See this thread for context: http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html
    See this patch for the introduction of the type: https://reviews.llvm.org/D64790

    Reviewers: courbet

    Subscribers: MatzeB, qcolombet, dschuff, arsenm, sdardis, nemanjai, jvesely, nhaehnle, hiraditya, kbarton, asb, rbar, johnrusso, simoncook, apazos, sabuasal, niosHD, jrtc27, MaskRay, zzheng, edward-jones, atanasyan, rogfer01, MartinMosbeck, brucehoult, the_o, PkmX, jocewei, jsji, Petar.Avramovic, asbirlea, s.egerton, pzheng, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D67433 (detail/ViewSVN)
    by gchatelet
  27. [mips][msa] Fix infinite loop for mips.nori.b intrinsic

    When value of immediate in `mips.nori.b` is 255 (which has all ones in
    binary form as 8bit integer) DAGCombiner and Legalizer would fall in an
    infinite loop. DAGCombiner would try to simplify `or %value, -1` by
    turning `%value` into UNDEF. Legalizer will turn it back into `Constant<0>`
    which would then be again turned into UNDEF by DAGCombiner. To avoid this
    loop we make UNDEF legal for MSA int types on Mips.

    Patch by Mirko Brkusanin.

    Differential Revision: https://reviews.llvm.org/D67280 (detail/ViewSVN)
    by atanasyan
  28. Fix -Wdocumentation warning - void function doesn't need a @returns. NFCI. (detail/ViewSVN)
    by rksimon
  29. [Diagnostics] Add -Wsizeof-array-div

    Summary: Clang version of https://www.viva64.com/en/examples/v706/

    Reviewers: rsmith

    Differential Revision: https://reviews.llvm.org/D67287 (detail/ViewSVN)
    by xbolva00
  30. [NFC] Updated objsize-64.ll test (detail/ViewSVN)
    by xbolva00
  31. [NFC] Fixed test (detail/ViewSVN)
    by xbolva00
  32. [InstCombine] Fixed handling of isOpNewLike (PR11748) (detail/ViewSVN)
    by xbolva00
  33. [LoopInterchange] Drop unused splitInnerLoopHeader declaration. (detail/ViewSVN)
    by fhahn
  34. Revert "clang-misexpect: Profile Guided Validation of Performance Annotations in LLVM"

    This reverts commit r371584. It introduced a dependency from compiler-rt
    to llvm/include/ADT, which is problematic for multiple reasons.

    One is that it is a novel dependency edge, which needs cross-compliation
    machinery for llvm/include/ADT (yes, it is true that right now
    compiler-rt included only header-only libraries, however, if we allow
    compiler-rt to depend on anything from ADT, other libraries will
    eventually get used).

    Secondly, depending on ADT from compiler-rt exposes ADT symbols from
    compiler-rt, which would cause ODR violations when Clang is built with
    the profile library. (detail/ViewSVN)
    by gribozavr
  35. [ARM] Take into account -mcpu and -mfpu options while handling 'crypto' feature

    Submittin in behalf of krisb (Kristina Bessonova) <ch.bessonova@gmail.com>

    Summary:
    '+crypto' means '+aes' and '+sha2' for arch >= ARMv8 when they were
    not disabled explicitly. But this is correctly handled only in case of
    '-march' option, though the feature may also be specified through
    the '-mcpu' or '-mfpu' options. In the following example:

      $ clang -mcpu=cortex-a57 -mfpu=crypto-neon-fp-armv8

    'aes' and 'sha2' are disabled that is quite unexpected:

      $ clang -cc1 -triple armv8--- -target-cpu cortex-a57
        <...> -target-feature -sha2 -target-feature -aes -target-feature +crypto

    This exposed by https://reviews.llvm.org/D63936 that makes
    the 'aes' and 'sha2' features disabled by default.

    So, while handling the 'crypto' feature we need to take into account:
      - a CPU name, as it provides the information about architecture
        (if no '-march' option specified),
      - features, specified by the '-mcpu' and '-mfpu' options.

    Reviewers: SjoerdMeijer, ostannard, labrinea, dnsampaio

    Reviewed By: dnsampaio

    Subscribers: ikudrin, javed.absar, kristof.beyls, cfe-commits

    Tags: #clang

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

    Author: krisb (detail/ViewSVN)
    by dnsampaio
  36. [LoopInterchange] Properly move condition, induction increment and ops to latch.

    Currently we only rely on the induction increment to come before the
    condition to ensure the required instructions get moved to the new
    latch.

    This patch duplicates and moves the required instructions to the
    newly created latch. We move the condition to the end of the new block,
    then process its operands. We stop at operands that are defined
    outside the loop, or are the induction PHI.

    We duplicate the instructions and update the uses in the moved
    instructions, to ensure other users remain intact. See the added
    test2 for such an example.

    Reviewers: efriedma, mcrosier

    Reviewed By: efriedma

    Differential Revision: https://reviews.llvm.org/D67367 (detail/ViewSVN)
    by fhahn
  37. [NFC][ARM] Add and modify tests

    Add test for ParallelDSP. (detail/ViewSVN)
    by sam_parker
  38. [LTO] Avoid calling GlobalValue::getGUID (MD5) twice (detail/ViewSVN)
    by maskray
  39. [TLI][AMDGPU] AMDPAL does not have library functions

    Configure TLI to say that r600/amdgpu does not have any library
    functions, such that InstCombine does not do anything like turn sin/cos
    into the library function @tan with sufficient fast math flags.

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

    Change-Id: I02f907d3e64832117ea9800e9f9285282856e5df (detail/ViewSVN)
    by tpr
  40. [llvm-objcopy] Simplify --prefix-alloc-sections

    Handle --prefix-alloc-sections after --rename-sections so that --prefix-alloc-sections code
    does not have to check if renaming has been performed.

    Reviewed By: jhenderson

    Differential Revision: https://reviews.llvm.org/D66931 (detail/ViewSVN)
    by maskray
  41. [Attributor] Implement "noalias" callsite argument deduction

    Summary: Now, `nocapture` is deduced in Attributor therefore, this patch introduces deduction for `noalias` callsite argument using `nocapture`.

    Reviewers: jdoerfert, sstefan1

    Reviewed By: jdoerfert

    Subscribers: lebedev.ri, hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D67286 (detail/ViewSVN)
    by uenoku
  42. [Attributor][Fix] Manifest nocapture only in CSArgument or Argument

    Summary:
    We can query to Attributor whether the value is captured in the scope or not on the following way:

    ```
        const auto & NoCapAA = A.getAAFor<AANoCapture>(*this, IRPosition::value(V));
    ```
    And if V is CallSiteReturned then `getDeducedAttribute` will add `nocatpure` to the callsite returned value. It is not valid.
    This patch checks the position is an argument or call site argument.

    This is tested in D67286.

    Reviewers: jdoerfert, sstefan1

    Reviewed By: jdoerfert

    Subscribers: hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D67342 (detail/ViewSVN)
    by uenoku
  43. [Debuginfo][Instcombiner] Do not clone dbg.declare.

    TryToSinkInstruction() has a bug: While updating debug info for
    sunk instruction, it could clone dbg.declare intrinsic.
    That is wrong. There could be only one dbg.declare.
    The fix is to not clone dbg.declare intrinsic and to update
    it`s arguments, to not to point to sunk instruction.

    Differential Revision: https://reviews.llvm.org/D67217 (detail/ViewSVN)
    by avl
  44. [CodeGen] Add alias for cpu_dispatch function with IFunc & Fix resolver linkage type

    Multi-versioned functions defined by cpu_dispatch and implemented with IFunc
    can not be called outside the translation units where they are defined due to
    lack of symbols. This patch add function aliases for these functions and thus
    make them visible outside.

    Differential Revision: https://reviews.llvm.org/D67058
    Patch by Senran Zhang (detail/ViewSVN)
    by maskray
  45. [llvm-nm] Fix -DBUILD_SHARED_LIBS=ON builds after D66160/r371576 (detail/ViewSVN)
    by maskray
  46. clang-misexpect: Profile Guided Validation of Performance Annotations in LLVM

    This patch contains the basic functionality for reporting potentially
    incorrect usage of __builtin_expect() by comparing the developer's
    annotation against a collected PGO profile. A more detailed proposal and
    discussion appears on the CFE-dev mailing list
    (http://lists.llvm.org/pipermail/cfe-dev/2019-July/062971.html) and a
    prototype of the initial frontend changes appear here in D65300

    We revised the work in D65300 by moving the misexpect check into the
    LLVM backend, and adding support for IR and sampling based profiles, in
    addition to frontend instrumentation.

    We add new misexpect metadata tags to those instructions directly
    influenced by the llvm.expect intrinsic (branch, switch, and select)
    when lowering the intrinsics. The misexpect metadata contains
    information about the expected target of the intrinsic so that we can
    check against the correct PGO counter when emitting diagnostics, and the
    compiler's values for the LikelyBranchWeight and UnlikelyBranchWeight.
    We use these branch weight values to determine when to emit the
    diagnostic to the user.

    A future patch should address the comment at the top of
    LowerExpectIntrisic.cpp to hoist the LikelyBranchWeight and
    UnlikelyBranchWeight values into a shared space that can be accessed
    outside of the LowerExpectIntrinsic pass. Once that is done, the
    misexpect metadata can be updated to be smaller.

    In the long term, it is possible to reconstruct portions of the
    misexpect metadata from the existing profile data. However, we have
    avoided this to keep the code simple, and because some kind of metadata
    tag will be required to identify which branch/switch/select instructions
    are influenced by the use of llvm.expect

    Patch By: paulkirth
    Differential Revision: https://reviews.llvm.org/D66324 (detail/ViewSVN)
    by phosek
  47. Emit -Wmicrosoft-enum-value warning instead of error in MS ABI

    Summary:
    The first NFC change is to replace a getCXXABI().isMicrosoft() check
    with getTriple().isWindowsMSVCEnvironment(). This code takes effect in
    non-C++ compilations, so it doesn't make sense to check the C++ ABI. In
    the MS ABI, enums are always considered to be "complete" because the
    underlying type of an unfixed enum will always be 'int'. This behavior
    was moved from -fms-compatibility to MS ABI back in r249656.

    The second change is functional, and it downgrades an error to a warning
    when the MS ABI is used rather than only under -fms-compatibility. The
    reasoning is that it's unreasonable for the following code to reject the
    following code for all MS ABI targets with -fno-ms-compatibility:
      enum Foo { Foo_Val = 0xDEADBEEF };
    This is valid code for any other target, but in the MS ABI, Foo_Val just
    happens to be negative. With this change, clang emits a
    -Wmicrosoft-enum-value warning on this code, but compiles it without
    error.

    Fixes PR38478

    Reviewers: hans, rsmith, STL_MSFT

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D67304 (detail/ViewSVN)
    by rnk
  48. Revert "llvm-reduce: Add pass to reduce parameters"

    Looks to be failing on asan buildbots

    This reverts commit r371567. (detail/ViewSVN)
    by dblaikie
  49. Revert "Adding support for overriding LLVM_ENABLE_RUNTIMES for runtimes builds."

    This reverts commit r371566: this doesn't correctly handle the case
    where LLVM_ENABLE_RUNTIMES contains a list of values. (detail/ViewSVN)
    by phosek

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18305
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18306
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18307
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18308
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18309
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18310
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18311
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18312
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18313
originally caused by:

Started by upstream project Clang Stage 2: cmake, R -g Tsan, using Stage 1 RA build number 18314
originally caused by:

This run spent:

  • 17 hr waiting;
  • 17 hr build duration;
  • 1 day 10 hr total from scheduled to completion.

Identified problems

Regression test failed

This build failed because a regression test in the test suite FAILed. See the test report for details.
Indication 1

Compile Error

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

Ninja target failed

Below is a link to the first failed ninja target.
Indication 3

Missing test results

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