Home | Search | fuchsia - Builders
Login

Builder clang-x64-mac Build 8927851917431887600 Apple OSX

Results:

Success

Trigger Info:

Projectfuchsia
Revision4d6ceeaae2b4553a2809d5f6c2d8cbc9d2b88722
Got Revision18890f471c420411aa3c989e104d090966ec9dbf

Execution:

Steps and Logfiles:

Show:
  1. ( 72 ms ) setup_build
    running recipe: "clang_toolchain"
  2. ( 528 ms ) create pkg dir
  3. ( 498 ms ) makedirs
  4. ( 36 ms ) git init
  5. ( 30 ms ) git remote
  6. ( 1 mins 55 secs ) git fetch
  7. ( 19 secs ) git checkout
  8. ( 32 ms ) git rev-parse
  9. ( 371 ms ) git clean
  10. ( 199 ms ) submodule sync
  11. ( 231 ms ) submodule update
  12. ( 10 ms ) xcrun
  13. ( 555 ms ) Version.inc
  14. ( 8 secs ) gsutil cp

Build Properties:

NameValueSource
$recipe_engine/path {"cache_dir": "/b/s/w/ir/cache", "temp_dir": "/b/s/w/ir/tmp/rt"} setup_build
$recipe_engine/runtime {"is_experimental": false, "is_luci": true} setup_build
bot_id "fuchsia-ci-mac-lab01-1004" setup_build
branch "refs/heads/master" setup_build
buildbucket {"build": {"bucket": "luci.fuchsia.ci", "created_by": "user:luci-scheduler@appspot.gserviceaccount.com", "created_ts": 1544133947874881, "id": "8927851917431887600", "project": "fuchsia", "tags": ["builder:clang-x64-mac", "buildset:commit/git/4d6ceeaae2b4553a2809d5f6c2d8cbc9d2b88722", "buildset:commit/gitiles/fuchsia.googlesource.com/third_party/llvm-project/+/4d6ceeaae2b4553a2809d5f6c2d8cbc9d2b88722", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9093281979603068480", "scheduler_job_id:fuchsia/clang-x64-mac", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"} setup_build
buildername "clang-x64-mac" setup_build
gcs_bucket "fuchsia-build" setup_build
packages ["topaz/packages/buildbot"] setup_build
path_config "generic" setup_build
platform "mac-amd64" setup_build
project "integration" setup_build
recipe "clang_toolchain" setup_build
remote "https://fuchsia.googlesource.com/integration" setup_build
repository "https://fuchsia.googlesource.com/third_party/llvm-project" setup_build
revision "4d6ceeaae2b4553a2809d5f6c2d8cbc9d2b88722" setup_build
got_revision "4d6ceeaae2b4553a2809d5f6c2d8cbc9d2b88722" git rev-parse

Blamelist:

  1. Louis Dionne (ldionneohnoyoudont@apple.com)
  2. Aaron Smith (aaron.smithohnoyoudont@microsoft.com)
  3. Nathan Lanza (nathanohnoyoudont@lanza.io)
  4. Louis Dionne (ldionneohnoyoudont@apple.com)
  5. Michal Gorny (mgornyohnoyoudont@gentoo.org)
  6. Sanjay Patel (spatelohnoyoudont@rotateright.com)
  7. Andrea Di Biagio (Andrea_DiBiagioohnoyoudont@sn.scee.net)
  8. Alexey Bataev (a.bataevohnoyoudont@hotmail.com)
  9. Louis Dionne (ldionneohnoyoudont@apple.com)
  10. Jim Ingham (jinghamohnoyoudont@apple.com)
  11. Sanjay Patel (spatelohnoyoudont@rotateright.com)
  12. Simon Pilgrim (llvm-devohnoyoudont@redking.me.uk)
  13. Sanjay Patel (spatelohnoyoudont@rotateright.com)

Timing:

Create Thursday, 06-Dec-18 22:05:47 UTC
Start Thursday, 06-Dec-18 22:05:52 UTC
End Friday, 07-Dec-18 00:53:17 UTC
Pending 4 secs
Execution 2 hrs 47 mins

All Changes:

  1. [libc++] Improve diagnostics for non-const comparators and hashers in associative containers

    Changed by Louis Dionne - ldionneohnoyoudont@apple.com
    Changed at Thursday, 06-Dec-18 21:46:17 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 4d6ceeaae2b4553a2809d5f6c2d8cbc9d2b88722

    Comments

    [libc++] Improve diagnostics for non-const comparators and hashers in associative containers
    
    Summary:
    When providing a non-const-callable comparator in a map or set, the
    warning diagnostic does not include the point of instantiation of
    the container that triggered the warning, which makes it difficult
    to track down the problem. This commit improves the diagnostic by
    placing it directly in the body of the associative container.
    
    The same change is applied to unordered associative containers, which
    had a similar problem.
    
    Finally, this commit cleans up the forward declarations of several
    map and unordered_map helpers, which are not needed anymore.
    
    <rdar://problem/41370747>
    
    Reviewers: EricWF, mclow.lists
    
    Subscribers: christof, dexonsmith, llvm-commits
    
    Differential Revision: https://reviews.llvm.org/D48955
    

    Changed files

    • libcxx/docs/UsingLibcxx.rst
    • libcxx/include/__hash_table
    • libcxx/include/__tree
    • libcxx/include/map
    • libcxx/include/set
    • libcxx/include/unordered_map
    • libcxx/include/unordered_set
    • libcxx/test/libcxx/containers/associative/non_const_comparator.fail.cpp
    • libcxx/test/libcxx/containers/unord/non_const_comparator.fail.cpp
  2. [pecoff] Implement ObjectFilePECOFF::GetDependedModules()

    Changed by Aaron Smith - aaron.smithohnoyoudont@microsoft.com
    Changed at Thursday, 06-Dec-18 21:36:39 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 93f281d789db9a99dd566e5c00e417e1efbba845

    Comments

    [pecoff] Implement ObjectFilePECOFF::GetDependedModules()
    
    Summary:
    This parses entries in pecoff import tables for imported DLLs and
    is intended as the first step to allow LLDB to load a PE's shared
    modules when creating a target on the LLDB console.
    
    
    Reviewers: rnk, zturner, aleksandr.urakov, lldb-commits, labath, asmith
    
    Reviewed By: labath, asmith
    
    Subscribers: labath, lemo, clayborg, Hui, mgorny, mgrang, teemperor
    
    Differential Revision: https://reviews.llvm.org/D53094
    

    Changed files

    • lldb/lit/Modules/PECOFF/dep-modules.yaml
    • lldb/lit/Modules/PECOFF/export-dllfunc.yaml
    • lldb/lit/Modules/PECOFF/lit.local.cfg
    • lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp
    • lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.h
    • lldb/tools/lldb-test/lldb-test.cpp
  3. Implement WindowsDYLD::DidAttach for use with gdb-server attach

    Changed by Nathan Lanza - nathanohnoyoudont@lanza.io
    Changed at Thursday, 06-Dec-18 21:13:03 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision cd6774859ae9c5760b6543743fb2847d550f9e8a

    Comments

    Implement WindowsDYLD::DidAttach for use with gdb-server attach
    
    Summary:
    Windows lldb debugging currently uses a process plugin to handle
    launching and attaching to a process. Launching a process via a debug
    server (e.g. ds2) and attaching to it with `gdb-remote port` currently
    doesn't communicate address information of the executable properly.
    
    Implement DynamicLoaderWindowsDYLD::DidAttach which allow us to
    obtain the proper executable load address.
    
    Differential Revision: https://reviews.llvm.org/D55383
    

    Changed files

    • lldb/source/Plugins/DynamicLoader/Windows-DYLD/DynamicLoaderWindowsDYLD.cpp
  4. [libcxx] Always convert 'use_system_cxx_lib' to an absolute path

    Changed by Louis Dionne - ldionneohnoyoudont@apple.com
    Changed at Thursday, 06-Dec-18 20:09:15 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision d6862bff4e4579df5fe8da78b303833b2e4b57de

    Comments

    [libcxx] Always convert 'use_system_cxx_lib' to an absolute path
    
    Otherwise, some tests would fail when a relative path was passed,
    because they'd use the relative path from a different directory
    than the current working directory.
    

    Changed files

    • libcxx/utils/libcxx/test/config.py
  5. [test] Add missing cmake include for building libFuzzer alone

    Changed by Michal Gorny - mgornyohnoyoudont@gentoo.org
    Changed at Thursday, 06-Dec-18 20:04:08 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 82abd0da80c26fe9afbbd67f2a6653303ab2670f

    Comments

    [test] Add missing cmake include for building libFuzzer alone
    
    Include CompilerRTCompile in fuzzer tests explicitly.  Otherwise, when
    building only libFuzzer, CMake fails due to:
    
    CMake Error at cmake/Modules/AddCompilerRT.cmake:395 (sanitizer_test_compile):
      Unknown CMake command "sanitizer_test_compile".
    Call Stack (most recent call first):
      lib/fuzzer/tests/CMakeLists.txt:53 (generate_compiler_rt_tests)
    
    Differential Revision: https://reviews.llvm.org/D55378
    

    Changed files

    • compiler-rt/lib/fuzzer/tests/CMakeLists.txt
  6. [DAGCombiner] reduce indent; NFC

    Changed by Sanjay Patel - spatelohnoyoudont@rotateright.com
    Changed at Thursday, 06-Dec-18 20:02:47 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 6a62eb943f65e94df837eda6612b2c13e6cca725

    Comments

    [DAGCombiner] reduce indent; NFC
    
    Unlike some of the folds in hoistLogicOpWithSameOpcodeHands()
    above this shuffle transform, this has the expected hasOneUse()
    checks in place.
    

    Changed files

    • llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  7. [DagCombiner][X86] Simplify a ConcatVectors of a scalar_to_vector with undef.

    Changed by Andrea Di Biagio - Andrea_DiBiagioohnoyoudont@sn.scee.net
    Changed at Thursday, 06-Dec-18 19:55:38 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 7f7a66f919c0516eac0ff902498dc0487d76cf8b

    Comments

    [DagCombiner][X86] Simplify a ConcatVectors of a scalar_to_vector with undef.
    
    This patch introduces a new DAGCombiner rule to simplify concat_vectors nodes:
    
    concat_vectors( bitcast (scalar_to_vector %A), UNDEF)
        --> bitcast (scalar_to_vector %A)
    
    This patch only partially addresses PR39257. In particular, it is enough to fix
    one of the two problematic cases mentioned in PR39257. However, it is not enough
    to fix the original test case posted by Craig; that particular case would
    probably require a more complicated approach (and knowledge about used bits).
    
    Before this patch, we used to generate the following code for function PR39257
    (-mtriple=x86_64 , -mattr=+avx):
    
    vmovsd  (%rdi), %xmm0           # xmm0 = mem[0],zero
    vxorps  %xmm1, %xmm1, %xmm1
    vblendps        $3, %xmm0, %xmm1, %xmm0 # xmm0 = xmm0[0,1],xmm1[2,3]
    vmovaps %ymm0, (%rsi)
    vzeroupper
    retq
    
    Now we generate this:
    
    vmovsd  (%rdi), %xmm0           # xmm0 = mem[0],zero
    vmovaps %ymm0, (%rsi)
    vzeroupper
    retq
    
    As a side note: that VZEROUPPER is completely redundant...
    
    I guess the vzeroupper insertion pass doesn't realize that the definition of
    %xmm0 from vmovsd is already zeroing the upper half of %ymm0. Note that on
    %-mcpu=btver2, we don't get that vzeroupper because pass vzeroupper insertion
    %pass is disabled.
    
    Differential Revision: https://reviews.llvm.org/D55274
    

    Changed files

    • llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    • llvm/test/CodeGen/X86/combine-concatvectors.ll
  8. [OPENMP][NVPTX]Correct type casting for printf args + simplified shfl64 function.

    Changed by Alexey Bataev - a.bataevohnoyoudont@hotmail.com
    Changed at Thursday, 06-Dec-18 19:45:48 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 94a61ea16fb9d2a9bbc0b261ae1cf1af700f4c7a

    Comments

    [OPENMP][NVPTX]Correct type casting for printf args + simplified shfl64 function.
    
    Summary:
    Explicitly casted printf's args to the required types + simplified
    shfl64 function.
    
    Reviewers: gtbercea, kkwli0
    
    Subscribers: guansong, jfb, caomhin, openmp-commits
    
    Differential Revision: https://reviews.llvm.org/D55379
    

    Changed files

    • openmp/libomptarget/deviceRTLs/nvptx/src/cancel.cu
    • openmp/libomptarget/deviceRTLs/nvptx/src/data_sharing.cu
    • openmp/libomptarget/deviceRTLs/nvptx/src/debug.h
    • openmp/libomptarget/deviceRTLs/nvptx/src/libcall.cu
    • openmp/libomptarget/deviceRTLs/nvptx/src/loop.cu
    • openmp/libomptarget/deviceRTLs/nvptx/src/omptarget-nvptx.cu
    • openmp/libomptarget/deviceRTLs/nvptx/src/parallel.cu
    • openmp/libomptarget/deviceRTLs/nvptx/src/reduction.cu
    • openmp/libomptarget/deviceRTLs/nvptx/src/supporti.h
    • openmp/libomptarget/deviceRTLs/nvptx/src/sync.cu
    • openmp/libomptarget/deviceRTLs/nvptx/src/task.cu
  9. [libcxx] Fix incorrect XFAILs for chrono tests on old macos deployment targets

    Changed by Louis Dionne - ldionneohnoyoudont@apple.com
    Changed at Thursday, 06-Dec-18 19:24:20 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision d6e65ac8632d378852f324acec399164bcad313e

    Comments

    [libcxx] Fix incorrect XFAILs for chrono tests on old macos deployment targets
    
    The tests were marked to fail based on the 'availability' LIT feature.
    However, those tests should really only be failing when we run them
    against the dylibs that were deployed on macosx10.7 and macosx10.8,
    which the deployment target has nothing to do with.
    
    This caused the tests to unexpectedly pass when running the tests
    with deployment target macosx10.{7,8} but running with a recent dylib.
    

    Changed files

    • libcxx/test/std/utilities/time/time.clock/time.clock.hires/consistency.pass.cpp
    • libcxx/test/std/utilities/time/time.clock/time.clock.steady/consistency.pass.cpp
    • libcxx/test/std/utilities/time/time.clock/time.clock.system/consistency.pass.cpp
  10. Fix the Xcode project build for the addition of NativePDB/DWARFLocationExpression.{h,cpp}

    Changed by Jim Ingham - jinghamohnoyoudont@apple.com
    Changed at Thursday, 06-Dec-18 19:23:21 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 778f0dfd4bfd0dd6ad76df6149d4ef4b0b3b2abc

    Comments

    Fix the Xcode project build for the addition of NativePDB/DWARFLocationExpression.{h,cpp}
    

    Changed files

    • lldb/lldb.xcodeproj/project.pbxproj
  11. [DAGCombiner] don't hoist logic op if operands have other uses, part 2

    Changed by Sanjay Patel - spatelohnoyoudont@rotateright.com
    Changed at Thursday, 06-Dec-18 19:18:56 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 584b59964195f11986cd099acb43989a18e899a9

    Comments

    [DAGCombiner] don't hoist logic op if operands have other uses, part 2
    
    The PPC test with 2 extra uses seems clearly better by avoiding this transform.
    With 1 extra use, we also prevent an extra register move (although that might
    be an RA problem). The general rule should be to only make a change here if
    it is always profitable. The x86 diffs are all neutral.
    

    Changed files

    • llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    • llvm/test/CodeGen/PowerPC/hoist-logic.ll
    • llvm/test/CodeGen/X86/packss.ll
    • llvm/test/CodeGen/X86/vector-shift-ashr-128.ll
    • llvm/test/CodeGen/X86/vector-shift-ashr-256.ll
    • llvm/test/CodeGen/X86/vector-shift-ashr-sub128.ll
  12. Fix Wdocumentation warning. NFCI.

    Changed by Simon Pilgrim - llvm-devohnoyoudont@redking.me.uk
    Changed at Thursday, 06-Dec-18 19:17:28 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision e76811b8ad4ea93f41f0b3f908f37100e83240c6

    Comments

    Fix Wdocumentation warning. NFCI.
    

    Changed files

    • llvm/lib/CodeGen/MachineOutliner.cpp
  13. [PowerPC] add tests for hoisting bitwise logic; NFC

    Changed by Sanjay Patel - spatelohnoyoudont@rotateright.com
    Changed at Thursday, 06-Dec-18 19:05:19 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 8e243cdda3e54d875c56e919d31b47c8eb8952a1

    Comments

    [PowerPC] add tests for hoisting bitwise logic; NFC
    

    Changed files

    • llvm/test/CodeGen/PowerPC/hoist-logic.ll