Started 3 mo 27 days ago
Took 9 hr 56 min on green-dragon-14

Success Build #4173 (Jul 19, 2019 8:27:19 PM)

Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 366625
  • http://llvm.org/svn/llvm-project/cfe/trunk : 366624
  • http://llvm.org/svn/llvm-project/compiler-rt/trunk : 366618
  • http://llvm.org/svn/llvm-project/lld/trunk : 366624
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 364589
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 366606
  • http://llvm.org/svn/llvm-project/test-suite/trunk : 366290
  • http://llvm.org/svn/llvm-project/clang-tools-extra/trunk : 366577
  • http://llvm.org/svn/llvm-project/polly/trunk : 366510
Changes
  1. [GlobalISel][AArch64] Contract trivial same-size cross-bank copies into G_STOREs

    Sometimes, you can end up with cross-bank copies between same-sized GPRs and
    FPRs, which feed into G_STOREs. When these copies feed only into stores, they
    aren't necessary; we can just store using the original register bank.

    This provides some minor code size savings for some floating point SPEC
    benchmarks. (Around 0.2% for 453.povray and 450.soplex)

    This issue doesn't seem to show up due to regbankselect or anything similar. So,
    this patch introduces an early select function, `contractCrossBankCopyIntoStore`
    which performs the contraction when possible. The selector then continues
    normally and selects the correct store opcode, eliminating needless copies
    along the way.

    Differential Revision: https://reviews.llvm.org/D65024 (detail/ViewSVN)
    by paquette
  2. [WebAssembly] Compute and export TLS block alignment

    Summary:
    Add immutable WASM global `__tls_align` which stores the alignment
    requirements of the TLS segment.

    Add `__builtin_wasm_tls_align()` intrinsic to get this alignment in Clang.

    The expected usage has now changed to:

        __wasm_init_tls(memalign(__builtin_wasm_tls_align(),
                                 __builtin_wasm_tls_size()));

    Reviewers: tlively, aheejin, sbc100, sunfish, alexcrichton

    Reviewed By: tlively

    Subscribers: dschuff, jgravelle-google, hiraditya, cfe-commits, llvm-commits

    Tags: #clang, #llvm

    Differential Revision: https://reviews.llvm.org/D65028 (detail/ViewSVN)
    by quantum
  3. [LTO] Always mark regular LTO units with EnableSplitLTOUnit=1

    Summary:
    Regular LTO modules do not need LTO Unit splitting, only ThinLTO does
    (they must be consistently split into regular and Thin units for
    optimizations such as whole program devirtualization and lower type
    tests). In order to avoid spurious errors from LTO when combining with
    split ThinLTO modules, always set this flag for regular LTO modules.

    Reviewers: pcc

    Subscribers: mehdi_amini, Prazek, inglorion, steven_wu, dexonsmith, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D65009 (detail/ViewSVN)
    by tejohnson
  4. Re-commit: r366610 and r366612: Expand pseudo-components before embedding in llvm-config

    There were two main problems:
    * The 'nativecodegen' pseudo-component was unconditionally adding
      ${native_tgt}CodeGen even though it conditionally added ${native_tgt}Info and
      ${native_tgt}Desc. This has been fixed by making ${native_tgt}CodeGen
      conditional too
    * The 'all' pseudo-component was causing library names like LLVMLLVMDemangle as
      the expansion was to a library name and not a component. There doesn't seem to
      be a list of available components anywhere so this has been fixed by moving the
      expansion of 'all' back where it was before. This manifested in different ways
      on different builders but it was the same root cause (detail/ViewSVN)
    by dsanders
  5. AMDGPU/GlobalISel: Legalize GEP for other 32-bit address spaces (detail/ViewSVN)
    by arsenm
  6. [NFC] Remove unused variable

    Change-Id: I5aee24dcdf6eebfbf788e52be22463387f23d927 (detail/ViewSVN)
    by dendibakh
  7. [AMDGPU] Autogenerate register sequences in tuples

    Differential Revision: https://reviews.llvm.org/D65007 (detail/ViewSVN)
    by rampitec
  8. Revert "Fix asan infinite loop on undefined symbol"

    This reverts commit 8e46275488ca453a24964cb26be0736172ad1db8.

    This was failing on sanitizer-x86_64-linux and our internal CI. (detail/ViewSVN)
    by ormris
  9. Disallow most calling convention attributes on PS4

    PS4 now only allows "cdecl", and its equivalent on PS4, "sysv_abi".

    Differential Revision: https://reviews.llvm.org/D64780 (detail/ViewSVN)
    by ssrivastava
  10. [AMDGPU] Fixed occupancy calculation for gfx10

    Differential Revision: https://reviews.llvm.org/D65010 (detail/ViewSVN)
    by rampitec
  11. Revert r366610 and r366612: Expand pseudo-components before embedding in llvm-config

    Some targets are missing LLVMDemangle, one is adding the LLVM prefix twice, and two
    are hitting the very error this patch fixes for my target. Reverting while I work
    through the reports. (detail/ViewSVN)
    by dsanders
  12. [InstCombine] Fix copy/paste mistake in the test cases I added for PR42691. NFC (detail/ViewSVN)
    by ctopper
  13. AMDGPU: Avoid custom predicates for stores with glue (detail/ViewSVN)
    by arsenm
  14. Fix a latent bug discovered by r366610: nativecodegen includes X86CodeGen when X86 is not compiled

    I believe this to have been a latent bug as the same expansion checks for the
    existence of ${native_tgt}Info and ${native_tgt}Desc and only adds them if
    they were compiled but unconditionally adds ${native_tgt}CodeGen.

    This should fix llvm-clang-x86_64-win-fast which builds ARM only on an X86 host and similar builders. (detail/ViewSVN)
    by dsanders
  15. [InstCombine] Add test cases for PR42691. NFC (detail/ViewSVN)
    by ctopper
  16. Expand pseudo-components before embedding in llvm-config

    Summary:
    If you use pseudo-targets like AllTargetsCodeGens in LLVM_DYLIB_COMPONENTS
    then a test will fail because `./bin/llvm-config --shared-mode` can't
    handle these targets. We can fix this by expanding them before embedding
    the string into llvm-config

    Reviewers: bogner

    Reviewed By: bogner

    Subscribers: mgorny, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D65011 (detail/ViewSVN)
    by dsanders
  17. AMDGPU: Redefine setcc condition PatLeafs

    Avoid using custom code predicates. (detail/ViewSVN)
    by arsenm
  18. AMDGPU: Don't rely on m0 being -1 for GWS offsets

    This only works if the high bits of m0 are also 0, so m0 would have to
    be set to 0xffff. (detail/ViewSVN)
    by arsenm
  19. AMDGPU: Force s_waitcnt after GWS instructions

    This is apparently required to be the immediately following
    instruction, so force it into a bundle with a waitcnt. (detail/ViewSVN)
    by arsenm
  20. [libc++] Mark libcpp_deallocate.sh.cpp as UNSUPPORTED instead of XFAIL on AppleClang 9

    Some minor versions of AppleClang 9 appear not to fail the test. It's
    such a mess that the only sane thing to do is to mark the test as
    UNSUPPORTED. (detail/ViewSVN)
    by Louis Dionne
  21. LiveIntervals: Fix handleMove asserting on BUNDLE

    The top-level BUNDLE instruction should behave as an ordinary
    instruction. It is supposed to have all relevant registers as implicit
    operands. Moving it should work as any other instruction. I believe
    the assert intended to avoid moving instructions inside bundles. (detail/ViewSVN)
    by arsenm
  22. Revert "[libc++] Integrate the PSTL into libc++"

    This reverts r366593, which caused unforeseen breakage on the build bots.
    I'm reverting until the problems have been figured out and fixed. (detail/ViewSVN)
    by Louis Dionne
  23. [AMDGPU] Add test case on crashing of `si-lower-sgpr-spills` pass

    Reviewers: arsenm

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

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D64273 (detail/ViewSVN)
    by hliao
  24. [libc++] Allow passing additional CMake arguments in macOS trunk CI script (detail/ViewSVN)
    by Louis Dionne
  25. Revert "Use the MachineBasicBlock symbol for a callbr target"

    This reverts commit r366523/ccbffefccaff42b0d094c9ef0f49fc3e8c8456ea.

    Two regressions were immediately reported:
    - https://github.com/ClangBuiltLinux/linux/issues/614
    - https://github.com/ClangBuiltLinux/linux/issues/615

    Reported-by: nathanchance (detail/ViewSVN)
    by nickdesaulniers
  26. [RISCV] Disable tests failing on buildbots.

    r366399 enabled a couple tests that are failing on a few buildbots. (detail/ViewSVN)
    by morehouse
  27. [AMDGPU] Allow register tuples to set asm names

    This change reverts most of the previous register name generation.
    The real problem is that RegisterTuple does not generate asm names.
    Added optional operand to RegisterTuple. This way we can simplify
    register name access and dramatically reduce the size of static
    tables for the backend.

    Differential Revision: https://reviews.llvm.org/D64967 (detail/ViewSVN)
    by rampitec
  28. AMDGPU/GlobalISel: Fix MMO flags for kernel argument loads

    The DAG lowering sets dereferencable and invariant, not nontemporal. (detail/ViewSVN)
    by arsenm
  29. GlobalISel: Add GINodeEquiv for fcopysign

    I don't need this at the moment, but it should be here. (detail/ViewSVN)
    by arsenm
  30. [llvm-lipo] Remove trailing whitespace. NFC (detail/ViewSVN)
    by smeenai
  31. [libc++] Integrate the PSTL into libc++

    Summary:
    This commit allows specifying LIBCXX_ENABLE_PARALLEL_ALGORITHMS when
    configuring libc++ in CMake. When that option is enabled, libc++ will
    assume that the PSTL can be found somewhere on the CMake module path,
    and it will provide the C++17 parallel algorithms based on the PSTL
    (that is assumed to be available).

    The commit also adds support for running the PSTL tests as part of
    the libc++ test suite.

    Reviewers: rodgert, EricWF

    Subscribers: mgorny, christof, jkorous, dexonsmith, libcxx-commits, mclow.lists, EricWF

    Tags: #libc

    Differential Revision: https://reviews.llvm.org/D60480 (detail/ViewSVN)
    by Louis Dionne
  32. [Format/ObjC] Avoid breaking between unary operators and operands

    Summary:
    Test Plan:
      New tests added. Ran tests with:
      % ninja FormatTests && ./tools/clang/unittests/Format/FormatTests
      Confirmed tests failed before change and passed after change.

    Reviewers: krasimir, djasper, sammccall, klimek

    Reviewed By: sammccall

    Subscribers: klimek, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64775 (detail/ViewSVN)
    by benhamilton
  33. AMDGPU: Add some function return test cases (detail/ViewSVN)
    by arsenm
  34. [AMDGPU] Regenerate test file for upcoming patch. NFCI. (detail/ViewSVN)
    by rksimon

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

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

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

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

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

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

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

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

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

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

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

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

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

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

This run spent:

  • 11 hr waiting;
  • 9 hr 56 min build duration;
  • 21 hr total from scheduled to completion.
LLVM/Clang Warnings: 12 warnings.