Started 2 days 5 hr ago
Took 38 min on green-dragon-08

Failed Build #5511 (Oct 11, 2019 12:41:28 PM)

  • : 374582
  • : 374581
  • : 374472
  • : 364589
  • : 374510
  • : 374551
  1. llvm-dwarfdump: Add verbose printing for debug_loclists (detail/ViewSVN)
    by dblaikie
  2. [Stats] Convert some ad-hoc header search stats to ALWAYS_ENABLED_STATISTIC.


    Reviewers: dsanders, bogner, rtereshin

    Reviewed By: dsanders

    Subscribers: hiraditya, jkorous, dexonsmith, ributzka, cfe-commits, llvm-commits

    Tags: #llvm

    Differential Revision: (detail/ViewSVN)
    by vsapsai
  3. Suppress false-positive -Wdeprecated-volatile warning from __is_*_assignable(volatile T&, U). (detail/ViewSVN)
    by rsmith
  4. [X86][SSE] Add support for v4i8 add reduction (detail/ViewSVN)
    by rksimon
  5. gn build: (manually) merge r374110 (detail/ViewSVN)
    by nico
  6. [clang][IFS] Fixing assert in clang interface stubs for enums, records, typedefs

    The clang IFS ASTConsumer was asserting on enums, records (struct definitions in
    C), and typedefs. All it needs to do is skip them because the stub just needs to
    expose global object instances and functions.

    Differential Revision: (detail/ViewSVN)
    by zer0
  7. Update clang module map for new excluded .def file. (detail/ViewSVN)
    by rsmith
  8. Fix test failure with 374562 on Hexagon

    __builtin_assume_aligned takes a size_t which is a 32 bit int on
    hexagon.  Thus, the constant gets converted to a 32 bit value, resulting
    in 0 not being a power of 2.  This patch changes the constant being
    passed to 2**30 so that it fails, but doesnt exceed 30 bits. (detail/ViewSVN)
    by erichkeane
  9. [AArch64] add tests for (v)select-of-constants; NFC

    These are copied from existing test files in x86/PPC. (detail/ViewSVN)
    by spatel
  10. [AArch64][SVE] Implement sdot and udot (lane) intrinsics

    Implements the following arithmetic intrinsics:
      - int_aarch64_sve_sdot
      - int_aarch64_sve_sdot_lane
      - int_aarch64_sve_udot
      - int_aarch64_sve_udot_lane

    This patch includes tests for the Subdivide4Argument type added by D67549

    Reviewers: sdesmalen, SjoerdMeijer, greened, rengolin, rovka

    Reviewed By: sdesmalen

    Subscribers: tschuett, kristof.beyls, rkruppe, psnobl, cfe-commits, llvm-commits

    Differential Revision: (detail/ViewSVN)
    by kmclaughlin
  11. [VPlan] Add moveAfter to VPRecipeBase.

    This patch adds a moveAfter method to VPRecipeBase, which can be used to
    move elements after other elements, across VPBasicBlocks, if necessary.

    Reviewers: dcaballe, hsaito, rengolin, hfinkel

    Reviewed By: dcaballe

    Differential Revision: (detail/ViewSVN)
    by fhahn
  12. [AIX] Use .space instead of .zero in assembly

    The AIX system assembler does not understand .zero, so we should prefer
    emitting .space.

    Subscribers: nemanjai, hiraditya, kbarton, MaskRay, jsji, llvm-commits

    Tags: #llvm

    Differential Revision: (detail/ViewSVN)
    by daltenty
  13. Reland r374450 with Richard Smith's comments and test fixed.

    The behavior from the original patch has changed, since we're no longer
    allowing LLVM to just ignore the alignment.  Instead, we're just
    assuming the maximum possible alignment.

    Differential Revision: (detail/ViewSVN)
    by erichkeane
  14. [AMDGPU][MC][GFX9][GFX10] Corrected number of src operands for ds_[read/write]_addtid_b32


    Reviewers: arsenm, rampitec

    Differential Revision: (detail/ViewSVN)
    by dpreobra
  15. gn build: Merge r374558 (detail/ViewSVN)
    by gnsyncbot
  16. [AMDGPU][MC][GFX6][GFX7][GFX10] Added instructions buffer_atomic_[fcmpswap/fmin/fmax]*


    Reviewers: arsenm, rampitec

    Differential Revision: (detail/ViewSVN)
    by dpreobra
  17. [libTooling] Move `RewriteRule` abstraction into its own header and impl.

    Summary: Move the `RewriteRule` class and related declarations into its own set
    of files (header, implementation). Only the `Transformer` class is left in the
    Transformer-named files. This change clarifies the distinction between the
    `RewriteRule` class, which is essential to the Transformer library, and the
    `Transformer` class, which is only one possible `RewriteRule` interpreter
    (compare to `TransformerClangTidyCheck`, a clang-tidy based interpreter).

    Reviewers: gribozavr

    Subscribers: jfb, cfe-commits

    Tags: #clang

    Differential Revision: (detail/ViewSVN)
    by ymandel
  18. [AMDGPU][MC][GFX10] Enabled null for 64-bit dst operands


    Reviewers: arsenm, rampitec

    Differential Revision: (detail/ViewSVN)
    by dpreobra
  19. [llvm] [ocaml] Support linking against dylib

    Support linking OCaml modules against LLVM dylib when requested,
    rather than against static libs that might not be installed at all.

    Differential Revision: (detail/ViewSVN)
    by mgorny
  20. [DAGCombiner] fold vselect-of-constants to shift

    The diffs suggest that we are missing some more basic
    analysis/transforms, but this keeps the vector path in
    sync with the scalar (rL374397). This is again a
    preliminary step for introducing the reverse transform
    in IR as proposed in D63382. (detail/ViewSVN)
    by spatel
  21. Fix compilation warnings. NFC. (detail/ViewSVN)
    by hliao
  22. [AMDGPU][MC] Corrected parsing of optional operands


    Reviewers: artem.tamazov, arsenm

    Differential Revision: (detail/ViewSVN)
    by dpreobra
  23. [libTooling] Change Stencil equality to use `toString()`

    Removes the `isEqual` method from StencilPartInterface and modifies equality to
    use the string representation returned by the `toString` method for comparison.

    This means the `run` and `selection` stencils return true by default, and
    clients should be cautious in relying on equality operator for comparison of
    stencils containing parts generated by these functions.

    It also means we no longer need the custom RTTI support (typeId() and
    down_cast()), so it has been removed.

    Patch by Harshal T. Lehri.

    Reviewers: gribozavr

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: (detail/ViewSVN)
    by ymandel
  24. Updated to create checker tests in the new directory (detail/ViewSVN)
    by gribozavr
  25. [ClangTidy] Separate tests for infrastructure and checkers, fixup

    Renamed a file that I missed in r374540. (detail/ViewSVN)
    by gribozavr
  26. [mips] Follow-up to r374544. Fix test case. (detail/ViewSVN)
    by atanasyan
  27. [Tests] Output of od can be lower or upper case (llvm-objcopy/yaml2obj).

    The command `od -t x` is used to dump data in hex format.
    The LIT tests assumes that the hex characters are in lowercase.
    However, there are also platforms which use uppercase letter.

    To solve this issue the tests are updated to use the new
    `--ignore-case` option of FileCheck.

    Reviewers: Bigcheese, jakehehrlich, rupprecht, espindola, alexshap, jhenderson

    Differential Revision: (detail/ViewSVN)
    by redstar
  28. [mips] Fix loading "double" immediate into a GPR and FPR

    If a "double" (64-bit) value has zero low 32-bits, it's possible to load
    such value into a GP/FP registers as an instruction immediate. But now
    assembler loads only high 32-bits of the value.

    For example, if a target register is GPR the `li.d $4, 1.0` instruction
    converts into the `lui $4, 16368` one. As a result, we get `0x3FF00000`
    in the register. While a correct representation of the `1.0` value is
    `0x3FF0000000000000`. The patch fixes that.

    Differential Revision: (detail/ViewSVN)
    by atanasyan
  29. [MS ABI]: Fix mangling function arguments for template types to be compatible with MSVC

    MS name mangling supports cache for first 10 distinct function
    arguments.  The error was when non cached template type occurred twice
    (e.g. 11th and 12th).  For such case in code there is another cache
    table TemplateArgStrings (for performance reasons).  Then one '@'
    character at the end of the mangled name taken from this table was
    missing.  For other cases the missing '@' character was added in
    the call to mangleSourceName(TemplateMangling) in the cache miss code,
    but the cache hit code didn't add it.

    This fixes a regression from r362560.

    Patch by Adam Folwarczny <>!

    Differential Revision: (detail/ViewSVN)
    by nico
  30. [llvm-readobj] - Remove excessive fields when dumping "Version symbols".

    This removes a few fields that are not useful:
    "Section Name", "Address", "Offset" and "Link"
    (they duplicated the information available under
    the "Sections [" tag).

    Differential revision: (detail/ViewSVN)
    by grimar
  31. [ClangTidy] Separate tests for infrastructure and checkers

    This change moves tests for checkers and infrastructure into separate
    directories, making it easier to find infrastructure tests. Tests for
    checkers are already easy to find because they are named after the
    checker. Tests for infrastructure were difficult to find because they
    were outnumbered by tests for checkers. Now they are in a separate

    Reviewers: jfb, jdoerfert, lebedev.ri

    Subscribers: srhines, nemanjai, aheejin, kbarton, christof, mgrang, arphaman, jfb, lebedev.ri, cfe-commits

    Tags: #clang

    Differential Revision: (detail/ViewSVN)
    by gribozavr
  32. Dead Virtual Function Elimination

    Currently, it is hard for the compiler to remove unused C++ virtual
    functions, because they are all referenced from vtables, which are referenced
    by constructors. This means that if the constructor is called from any live
    code, then we keep every virtual function in the final link, even if there
    are no call sites which can use it.

    This patch allows unused virtual functions to be removed during LTO (and
    regular compilation in limited circumstances) by using type metadata to match
    virtual function call sites to the vtable slots they might load from. This
    information can then be used in the global dead code elimination pass instead
    of the references from vtables to virtual functions, to more accurately
    determine which functions are reachable.

    To make this transformation safe, I have changed clang's code-generation to
    always load virtual function pointers using the llvm.type.checked.load
    intrinsic, instead of regular load instructions. I originally tried writing
    this using clang's existing code-generation, which uses the llvm.type.test
    and llvm.assume intrinsics after doing a normal load. However, it is possible
    for optimisations to obscure the relationship between the GEP, load and
    llvm.type.test, causing GlobalDCE to fail to find virtual function call

    The existing linkage and visibility types don't accurately describe the scope
    in which a virtual call could be made which uses a given vtable. This is
    wider than the visibility of the type itself, because a virtual function call
    could be made using a more-visible base class. I've added a new
    !vcall_visibility metadata type to represent this, described in
    TypeMetadata.rst. The internalization pass and libLTO have been updated to
    change this metadata when linking is performed.

    This doesn't currently work with ThinLTO, because it needs to see every call
    to llvm.type.checked.load in the linkage unit. It might be possible to
    extend this optimisation to be able to use the ThinLTO summary, as was done
    for devirtualization, but until then that combination is rejected in the
    clang driver.

    To test this, I've written a fuzzer which generates random C++ programs with
    complex class inheritance graphs, and virtual functions called through object
    and function pointers of different types. The programs are spread across
    multiple translation units and DSOs to test the different visibility

    I've also tried doing bootstrap builds of LLVM to test this. This isn't
    ideal, because only classes in anonymous namespaces can be optimised with
    -fvisibility=default, and some parts of LLVM (plugins and bugpoint) do not
    work correctly with -fvisibility=hidden. However, there are only 12 test
    failures when building with -fvisibility=hidden (and an unmodified compiler),
    and this change does not cause any new failures for either value of

    On the 7 C++ sub-benchmarks of SPEC2006, this gives a geomean code-size
    reduction of ~6%, over a baseline compiled with "-O2 -flto
    -fvisibility=hidden -fwhole-program-vtables". The best cases are reductions
    of ~14% in 450.soplex and 483.xalancbmk, and there are no code size

    I've also run this on a set of 8 mbed-os examples compiled for Armv7M, which
    show a geomean size reduction of ~3%, again with no size increases.

    I had hoped that this would have no effect on performance, which would allow
    it to awlays be enabled (when using -fwhole-program-vtables). However, the
    changes in clang to use the llvm.type.checked.load intrinsic are causing ~1%
    performance regression in the C++ parts of SPEC2006. It should be possible to
    recover some of this perf loss by teaching optimisations about the
    llvm.type.checked.load intrinsic, which would make it worth turning this on
    by default (though it's still dependent on -fwhole-program-vtables).

    Differential revision: (detail/ViewSVN)
    by ostannard
  33. [FileCheck] Implement --ignore-case option.

    The FileCheck utility is enhanced to support a `--ignore-case`
    option. This is useful in cases where the output of Unix tools
    differs in case (e.g. case not specified by Posix).

    Reviewers: Bigcheese, jakehehrlich, rupprecht, espindola, alexshap, jhenderson, MaskRay

    Differential Revision: (detail/ViewSVN)
    by redstar
  34. [SCEV] Add stricter verification option.

    Currently -verify-scev only fails if there is a constant difference
    between two BE counts. This misses a lot of cases.

    This patch adds a -verify-scev-strict options, which fails for any
    non-zero differences, if used together with -verify-scev.

    With the stricter checking, some unit tests fail because
    of mis-matches, especially around IndVarSimplify.

    If there is no reason I am missing for just checking constant deltas, I
    am planning on looking into the various failures.

    Reviewers: efriedma,, reames, atrick

    Reviewed By:

    Differential Revision: (detail/ViewSVN)
    by fhahn
  35. [X86] isFNEG - add recursion depth limit

    Now that its used by isNegatibleForFree we should try to avoid costly deep recursion (detail/ViewSVN)
    by rksimon
  36. [llvm-exegesis] Show noise cluster in analysis output.

    Reviewers: gchatelet

    Subscribers: tschuett, llvm-commits

    Tags: #llvm

    Differential Revision: (detail/ViewSVN)
    by courbet
  37. [Windows] Use information from the PE32 exceptions directory to construct unwind plans

    This patch adds an implementation of unwinding using PE EH info. It allows to
    get almost ideal call stacks on 64-bit Windows systems (except some epilogue
    cases, but I believe that they can be fixed with unwind plan disassembly
    augmentation in the future).

    To achieve the goal the CallFrameInfo abstraction was made. It is based on the
    DWARFCallFrameInfo class interface with a few changes to make it less

    To implement the new interface for PECOFF object files the class PECallFrameInfo
    was written. It uses the next helper classes:

    - UnwindCodesIterator helps to iterate through UnwindCode structures (and
      processes chained infos transparently);
    - EHProgramBuilder with the use of UnwindCodesIterator constructs EHProgram;
    - EHProgram is, by fact, a vector of EHInstructions. It creates an abstraction
      over the low-level unwind codes and simplifies work with them. It contains
      only the information that is relevant to unwinding in the unified form. Also
      the required unwind codes are read from the object file only once with it;
    - EHProgramRange allows to take a range of EHProgram and to build an unwind row
      for it.

    So, PECallFrameInfo builds the EHProgram with EHProgramBuilder, takes the ranges
    corresponding to every offset in prologue and builds the rows of the resulted
    unwind plan. The resulted plan covers the whole range of the function except the

    Reviewers: jasonmolenda, asmith, amccarth, clayborg, JDevlieghere, stella.stamenova, labath, espindola

    Reviewed By: jasonmolenda

    Subscribers: leonid.mashinskiy, emaste, mgorny, aprantl, arichardson, MaskRay, lldb-commits, llvm-commits

    Tags: #lldb

    Differential Revision: (detail/ViewSVN)
    by aleksandr.urakov
  38. Insert module constructors in a module pass

    If we insert them from function pass some analysis may be missing or invalid.
    Fixes PR42877.

    Reviewers: eugenis, leonardchan

    Reviewed By: leonardchan

    Subscribers: hiraditya, cfe-commits, llvm-commits

    Tags: #clang, #llvm

    Differential Revision:

    llvm-svn: 374481
    Signed-off-by: Vitaly Buka <> (detail/ViewSVN)
    by Vitaly Buka
  39. [TableGen] Fix a bug that MCSchedClassDesc is interfered between different SchedModel

    Assume that, ModelA has scheduling resource for InstA and ModelB has scheduling resource for InstB. This is what the llvm::MCSchedClassDesc looks like:

    llvm::MCSchedClassDesc ModelASchedClasses[] = {
    InstA, 0, ...
    InstB, -1,...

    llvm::MCSchedClassDesc ModelBSchedClasses[] = {
    InstA, -1,...
    InstB, 0,...
    The -1 means invalid num of macro ops, while it is valid if it is >=0. This is what we look like now:

    llvm::MCSchedClassDesc ModelASchedClasses[] = {
    InstA, 0, ...
    InstB, 0,...

    llvm::MCSchedClassDesc ModelBSchedClasses[] = {
    InstA, 0,...
    InstB, 0,...
    And compiler hit the assertion here because the SCDesc is valid now for both InstA and InstB.

    Differential Revision: (detail/ViewSVN)
    by qshanz
  40. [X86] Add v8i64->v8i8 ssat/usat/packus truncate tests to min-legal-vector-width.ll

    I wonder if we should split the v8i8 stores in order to form
    two v4i8 saturating truncating stores. This would remove the
    unpckl needed to concatenated the v4i8 results to make a
    single store. (detail/ViewSVN)
    by ctopper
  41. [ADT][Statistics] Fix test after rL374490 (detail/ViewSVN)
    by kadircet
  42. Fix modules build for r374337

    A modules build failed with the following error:
      call to function 'operator&' that is neither visible in the template definition nor found by argument-dependent lookup

    Fix that by declaring the appropriate operators in the llvm::minidump
    namespace. (detail/ViewSVN)
    by labath
  43. [X86] Always define the tzcnt intrinsics even when _MSC_VER is defined.

    These intrinsics use llvm.cttz intrinsics so are always available
    even without the bmi feature. We already don't check for the bmi
    feature on the intrinsics themselves. But we were blocking the
    include of the header file with _MSC_VER unless BMI was enabled
    on the command line.

    Fixes PR30506. (detail/ViewSVN)
    by ctopper
  44. [PowerPC] Remove assertion "Shouldn't overwrite a register before it is killed"

    The assertion is everzealous and fail tests like:

      renamable $x3 = LI8 0
      STD renamable $x3, 16, $x1
      renamable $x3 = LI8 0

    Remove the assertion since killed flag of $x3 is not mandentory.

    Differential Revision: (detail/ViewSVN)
    by yi-hong.lyu
  45. [NFC] run specific pass instead of whole -O3 pipeline for popcount recoginzation testcase. (detail/ViewSVN)
    by shchenz
  46. [InstCombine] recognize popcount.

      This patch recognizes popcount intrinsic according to algorithm from website

    Differential Revision: (detail/ViewSVN)
    by shchenz
  47. [libc++] Fix linker script generation

    Handle the case when libc++abi and libunwind are being built together
    with libc++ in the runtimes build. This logic was used in the previous
    implementation but dropped in r374116.

    Differential Revision: (detail/ViewSVN)
    by phosek
  48. [X86] Add a DAG combine to turn v16i16->v16i8 VTRUNCUS+store into a saturating truncating store. (detail/ViewSVN)
    by ctopper
  49. [X86] Add test case for trunc_packus_v16i32_v16i8_store to min-legal-vector-width.ll

    We aren't folding the vpmovuswb into the store. (detail/ViewSVN)
    by ctopper
  50. [CVP] Remove a masking operation if range information implies it's a noop

    This is really a known bits style transformation, but known bits isn't context sensitive. The particular case which comes up happens to involve a range which allows range based reasoning to eliminate the mask pattern, so handle that case specifically in CVP.

    InstCombine likes to generate the mask-by-low-bits pattern when widening an arithmetic expression which includes a zext in the middle.

    Differential Revision: (detail/ViewSVN)
    by reames
  51. [X86] Add more packus/ssat/usat truncate tests from legal vectors to less than 128-bit vectors.

    Some of these have sub-optimal codegen for avx512 relative to avx2. (detail/ViewSVN)
    by ctopper
  52. Revert 374481 "[tsan,msan] Insert module constructors in a module pass"

    CodeGen/sanitizer-module-constructor.c fails on mac and windows, see e.g. (detail/ViewSVN)
    by nico
  53. [CUDA][HIP} Add a test for constexpr default ctor

    Differential Revision: (detail/ViewSVN)
    by yaxunl
  54. [JITLink] Disable the MachO/AArch64 testcase while investigating bot failures.

    The windows bots are failing due to a memory layout error. Temporarily disabling
    while I investigate whether this can be worked around, or whether the test
    should be disabled on Windows. (detail/ViewSVN)
    by Lang Hames
  55. [JITLink] Fix MachO/arm64 GOTPAGEOFF encoding.

    The original implementation failed to shift the immediate down.

    This should fix some of the bot failures due to r374476. (detail/ViewSVN)
    by Lang Hames
  56. [Attributor][FIX] Do not replace musstail calls with constant (detail/ViewSVN)
    by jdoerfert
  57. Fix assertion failure for a cv-qualified array as a non-type template
    parameter type.

    We were both failing to decay the array type to a pointer and failing to
    remove the top-level cv-qualifications. Fix this by decaying array
    parameters even if the parameter type is dependent. (detail/ViewSVN)
    by rsmith
  58. AMDGPU: Move SelectFlatOffset back into AMDGPUISelDAGToDAG (detail/ViewSVN)
    by arsenm
  59. Get ClangdXPC.framework building (barely) with CMake's Xcode generator

    The output directories for CMake's Xcode project generator are
    specific to the configuration, and so looking in
    CMAKE_LIBRARY_OUTPUT_DIRECTORY isn't going to work. Fortunately, CMake
    already provides generator expressions to find the output of a given

    I call this "barely" building because the built framework isn't going
    to respect the configuration; that is, I can't have both Debug and
    RelWithDebInfo variants of ClangdXPC.framework at the same time like I
    can with normal library or executable targets. To do that we'd have to
    put the framework in a configuration-specific output directory or use
    CMake's native support for frameworks instead. (detail/ViewSVN)
    by jrose
  60. [Stats] Add ALWAYS_ENABLED_STATISTIC enabled regardless of LLVM_ENABLE_STATS.

    The intended usage is to measure relatively expensive operations. So the
    cost of the statistic is negligible compared to the cost of a measured
    operation and can be enabled all the time without impairing the
    compilation time.


    Reviewers: dsanders, bogner, rtereshin

    Reviewed By: dsanders

    Subscribers: hiraditya, jkorous, dexonsmith, ributzka, cfe-commits, llvm-commits

    Tags: #llvm

    Differential Revision: (detail/ViewSVN)
    by vsapsai
  61. Include whether the destructor is constexpr in -ast-dump output for a
    clss. (detail/ViewSVN)
    by rsmith
  62. [X86] Improve the AVX512 bailout in combineTruncateWithSat to allow pack instructions in more situations.

    If we don't have VLX we won't end up selecting a saturating
    truncate for 256-bit or smaller vectors so we should just use
    the pack lowering. (detail/ViewSVN)
    by ctopper
  63. [X86] Update trunc_packus_v32i32_v32i8 test in min-legal-vector-width.ll to use a load for the large type and add the min-legal-vector-width attribute.

    The attribute is needed to avoid zmm registers. Using memory
    avoids argument splitting for large vectors. (detail/ViewSVN)
    by ctopper
  64. Move most CXXRecordDecl::DefinitionData bit-fields out into a separate

    Reduces duplication and thereby reduces the risk that someone will
    forget to update one of these places, as I did when adding
    DefaultedDestructorIsConstexpr (though I've been unable to produce
    a testcase for which that matters so far). (detail/ViewSVN)
    by rsmith
  65. gn build: Merge r374476 (detail/ViewSVN)
    by gnsyncbot
  66. [tsan,msan] Insert module constructors in a module pass

    If we insert them from function pass some analysis may be missing or invalid.
    Fixes PR42877.

    Reviewers: eugenis, leonardchan

    Reviewed By: leonardchan

    Subscribers: hiraditya, cfe-commits, llvm-commits

    Tags: #clang, #llvm

    Differential Revision: (detail/ViewSVN)
    by Vitaly Buka
  67. [msan, NFC] Move option parsing into constructor (detail/ViewSVN)
    by Vitaly Buka
  68. Fix compilation warning due to typo. (detail/ViewSVN)
    by hliao
  69. [tooling] Fix assertion on MacOSX. (detail/ViewSVN)
    by hliao
  70. [JITLink] Add an initial implementation of JITLink for MachO/AArch64.

    This implementation has support for all relocation types except TLV.

    Compact unwind sections are not yet supported, so exceptions/unwinding will not
    work. (detail/ViewSVN)
    by Lang Hames
  71. [JITLink] Move MachO/x86 got test further down in the data section.

    'named_data' should be the first symbol in the data section. (detail/ViewSVN)
    by Lang Hames
  72. docs/DeveloperPolicy: Add instructions for requesting GitHub commit access

    Subscribers: mehdi_amini, jtony, xbolva00, llvm-commits

    Tags: #llvm

    Differential Revision: (detail/ViewSVN)
    by tstellar
  73. Fix check-interception link error in compiler-rt debug mode (detail/ViewSVN)
    by rnk
  74. [MemorySSA] Update Phi simplification.

    When simplifying a Phi to the unique value found incoming, check that
    there wasn't a Phi already created to break a cycle. If so, remove it.
    Resolves PR43541.

    Some additional nits included. (detail/ViewSVN)
    by asbirlea
  75. [tooling] Teach Tooling to understand compilation with offloading.

    - So far, we only recognize the host compilation with offloading and
      skip the offloading part.

    Reviewers: tra, yaxunl

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: (detail/ViewSVN)
    by hliao
  76. Fix help message for -ffp-contract

    Differential Revision: (detail/ViewSVN)
    by yaxunl
  77. PR43629: Fix crash evaluating constexpr placement new on a subobject of
    an out-of-lifetime object. (detail/ViewSVN)
    by rsmith
  78. [GISel] Simplifying return from else in function. NFC

    Forgot to integrate this little change in previous commit (detail/ViewSVN)
    by mggm
  79. [X86] Guard against leaving a dangling node in combineTruncateWithSat.

    When handling the packus pattern for i32->i8 we do a two step
    process using a packss to i16 followed by a packus to i8. If the
    final i8 step is a type with less than 64-bits the packus step
    will return SDValue(), but the i32->i16 step might have succeeded.
    This leaves the nodes from the middle step dangling.

    Guard against this by pre-checking that the number of elements is
    at least 8 before doing the middle step.

    With that check in place this should mean the only other
    case the middle step itself can fail is when SSE2 is disabled. So
    add an early SSE2 check then just assert that neither the middle
    or final step ever fail. (detail/ViewSVN)
    by ctopper
  80. [X86] Add test cases for packus/ssat/usat 32i32->v32i8 test cases. NFC (detail/ViewSVN)
    by ctopper
  81. [GISel] Allow getConstantVRegVal() to return G_FCONSTANT values.

    In GISel we have both G_CONSTANT and G_FCONSTANT, but because
    in GISel we don't really have a concept of Float vs Int value
    the only difference between the two is where the data originates

    What both G_CONSTANT and G_FCONSTANT return is just a bag of bits
    with the constant representation in it.

    By making getConstantVRegVal() return G_FCONSTANTs bit representation
    as well we allow ConstantFold and other things to operate with

    Adding tests that show ConstantFolding to work on mixed G_CONSTANT
    and G_FCONSTANT sources.

    Differential Revision: (detail/ViewSVN)
    by mggm
  82. Fix some errors in <arm_neon.h> tests that cause them to fail with lax
    vector conversions disabled. (detail/ViewSVN)
    by rsmith
  83. Revert 374450 "Fix __builtin_assume_aligned with too large values."

    The test fails on Windows, with

      error: 'warning' diagnostics expected but not seen:
        File builtin-assume-aligned.c Line 62: requested alignment
            must be 268435456 bytes or smaller; assumption ignored
      error: 'warning' diagnostics seen but not expected:
        File builtin-assume-aligned.c Line 62: requested alignment
            must be 8192 bytes or smaller; assumption ignored (detail/ViewSVN)
    by nico
  84. [AMDGPU] Handle undef old operand in DPP combine

    It was missing an undef flag.

    Differential Revision: (detail/ViewSVN)
    by rampitec
  85. [ValueTracking] Improve pointer offset computation for cases of same base

    This patch improves the handling of pointer offset in GEP expressions where
    one argument is the base pointer. isPointerOffset() is being used by memcpyopt
    where current code synthesizes consecutive 32 bytes stores to one store and
    two memset intrinsic calls. With this patch, we convert the stores to one
    memset intrinsic.

    Differential Revision: (detail/ViewSVN)
    by xur
  86. [InstCombine] Add test case for PR43617 (NFC)

    Also, refactor check in `LibCallSimplifier::optimizeLog()`. (detail/ViewSVN)
    by evandro
  87. [lit] Break main into smaller functions

    This change is purely mechanical.  I will do further cleanups of
    parameter usages.

    Reviewed By: rnk

    Differential Revision: (detail/ViewSVN)
    by yln
  88. Fix __builtin_assume_aligned with too large values.

    Code to handle __builtin_assume_aligned was allowing larger values, but
    would convert this to unsigned along the way. This patch removes the
    EmitAssumeAligned overloads that take unsigned to do away with this

    Additionally, it adds a warning that values greater than 1 <<29 are
    ignored by LLVM.

    Differential Revision: (detail/ViewSVN)
    by erichkeane
  89. Add -fgnuc-version= to control __GNUC__ and other GCC macros

    I noticed that compiling on Windows with -fno-ms-compatibility had the
    side effect of defining __GNUC__, along with __GNUG__, __GXX_RTTI__, and
    a number of other macros for GCC compatibility. This is undesirable and
    causes Chromium to do things like mix __attribute__ and __declspec,
    which doesn't work. We should have a positive language option to enable
    GCC compatibility features so that we can experiment with
    -fno-ms-compatibility on Windows. This change adds -fgnuc-version= to be
    that option.

    My issue aside, users have, for a long time, reported that __GNUC__
    doesn't match their expectations in one way or another. We have
    encouraged users to migrate code away from this macro, but new code
    continues to be written assuming a GCC-only environment. There's really
    nothing we can do to stop that. By adding this flag, we can allow them
    to choose their own adventure with __GNUC__.

    This overlaps a bit with the "GNUMode" language option from -std=gnu*.
    The gnu language mode tends to enable non-conforming behaviors that we'd
    rather not enable by default, but the we want to set things like
    __GXX_RTTI__ by default, so I've kept these separate.

    Helps address PR42817

    Reviewed By: hans, nickdesaulniers, MaskRay

    Differential Revision: (detail/ViewSVN)
    by rnk
  90. Add a missing include in test.

    A fix for r373993. (detail/ViewSVN)
    by eugenis
  91. [MemorySSA] Additional handling of unreachable blocks.

    Whenever we get the previous definition, the assumption is that the
    recursion starts ina  reachable block.
    If the recursion starts in an unreachable block, we may recurse
    indefinitely. Handle this case by returning LoE if the block is

    Resolves PR43426.

    Reviewers: george.burgess.iv

    Subscribers: Prazek,, llvm-commits

    Tags: #llvm

    Differential Revision: (detail/ViewSVN)
    by asbirlea
  92. [System Model] [TTI] Move default cache/prefetch implementations

    Move the default implementations of cache and prefetch queries to
    TargetTransformInfoImplBase and delete them from NoTIIImpl.  This brings these
    interfaces in line with how other TTI interfaces work.

    Differential Revision: (detail/ViewSVN)
    by greened
  93. Fix a documentation warning from GSYM commit. (detail/ViewSVN)
    by gclayton
  94. [MSVC] Automatically add atlmfc folder to include and libpath.

    Differential Revision: (detail/ViewSVN)
    by zturner
  95. [PDB] Fix bug when using multiple PCH header objects with the same name.

    A common pattern in Windows is to have all your precompiled headers
    use an object named stdafx.obj.  If you've got a project with many
    different static libs, you might use a separate PCH for each one of

    During the final link step, a file from A might reference the PCH
    object from A, but it will have the same name (stdafx.obj) as any
    other PCH from another project.  The only difference will be the
    path.  For example, A might be A/stdafx.obj while B is B/stdafx.obj.

    The existing algorithm checks only the filename that was passed on
    the command line (or stored in archive), but this is insufficient in
    the case where relative paths are used, because depending on the
    command line object file / library order, it might find the wrong
    PCH object first resulting in a signature mismatch.

    The fix here is to simply check whether the absolute path of the
    PCH object (which is stored in the input obj file for the file that
    references the PCH) *ends with* the full relative path of whatever
    is specified on the command line (or is in the archive).

    Differential Revision: (detail/ViewSVN)
    by zturner
  96. [lit] Add comment explaining the LIT_OPTS env var overrides command line options

    Normally, command line options override environment variables.  Add
    comment to state that we are doing the reverse on purpose. (detail/ViewSVN)
    by yln
  97. ADT: Save a word in every StringSet entry

    Add a specialization to StringMap (actually StringMapEntry) for a
    value type of NoneType (the type of llvm::None), and use it for
    StringSet. This'll save us a word from every entry in a StringSet,
    used for alignment with the size_t that stores the string length.

    I could have gone all the way to some kind of empty base class
    optimization, but that seemed like overkill. Someone can consider
    adding that in the future, though. (detail/ViewSVN)
    by jrose
  98. [ScanDeps] clang-format, 80 cols. (detail/ViewSVN)
    by mspencer
  99. [OPENMP]Update doc for supported constructs, NFC. (detail/ViewSVN)
    by abataev
  100. [OPENMP50]Support for 'master taskloop' directive.

    Added full support for master taskloop directive. (detail/ViewSVN)
    by abataev
  101. [x86] reduce duplicate test assertions; NFC (detail/ViewSVN)
    by spatel

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

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

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

This run spent:

  • 22 hr waiting;
  • 38 min build duration;
  • 23 hr total from scheduled to completion.

Identified problems

Ninja target failed

Below is a link to the first failed ninja target.
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

Missing test results

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