Started 10 hr ago
Build has been executing for 10 hr on green-dragon-08

Progress:
In progress Build #5553 (Oct 22, 2019 6:54:32 AM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 375501
  • http://llvm.org/svn/llvm-project/cfe/trunk : 375491
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 375490
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 364589
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 375469
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 375316
Changes
  1. [MIPS GlobalISel] Select MSA vector generic and builtin add

    Select vector G_ADD for MIPS32 with MSA. We have to set bank
    for vector operands to fprb and selectImpl will do the rest.
    __builtin_msa_addv_<format> will be transformed into G_ADD
    in legalizeIntrinsic and selected in the same way.
    __builtin_msa_addvi_<format> will be directly selected into
    ADDVI_<format> in legalizeIntrinsic. MIR tests for it have
    unnecessary additional copies. Capture current state of tests
    with run-pass=legalizer with a test in test/CodeGen/MIR/Mips.

    Differential Revision: https://reviews.llvm.org/D68984 (detail/ViewSVN)
    by petar.avramovic
  2. [ThinLTO] Add code comment. NFC (detail/ViewSVN)
    by evgeny777
  3. [Alignment][NFC] Convert StoreInst to MaybeAlign

    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: hiraditya, jfb, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D69303 (detail/ViewSVN)
    by gchatelet
  4. [Alignment][NFC] Convert LoadInst to MaybeAlign

    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: hiraditya, jfb, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D69302 (detail/ViewSVN)
    by gchatelet
  5. [PowerPC] Turn on CR-Logical reducer pass

    This re-commits r375152 which was pulled in r375233 because it broke
    the EXPENSIVE_CHECKS bot on Windows.

    The reason for the failure was a bug in the pass that the commit turned
    on by default. This patch fixes that bug and turns the pass back on.
    This patch has been verified on the buildbot that originally failed
    thanks to Simon Pilgrim.

    Differential revision: https://reviews.llvm.org/D52431 (detail/ViewSVN)
    by nemanjai
  6. [Alignment][NFC] Use MaybeAlign in AttrBuilder

    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: hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D69300 (detail/ViewSVN)
    by gchatelet
  7. [Alignment][NFC] Attributes use Align/MaybeAlign

    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: jholewinski, hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D69278 (detail/ViewSVN)
    by gchatelet
  8. [ThinLTO] Don't internalize during promotion

    Differential revision: https://reviews.llvm.org/D69107 (detail/ViewSVN)
    by evgeny777
  9. [LLVMDebugInfoPDB] - Use cantFail() instead of assert().

    Currently injected-sources-native.test fails with "Expected<T>
    value was in success state.
    (Note: Expected<T> values in success mode must still be checked
    prior to being destroyed)"
    when llvm is compiled with LLVM_ENABLE_ABI_BREAKING_CHECKS in Release.

    The problem is that getStringForID returns Expected<StringRef>
    and Expected value must always be checked, even if it is in success state.
    Checking with assert only helps in Debug and is wrong.

    Differential revision: https://reviews.llvm.org/D69251 (detail/ViewSVN)
    by grimar
  10. [FrontendTests] Try again to make test not write an output file

    Setting the output stream to nulls seems to work. (detail/ViewSVN)
    by d0k
  11. [builtins][test] Avoid unportable mmap call in clear_cache_test.c

    Within the last two weeks, the Builtins-*-sunos :: clear_cache_test.c started to FAIL
    on Solaris.  Running it under truss shows

      mmap(0x00000000, 128, PROT_READ|PROT_WRITE|PROT_EXEC, MAP_PRIVATE|MAP_ANON, 0, 0) Err#22 EINVAL
      _exit(1)

    While there are several possible reasons mmap can return EINVAL on Solaris, it turns
    out it's this one (from mmap(2)):

      MAP_ANON  was  specified,  but the file descriptor was not
       -1.

    And indeed even the Linux mmap(2) documents this as unportable:

      MAP_ANONYMOUS
              The mapping is not backed by any file; its contents are initial‐
              ized to zero.  The fd argument is ignored; however, some  imple‐
              mentations require fd to be -1 if MAP_ANONYMOUS (or MAP_ANON) is
              specified, and portable applications should  ensure  this.   The

    This patch follows this advise.  Tested on x86_64-pc-linux-gnu, amd64-pc-solaris2.11
    and sparcv9-sun-solaris2.11.

    Differential Revision: https://reviews.llvm.org/D68455 (detail/ViewSVN)
    by ro
  12. Revert "[FrontendTests] Don't actually run the full compiler, parsing is sufficient."

    This reverts commit 375488. (detail/ViewSVN)
    by d0k
  13. [FrontendTests] Don't actually run the full compiler, parsing is sufficient. (detail/ViewSVN)
    by d0k
  14. [clang-fuzzer] Update proto fuzzer example for r375453. (detail/ViewSVN)
    by d0k
  15. [CMake] [WinMsvc] Look for includes and libs in ${MSVC_BASE}/atlmfc

    This is necessary if building with the DIA SDK enabled.

    Differential Revision: https://reviews.llvm.org/D69240 (detail/ViewSVN)
    by mstorsjo
  16. [CMake] Allow overriding MSVC_DIA_SDK_DIR via CMake

    This eases using it in cross-msvc setups.

    Differential Revision: https://reviews.llvm.org/D69239 (detail/ViewSVN)
    by mstorsjo
  17. gn build: Merge r375483 (detail/ViewSVN)
    by gnsyncbot
  18. Refactor DependencyScanningTool to its own file

    Summary:
    There's no behavior change - just moving DependencyScanningTool to its own file
    since this tool can be reused across both clang-scan-deps binary and an interface
    exposed as part of libClang APIs.

    Reviewers: arphaman, jkorous, Bigcheese, dexonsmith

    Subscribers: mgorny, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D69186 (detail/ViewSVN)
    by kousikk
  19. Minor coding style fix. NFC. (detail/ViewSVN)
    by hliao
  20. [lit] Move increase_process_limit to ParallelRun

    Increasing the process limit only makes sense when we use multiple
    processes. (detail/ViewSVN)
    by yln
  21. Fix lld detection in standalone compiler-rt.

    Summary:
    Right now all hwasan tests on Android are silently disabled because they
    require "has_lld" and standalone compiler-rt can not (and AFAIK was
    never able to) set it.

    Reviewers: pcc, dyung

    Subscribers: dberris, mgorny, #sanitizers, llvm-commits

    Tags: #sanitizers, #llvm

    Differential Revision: https://reviews.llvm.org/D69196 (detail/ViewSVN)
    by eugenis
  22. [hwasan] Workaround unwinder issues in try-catch test.

    Android links the unwinder library to every DSO. The problem is,
    unwinder has global state, and hwasan implementation of personality
    function wrapper happens to rub it the wrong way.

    Switch the test to static libc++ as a temporary workaround. (detail/ViewSVN)
    by eugenis
  23. [X86][BMI] Pull out schedule classes from bmi_andn<> and bmi_bls<>

    Stop hardwiring classes (detail/ViewSVN)
    by rksimon
  24. [NFC] Fix typos in CMake comment (detail/ViewSVN)
    by Louis Dionne
  25. Fix -fuse-init-array decision logic on NetBSD

    For NetBSD 9 and later, it is the default. On older versions, only ARM
    and AArch64 use it by default. (detail/ViewSVN)
    by joerg
  26. [Implicit Modules] Add -cc1 option -fmodules-strict-context-hash which includes search paths and diagnostics.

    This is a recommit of r375322 and r375327 with a fix for the Windows test breakage. (detail/ViewSVN)
    by mspencer
  27. [X86][SSE] Add OR(EXTRACTELT(X,0),OR(EXTRACTELT(X,1))) -> MOVMSK+CMP reduction combine (detail/ViewSVN)
    by rksimon
  28. [X86][SSE] Add OR(EXTRACTELT(X,0),OR(EXTRACTELT(X,1))) movmsk v2X64 tests (detail/ViewSVN)
    by rksimon
  29. [examples] Add a dependency on ExecutionEngine to LLJITWithObjectCache example.

    ExecutionEngine.cpp contains the anchor() for the ObjectCache base class, so we
    need an explicit dependency on it.

    Patch by Stephen Neuendorffer. Thanks Stephen! (detail/ViewSVN)
    by Lang Hames
  30. AMDGPU/GlobalISel: Legalize fast unsafe FDIV

    Reviewers: arsenm

    Reviewed By: arsenm

    Subscribers: kzhuravl, jvesely, wdng, nhaehnle, yaxunl, rovka, dstuttard, tpr, t-tye, hiraditya, Petar.Avramovic, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D69231 (detail/ViewSVN)
    by kerbowa
  31. [lit] Simplify test scheduling via multiprocessing.Pool (detail/ViewSVN)
    by yln
  32. AMDGPU: Select basic interp directly from intrinsics (detail/ViewSVN)
    by arsenm
  33. [lit] Remove redundancy from names and comments (detail/ViewSVN)
    by yln
  34. [CVP] No-wrap deduction for `shl`

    Summary:
    This is the last `OverflowingBinaryOperator` for which we don't deduce flags.
    D69217 taught `ConstantRange::makeGuaranteedNoWrapRegion()` about it.

    The effect is better than of the `mul` patch (D69203):

    | statistic                              |     old |     new | delta | % change |
    | correlated-value-propagation.NumAddNUW |    7145 |    7144 |    -1 | -0.0140% |
    | correlated-value-propagation.NumAddNW  |   12126 |   12125 |    -1 | -0.0082% |
    | correlated-value-propagation.NumAnd    |     443 |     446 |     3 |  0.6772% |
    | correlated-value-propagation.NumNSW    |    5986 |    7158 |  1172 | 19.5790% |
    | correlated-value-propagation.NumNUW    |   10512 |   13304 |  2792 | 26.5601% |
    | correlated-value-propagation.NumNW     |   16498 |   20462 |  3964 | 24.0272% |
    | correlated-value-propagation.NumShlNSW |       0 |    1172 |  1172 |          |
    | correlated-value-propagation.NumShlNUW |       0 |    2793 |  2793 |          |
    | correlated-value-propagation.NumShlNW  |       0 |    3965 |  3965 |          |
    | instcount.NumAShrInst                  |   13824 |   13790 |   -34 | -0.2459% |
    | instcount.NumAddInst                   |  277584 |  277586 |     2 |  0.0007% |
    | instcount.NumAndInst                   |   66061 |   66056 |    -5 | -0.0076% |
    | instcount.NumBrInst                    |  709153 |  709147 |    -6 | -0.0008% |
    | instcount.NumICmpInst                  |  483709 |  483708 |    -1 | -0.0002% |
    | instcount.NumSExtInst                  |   79497 |   79496 |    -1 | -0.0013% |
    | instcount.NumShlInst                   |   40691 |   40654 |   -37 | -0.0909% |
    | instcount.NumSubInst                   |   61997 |   61996 |    -1 | -0.0016% |
    | instcount.NumZExtInst                  |   68208 |   68211 |     3 |  0.0044% |
    | instcount.TotalBlocks                  |  843916 |  843910 |    -6 | -0.0007% |
    | instcount.TotalInsts                   | 7387528 | 7387448 |   -80 | -0.0011% |

    Reviewers: nikic, reames, sanjoy, timshen

    Reviewed By: nikic

    Subscribers: hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D69277 (detail/ViewSVN)
    by lebedevri
  35. [clang-fuzzer] Add new fuzzer target for Objective-C

    Summary:
    - Similar to that of `clang-fuzzer` itself but instead only
      targets Objective-C source files via cc1

    - Also adds an example corpus directory containing some
      input for Objective-C

    Subscribers: mgorny, jfb, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D69171 (detail/ViewSVN)
    by dgoldman
  36. [GISel][CombinerHelper] Add a combine turning shuffle_vector into concat_vectors

    Teach the CombinerHelper how to turn shuffle_vectors, that
    concatenate vectors, into concat_vectors and add this combine
    to the AArch64 pre-legalizer combiner.

    Differential Revision: https://reviews.llvm.org/D69149 (detail/ViewSVN)
    by qcolombet
  37. AMDGPU: Use CopyToReg for interp intrinsic lowering

    This doesn't use the default value, so doesn't benefit from the hack
    to help optimize it. (detail/ViewSVN)
    by arsenm
  38. AMDGPU: Erase redundant redefs of m0 in SIFoldOperands

    Only handle simple inter-block redefs of m0 to the same value. This
    avoids interference from redefs of m0 in SILoadStoreOptimzer. I was
    initially teaching that pass to ignore redefs of m0, but having them
    not exist beforehand is much simpler.

    This is in preparation for deleting the current special m0 handling in
    SIFixSGPRCopies to allow the register coalescer to handle the
    difficult cases. (detail/ViewSVN)
    by arsenm
  39. AMDGPU: Stop adding m0 implicit def to SGPR spills

    r375293 removed the SGPR spilling with scalar stores path, so this is
    no longer necessary. This also always had the defect of adding the def
    even when this path wasn't in use. (detail/ViewSVN)
    by arsenm
  40. AMDGPU: Slightly restructure m0 init code

    This will allow using another operation to produce the glue in a
    future change. (detail/ViewSVN)
    by arsenm
  41. [AMDGPU] Select AGPR in PHI operand legalization

    If a PHI defines AGPR legalize its operands to AGPR.
    At the moment we can get an AGPR PHI with VGPR operands.
    I am not aware of any problems as it seems to be handled
    gracefully in RA, but this is not right anyway.

    It also slightly decreases VGPR pressure in some cases
    because we do not have to a copy via VGPR.

    Differential Revision: https://reviews.llvm.org/D69206 (detail/ViewSVN)
    by rampitec
  42. [X86] Rename matchBitOpReduction to matchScalarReduction. NFCI.

    This doesn't need to be just for bitops, but the ops do need to be fully associative. (detail/ViewSVN)
    by rksimon
  43. Reverted r375425 as it broke some buildbots. (detail/ViewSVN)
    by s.desmalen
  44. SemaTemplateDeduction - silence static analyzer getAs<> null dereference warnings. NFCI.

    The static analyzer is warning about potential null dereferences, but in these cases we should be able to use castAs<> directly and if not assert will fire for us. (detail/ViewSVN)
    by rksimon
  45. [Clang] Add VerboseOutputStream to CompilerInstance

    Remove one instance of a hardcoded output stream in
    CompilerInstance::ExecuteAction. There are still other cases of output
    being hard-coded to standard streams in ExecuteCompilerInvocation, but
    this patch covers the case when no flags like -version or -help are
    passed, namely the "X warnings and Y errors generated." diagnostic.

    Differential Revision: https://reviews.llvm.org/D53768 (detail/ViewSVN)
    by scott.linder
  46. [NFC][CVP] Add `shl` no-wrap deduction test coverage (detail/ViewSVN)
    by lebedevri
  47. SemaExceptionSpec - silence static analyzer getAs<> null dereference warnings. NFCI.

    The static analyzer is warning about potential null dereferences, but in these cases we should be able to use castAs<> directly and if not assert will fire for us. (detail/ViewSVN)
    by rksimon
  48. [test] Merge Driver/as-w-warnings.c into as-no-warnings.c

    For -integrated-as RUN lines we can remove -target. (detail/ViewSVN)
    by maskray
  49. Prune include of DataLayout.h from include/clang/Basic/TargetInfo.h. NFC

    Summary:
    Use a forward declaration of DataLayout instead of including
    DataLayout.h in clangs TargetInfo.h. This reduces include
    dependencies toward DataLayout.h (and other headers such as
    DerivedTypes.h, Type.h that is included by DataLayout.h).

    Needed to move implemantation of TargetInfo::resetDataLayout
    from TargetInfo.h to TargetInfo.cpp.

    Reviewers: rnk

    Reviewed By: rnk

    Subscribers: jvesely, nhaehnle, cfe-commits, llvm-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D69262 (detail/ViewSVN)
    by bjope
  50. Prune Pass.h include from DataLayout.h. NFCI

    Summary:
    Reduce include dependencies by no longer including Pass.h from
    DataLayout.h. That include seemed irrelevant to DataLayout, as
    well as being irrelevant to several users of DataLayout.

    Reviewers: rnk

    Reviewed By: rnk

    Subscribers: mehdi_amini, hiraditya, cfe-commits, llvm-commits

    Tags: #clang, #llvm

    Differential Revision: https://reviews.llvm.org/D69261 (detail/ViewSVN)
    by bjope
  51. [PowerPC] Regenerate test for D52431 (detail/ViewSVN)
    by rksimon
  52. Fix Wdocumentation warning. NFCI. (detail/ViewSVN)
    by rksimon
  53. [NFC] Add missing include to fix modules build

    This header doesn't seem to be parsable on its own and breaks the module build therefore with
    the following error:

    While building module 'LLVM_Backend' imported from llvm-project/llvm/lib/CodeGen/MachineScheduler.cpp:14:
    In file included from <module-includes>:62:
    llvm-project/llvm/include/llvm/CodeGen/MachinePipeliner.h:91:20: error: declaration of 'AAResultsWrapperPass' must be imported from module 'LLVM_Analysis.AliasAnalysis' before it is required
        AU.addRequired<AAResultsWrapperPass>();
                       ^
    llvm-project/llvm/include/llvm/Analysis/AliasAnalysis.h:1157:7: note: previous declaration is here
    class AAResultsWrapperPass : public FunctionPass {
          ^
    llvm-project/llvm/lib/CodeGen/MachineScheduler.cpp:14:10: fatal error: could not build module 'LLVM_Backend'
    ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    2 errors generated. (detail/ViewSVN)
    by Raphael Isemann
  54. SystemZISelLowering - supportedAddressingMode - silence static analyzer dyn_cast<> null dereference warning. NFCI.

    The static analyzer is warning about a potential null dereference, but we should be able to use cast<> directly and if not assert will fire for us. (detail/ViewSVN)
    by rksimon
  55. GVNHoist - silence static analyzer dyn_cast<> null dereference warning in hasEHOrLoadsOnPath call. NFCI.

    The static analyzer is warning about a potential null dereference, but we should be able to use cast<> directly and if not assert will fire for us. (detail/ViewSVN)
    by rksimon
  56. GuardWidening - silence static analyzer null dereference warning with assertion. NFCI. (detail/ViewSVN)
    by rksimon
  57. CrossDSOCFI - silence static analyzer dyn_cast<> null dereference warning. NFCI.

    The static analyzer is warning about a potential null dereference, but we should be able to use cast<> directly and if not assert will fire for us. (detail/ViewSVN)
    by rksimon
  58. IndVarSimplify - silence static analyzer dyn_cast<> null dereference warning. NFCI.

    The static analyzer is warning about a potential null dereference, but we should be able to use cast<> directly and if not assert will fire for us. (detail/ViewSVN)
    by rksimon
  59. [AArch64][DebugInfo] Do not recompute CalleeSavedStackSize (Take 2)

    Commit message from D66935:

    This patch fixes a bug exposed by D65653 where a subsequent invocation
    of `determineCalleeSaves` ends up with a different size for the callee
    save area, leading to different frame-offsets in debug information.

    In the invocation by PEI, `determineCalleeSaves` tries to determine
    whether it needs to spill an extra callee-saved register to get an
    emergency spill slot. To do this, it calls 'estimateStackSize' and
    manually adds the size of the callee-saves to this. PEI then allocates
    the spill objects for the callee saves and the remaining frame layout
    is calculated accordingly.

    A second invocation in LiveDebugValues causes estimateStackSize to return
    the size of the stack frame including the callee-saves. Given that the
    size of the callee-saves is added to this, these callee-saves are counted
    twice, which leads `determineCalleeSaves` to believe the stack has
    become big enough to require spilling an extra callee-save as emergency
    spillslot. It then updates CalleeSavedStackSize with a larger value.

    Since CalleeSavedStackSize is used in the calculation of the frame
    offset in getFrameIndexReference, this leads to incorrect offsets for
    variables/locals when this information is recalculated after PEI.

    This patch fixes the lldb unit tests in `functionalities/thread/concurrent_events/*`

    Changes after D66935:

    Ensures AArch64FunctionInfo::getCalleeSavedStackSize does not return
    the uninitialized CalleeSavedStackSize when running `llc` on a specific
    pass where the MIR code has already been expected to have gone through PEI.

    Instead, getCalleeSavedStackSize (when passed the MachineFrameInfo) will try
    to recalculate the CalleeSavedStackSize from the CalleeSavedInfo. In debug
    mode, the compiler will assert the recalculated size equals the cached
    size as calculated through a call to determineCalleeSaves.

    This fixes two tests:
      test/DebugInfo/AArch64/asan-stack-vars.mir
      test/DebugInfo/AArch64/compiler-gen-bbs-livedebugvalues.mir
    that otherwise fail when compiled using msan.

    Reviewed By: omjavaid, efriedma

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D68783 (detail/ViewSVN)
    by s.desmalen
  60. PCH debug info: Avoid appending the source directory to an absolute path

    When building a precompiled header in -fmodule-format=obj (i.e.,
    `-gmodules) in an absolute path, the locig in
    CGDebugInfo::createCompileUnit would unconditionally append the source
    directory to the -main-file-name. This patch avoids that behavior for
    absolute paths.

    rdar://problem/46045865

    Differential Revision: https://reviews.llvm.org/D69213 (detail/ViewSVN)
    by Adrian Prantl

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

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

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

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

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

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

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

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

This run spent 20 hr waiting in the queue.