Started 2 yr 8 mo ago
Took 1 hr 39 min on green-dragon-09

Success Build #5 (Aug 3, 2017 11:34:37 AM)

Revisions
  • http://llvm.org/svn/llvm-project/clang-tools-extra/branches/release_50 : 309593
  • http://llvm.org/svn/llvm-project/libcxx/branches/release_50 : 309958
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 303903
  • http://llvm.org/svn/llvm-project/zorg/trunk : 309970
  • http://llvm.org/svn/llvm-project/llvm/branches/release_50 : 309965
  • http://llvm.org/svn/llvm-project/compiler-rt/branches/release_50 : 308446
  • http://llvm.org/svn/llvm-project/cfe/branches/release_50 : 309954
Changes
  1. [zorg] Remove test_polly_format step from Polly buildbot

    Summary:
    The test_polly_format does "make polly-check-format" which is already covered by
    the "make check-polly" step.

    Reviewers: mgrang, zinob, grosser, gkistanova, vleschuk, bollu

    Reviewed By: grosser

    Subscribers: llvm-commits

    Differential Revision: https://reviews.llvm.org/D36276 (detail)
    by pzheng
  2. Merging r309928:
    ------------------------------------------------------------------------
    r309928 | ewancrawford | 2017-08-03 02:23:03 -0700 (Thu, 03 Aug 2017) | 56 lines

    [Cloning] Move distinct GlobalVariable debug info metadata in CloneModule

    Duplicating the distinct Subprogram and CU metadata nodes seems like the incorrect thing to do in CloneModule for GlobalVariable debug info. As it results in the scope of the GlobalVariable DI no longer being consistent with the rest of the module, and the new CU is absent from llvm.dbg.cu.

    Fixed by adding RF_MoveDistinctMDs to MapMetadata flags for GlobalVariables.

    Current unit test IR after clone:
    ```
    @gv = global i32 1, comdat($comdat), !dbg !0, !type !5

    define private void @f() comdat($comdat) personality void ()* @persfn !dbg !14 {

    !llvm.dbg.cu = !{!10}

    !0 = !DIGlobalVariableExpression(var: !1)
    !1 = distinct !DIGlobalVariable(name: "gv", linkageName: "gv", scope: !2, file: !3, line: 1, type: !9, isLocal: false, isDefinition: true)
    !2 = distinct !DISubprogram(name: "f", linkageName: "f", scope: null, file: !3, line: 4, type: !4, isLocal: true, isDefinition: true, scopeLine: 3, isOptimized: false, unit: !6, variables: !5)
    !3 = !DIFile(filename: "filename.c", directory: "/file/dir/")
    !4 = !DISubroutineType(types: !5)
    !5 = !{}
    !6 = distinct !DICompileUnit(language: DW_LANG_C99, file: !7, producer: "CloneModule", isOptimized: false, runtimeVersion: 0, emissionKind: FullDebug, enums: !5, globals: !8)
    !7 = !DIFile(filename: "filename.c", directory: "/file/dir")
    !8 = !{!0}
    !9 = !DIBasicType(tag: DW_TAG_unspecified_type, name: "decltype(nullptr)")
    !10 = distinct !DICompileUnit(language: DW_LANG_C99, file: !7, producer: "CloneModule", isOptimized: false, runtimeVersion: 0, emissionKind: FullDebug, enums: !5, globals: !11)
    !11 = !{!12}
    !12 = !DIGlobalVariableExpression(var: !13)
    !13 = distinct !DIGlobalVariable(name: "gv", linkageName: "gv", scope: !14, file: !3, line: 1, type: !9, isLocal: false, isDefinition: true)
    !14 = distinct !DISubprogram(name: "f", linkageName: "f", scope: null, file: !3, line: 4, type: !4, isLocal: true, isDefinition: true, scopeLine: 3, isOptimized: false, unit: !10, variables: !5)
    ```

    Patched IR after clone:
    ```
    @gv = global i32 1, comdat($comdat), !dbg !0, !type !5

    define private void @f() comdat($comdat) personality void ()* @persfn !dbg !2 {

    !llvm.dbg.cu = !{!6}

    !0 = !DIGlobalVariableExpression(var: !1)
    !1 = distinct !DIGlobalVariable(name: "gv", linkageName: "gv", scope: !2, file: !3, line: 1, type: !9, isLocal: false, isDefinition: true)
    !2 = distinct !DISubprogram(name: "f", linkageName: "f", scope: null, file: !3, line: 4, type: !4, isLocal: true, isDefinition: true, scopeLine: 3, isOptimized: false, unit: !6, variables: !5)
    !3 = !DIFile(filename: "filename.c", directory: "/file/dir/")
    !4 = !DISubroutineType(types: !5)
    !5 = !{}
    !6 = distinct !DICompileUnit(language: DW_LANG_C99, file: !7, producer: "CloneModule", isOptimized: false, runtimeVersion: 0, emissionKind: FullDebug, enums: !5, globals: !8)
    !7 = !DIFile(filename: "filename.c", directory: "/file/dir")
    !8 = !{!0}
    !9 = !DIBasicType(tag: DW_TAG_unspecified_type, name: "decltype(nullptr)")
    ```

    Reviewers: aprantl, probinson, dblaikie, echristo, loladiro
    Reviewed By: aprantl
    Subscribers: llvm-commits
    Differential Revision: https://reviews.llvm.org/D36082

    ------------------------------------------------------------------------ (detail)
    by hans
  3. Merging r309917:
    ------------------------------------------------------------------------
    r309917 | ericwf | 2017-08-02 19:50:43 -0700 (Wed, 02 Aug 2017) | 4 lines

    Add system header pragma to BSD locale fallback headers.

    This prevent leaking warnings to the user about use of C++11
    extensions in C++03.
    ------------------------------------------------------------------------ (detail)
    by hans
  4. Merging r309651 and r309849:
    ------------------------------------------------------------------------
    r309651 | inouehrs | 2017-07-31 20:32:15 -0700 (Mon, 31 Jul 2017) | 16 lines

    [StackColoring] Update AliasAnalysis information in stack coloring pass

    Stack coloring pass need to maintain AliasAnalysis information when merging stack slots of different types.
    Actually, there is a FIXME comment in StackColoring.cpp

    // FIXME: In order to enable the use of TBAA when using AA in CodeGen,
    // we'll also need to update the TBAA nodes in MMOs with values
    // derived from the merged allocas.

    But, TBAA has been already enabled in CodeGen without fixing this pass.
    The incorrect TBAA metadata results in recent failures in bootstrap test on ppc64le (PR33928) by allowing unsafe instruction scheduling.
    Although we observed the problem on ppc64le, this is a platform neutral issue.

    This patch makes the stack coloring pass maintains AliasAnalysis information when merging multiple stack slots.
    ------------------------------------------------------------------------

    ------------------------------------------------------------------------
    r309849 | inouehrs | 2017-08-02 11:16:32 -0700 (Wed, 02 Aug 2017) | 19 lines

    [StackColoring] Update AliasAnalysis information in stack coloring pass (part 2)

    This patch is update after the first patch (https://reviews.llvm.org/rL309651) based on the post-commit comments.

    Stack coloring pass need to maintain AliasAnalysis information when merging stack slots of different types.
    Actually, there is a FIXME comment in StackColoring.cpp

    // FIXME: In order to enable the use of TBAA when using AA in CodeGen,
    // we'll also need to update the TBAA nodes in MMOs with values
    // derived from the merged allocas.

    But, TBAA has been already enabled in CodeGen without fixing this pass.
    The incorrect TBAA metadata results in recent failures in bootstrap test on ppc64le (PR33928) by allowing unsafe instruction scheduling.
    Although we observed the problem on ppc64le, this is a platform neutral issue.

    This patch makes the stack coloring pass maintains AliasAnalysis information when merging multiple stack slots.

    This patch fixes PR33928.
    ------------------------------------------------------------------------ (detail)
    by hans
  5. Merging r309930:
    ------------------------------------------------------------------------
    r309930 | sdardis | 2017-08-03 02:38:46 -0700 (Thu, 03 Aug 2017) | 19 lines

    [SelectionDAG] Resolve PR33978.

    rL306209 taught SelectionDAG how to add the dereferenceable flag when
    expanding memcpy and memmove. The fix however contained a nit where
    the offset + size was constructed as an APInt of PointerSize rather
    than PointerSizeInBits.

    This lead to isDereferenceableAndAlignedPointer() get truncated values or
    values which would be sign extended within that function leading to
    incorrect results.

    Thanks to Alex Crichton for reporting the issue!

    This resolves PR33978.

    Reviewers: inouehrs

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

    ------------------------------------------------------------------------ (detail)
    by hans
  6. Merging r308996:
    ------------------------------------------------------------------------
    r308996 | gornishanov | 2017-07-25 11:01:49 -0700 (Tue, 25 Jul 2017) | 9 lines

    [coroutines] Add serialization/deserialization of coroutines

    Reviewers: rsmith

    Reviewed By: rsmith

    Subscribers: EricWF, cfe-commits

    Differential Revision: https://reviews.llvm.org/D35383
    ------------------------------------------------------------------------ (detail)
    by hans

Started by an SCM change (5 times)

This run spent:

  • 2 hr 18 min waiting;
  • 1 hr 39 min build duration;
  • 3 hr 57 min total from scheduled to completion.
Test Result (no failures)