Home | Search | fuchsia - Builders
Login

Builder clang-arm64-linux Build 8927840576582556848

Results:

Success

Trigger Info:

Projectfuchsia
Revisionc43f8cda3854de2634936058d9af330c81c1516a
Got Revision18890f471c420411aa3c989e104d090966ec9dbf

Execution:

Steps and Logfiles:

Show:
  1. ( 56 ms ) setup_build
    running recipe: "clang_toolchain"
  2. ( 453 ms ) create pkg dir
  3. ( 437 ms ) makedirs
  4. ( 63 ms ) git init
  5. ( 11 ms ) git remote
  6. ( 4 mins 33 secs ) git fetch
  7. ( 7 secs ) git checkout
  8. ( 22 ms ) git rev-parse
  9. ( 247 ms ) git clean
  10. ( 88 ms ) submodule sync
  11. ( 74 ms ) submodule update
  12. ( 441 ms ) Version.inc
  13. ( 6 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-debian-32-c272138d-us-central1-b-0v1s" setup_build
branch "refs/heads/master" setup_build
buildbucket {"build": {"bucket": "luci.fuchsia.ci", "created_by": "user:luci-scheduler@appspot.gserviceaccount.com", "created_ts": 1544144763351959, "id": "8927840576582556848", "project": "fuchsia", "tags": ["builder:clang-arm64-linux", "buildset:commit/git/c43f8cda3854de2634936058d9af330c81c1516a", "buildset:commit/gitiles/fuchsia.googlesource.com/third_party/llvm-project/+/c43f8cda3854de2634936058d9af330c81c1516a", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9093270638595047776", "scheduler_job_id:fuchsia/clang-arm64-linux", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"} setup_build
buildername "clang-arm64-linux" setup_build
gcs_bucket "fuchsia-build" setup_build
packages ["topaz/packages/buildbot"] setup_build
path_config "generic" setup_build
platform "linux-arm64" 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 "c43f8cda3854de2634936058d9af330c81c1516a" setup_build
got_revision "c43f8cda3854de2634936058d9af330c81c1516a" git rev-parse

Blamelist:

  1. Nikita Popov (nikita.ppvohnoyoudont@gmail.com)
  2. Jason Molenda (jmolendaohnoyoudont@apple.com)
  3. Saleem Abdulrasool (compnerdohnoyoudont@compnerd.org)
  4. Stephen Kelly (steveireohnoyoudont@gmail.com)
  5. Sanjay Patel (spatelohnoyoudont@rotateright.com)
  6. Armando Montanez (amontanezohnoyoudont@google.com)
  7. Sanjay Patel (spatelohnoyoudont@rotateright.com)
  8. Nikita Popov (nikita.ppvohnoyoudont@gmail.com)
  9. Nikita Popov (nikita.ppvohnoyoudont@gmail.com)
  10. Sanjay Patel (spatelohnoyoudont@rotateright.com)
  11. Stephen Kelly (steveireohnoyoudont@gmail.com)
  12. Stephen Kelly (steveireohnoyoudont@gmail.com)
  13. Stella Stamenova (stilisohnoyoudont@microsoft.com)
  14. Stephen Kelly (steveireohnoyoudont@gmail.com)
  15. Stella Stamenova (stilisohnoyoudont@microsoft.com)
  16. Stephen Kelly (steveireohnoyoudont@gmail.com)
  17. Alex Lorenz (arphamanohnoyoudont@gmail.com)
  18. Nico Weber (nicolasweberohnoyoudont@gmx.de)
  19. Nico Weber (nicolasweberohnoyoudont@gmx.de)
  20. Craig Topper (craig.topperohnoyoudont@intel.com)
  21. Sanjay Patel (spatelohnoyoudont@rotateright.com)
  22. George Karpenkov (ekarpenkovohnoyoudont@apple.com)
  23. George Karpenkov (ekarpenkovohnoyoudont@apple.com)
  24. George Karpenkov (ekarpenkovohnoyoudont@apple.com)
  25. Sanjay Patel (spatelohnoyoudont@rotateright.com)

Timing:

Create Friday, 07-Dec-18 01:06:03 UTC
Start Friday, 07-Dec-18 01:06:10 UTC
End Friday, 07-Dec-18 02:19:44 UTC
Pending 7 secs
Execution 1 hrs 13 mins

All Changes:

  1. Revert "[DemandedBits][BDCE] Support vectors of integers"

    Changed by Nikita Popov - nikita.ppvohnoyoudont@gmail.com
    Changed at Friday, 07-Dec-18 00:42:03 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision c43f8cda3854de2634936058d9af330c81c1516a

    Comments

    Revert "[DemandedBits][BDCE] Support vectors of integers"
    
    This reverts commit r348549. Causing assertion failures during
    clang build.
    

    Changed files

    • llvm/include/llvm/Analysis/DemandedBits.h
    • llvm/lib/Analysis/DemandedBits.cpp
    • llvm/lib/Transforms/Scalar/BDCE.cpp
    • llvm/test/Analysis/DemandedBits/vectors.ll
    • llvm/test/Transforms/BDCE/vectors.ll
  2. Change the amount of data that Platform::PutFile will try to transfer

    Changed by Jason Molenda - jmolendaohnoyoudont@apple.com
    Changed at Friday, 07-Dec-18 00:35:26 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision aa22524a77694302bf02a700cf20194839cd9f09

    Comments

    Change the amount of data that Platform::PutFile will try to transfer
    in one packet from 1k bytes to 16k bytes.  Sending a large file to an
    iOS device directly connected by USB cable, to lldb-server running in
    platform mode, this speeds up the file xfer by 77%.  Sending the file
    in 32k blocks speeds up the file xfer by 80% versus 1k blocks, starting
    with 16k to make sure we don't have any problems with android testing.
    
    We may not have the same perf characteristics over ethernet, but with
    USB it's faster to send fewer larger packets than many small packets.
    

    Changed files

    • lldb/source/Target/Platform.cpp
  3. Host: remove Yield on Windows

    Changed by Saleem Abdulrasool - compnerdohnoyoudont@compnerd.org
    Changed at Friday, 07-Dec-18 00:31:34 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 3aaf15d81c054c766a1e1b9a1fe9e4ac6f503924

    Comments

    Host: remove Yield on Windows
    
    Windows provides a Yield function-like macro that allows a thread to
    yield the CPU.  However, this conflicts with `Yield` in swift.  Undefine
    `Yield` to allow building lldb with swift support.
    

    Changed files

    • lldb/include/lldb/Host/windows/windows.h
  4. Add test for InitListExpr

    Changed by Stephen Kelly - steveireohnoyoudont@gmail.com
    Changed at Friday, 07-Dec-18 00:08:14 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision d1aa9f9918053f582e7a9fb68f2b21ca81b5aef8

    Comments

    Add test for InitListExpr
    

    Changed files

    • clang/test/AST/ast-dump-stmt.cpp
  5. [DAGCombiner] use root SDLoc for all nodes created by logic fold

    Changed by Sanjay Patel - spatelohnoyoudont@rotateright.com
    Changed at Friday, 07-Dec-18 00:01:57 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 8f8325032b7c2e07dffef032b5372227ffb9d80f

    Comments

    [DAGCombiner] use root SDLoc for all nodes created by logic fold
    
    If this is not a valid way to assign an SDLoc, then we get this
    wrong all over SDAG.
    
    I don't know enough about the SDAG to explain this. IIUC, theoretically,
    debug info is not supposed to affect codegen. But here it has clearly
    affected 3 different targets, and the x86 change is an actual improvement.
    

    Changed files

    • llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    • llvm/test/CodeGen/AArch64/unfold-masked-merge-scalar-variablemask.ll
    • llvm/test/CodeGen/PowerPC/setcc-logic.ll
    • llvm/test/CodeGen/X86/unfold-masked-merge-vector-variablemask.ll
  6. [llvm-tapi] Don't override SequenceTraits for std::string

    Changed by Armando Montanez - amontanezohnoyoudont@google.com
    Changed at Thursday, 06-Dec-18 23:59:32 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 1b057a4f98390f8b53218cb8f0e2b49acbcd4581

    Comments

    [llvm-tapi] Don't override SequenceTraits for std::string
    
    Change the ELF YAML implementation of TextAPI so NeededLibs uses flow
    sequence vector correctly instead of overriding the YAML implementation
    for std::vector<std::string>>.
    
    This should fix the test failure with the LLVM_LINK_LLVM_DYLIB build mentioned in D55381.
    
    Still passes existing tests that cover this.
    
    Differential Revision: https://reviews.llvm.org/D55390
    

    Changed files

    • llvm/lib/TextAPI/ELF/TBEHandler.cpp
  7. [DAGCombiner] don't bother saving a SDLoc for a node that's dead; NFCI

    Changed by Sanjay Patel - spatelohnoyoudont@rotateright.com
    Changed at Thursday, 06-Dec-18 23:53:58 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 2404506e324d2c0be3a23ef585e12d54c4844eeb

    Comments

    [DAGCombiner] don't bother saving a SDLoc for a node that's dead; NFCI
    
    We shouldn't care about the debug location for a node that
    we're creating, but attaching the root of the pattern should
    be the best effort. (If this is not true, then we are doing
    it wrong all over the SDAG).
    
    This is no-functional-change-intended, and there are no
    regression test diffs...and that's what I expected. But
    there's a similar line above this diff, where those
    assumptions apparently do not hold.
    

    Changed files

    • llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  8. [DemandedBits][BDCE] Support vectors of integers

    Changed by Nikita Popov - nikita.ppvohnoyoudont@gmail.com
    Changed at Thursday, 06-Dec-18 23:50:32 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision e5cd92a046c4e3f4ac111470790f3427afdfdc1b

    Comments

    [DemandedBits][BDCE] Support vectors of integers
    
    DemandedBits and BDCE currently only support scalar integers. This
    patch extends them to also handle vector integer operations. In this
    case bits are not tracked for individual vector elements, instead a
    bit is demanded if it is demanded for any of the elements. This matches
    the behavior of computeKnownBits in ValueTracking and
    SimplifyDemandedBits in InstCombine.
    
    The getDemandedBits() method can now only be called on instructions that
    have integer or vector of integer type. Previously it could be called on
    any sized instruction (even if it was not particularly useful). The size
    of the return value is now always the scalar size in bits (while
    previously it was the type size in bits).
    
    Differential Revision: https://reviews.llvm.org/D55297
    

    Changed files

    • llvm/include/llvm/Analysis/DemandedBits.h
    • llvm/lib/Analysis/DemandedBits.cpp
    • llvm/lib/Transforms/Scalar/BDCE.cpp
    • llvm/test/Analysis/DemandedBits/vectors.ll
    • llvm/test/Transforms/BDCE/vectors.ll
  9. [BDCE] Add tests for BDCE applied to vector instructions; NFC

    Changed by Nikita Popov - nikita.ppvohnoyoudont@gmail.com
    Changed at Thursday, 06-Dec-18 23:50:19 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 970d12d97b5335bfb0e69f51edc2bd542edd808d

    Comments

    [BDCE] Add tests for BDCE applied to vector instructions; NFC
    
    These are baseline tests for D55297.
    

    Changed files

    • llvm/test/Transforms/BDCE/vectors.ll
  10. [DAGCombiner] more clean up in hoistLogicOpWithSameOpcodeHands(); NFC

    Changed by Sanjay Patel - spatelohnoyoudont@rotateright.com
    Changed at Thursday, 06-Dec-18 23:39:28 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 18024c83a424577bd342dcaed425e73e0bd63da8

    Comments

    [DAGCombiner] more clean up in hoistLogicOpWithSameOpcodeHands(); NFC
    
    This code can still misbehave.
    

    Changed files

    • llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  11. NFC: Move VisitExpr code to dumpStmt

    Changed by Stephen Kelly - steveireohnoyoudont@gmail.com
    Changed at Thursday, 06-Dec-18 23:33:33 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision c70aa6cc61372c880acab337c711701168386cad

    Comments

    NFC: Move VisitExpr code to dumpStmt
    
    Summary:
    The call is duplicated in the handlers of all Expr subclasses.
    
    This change makes it easy to split statement handling out to
    TextNodeDumper.
    
    Reviewers: aaron.ballman
    
    Subscribers: cfe-commits
    
    Differential Revision: https://reviews.llvm.org/D55339
    

    Changed files

    • clang/lib/AST/ASTDumper.cpp
  12. NFC: Move VisitStmt code to dumpStmt

    Changed by Stephen Kelly - steveireohnoyoudont@gmail.com
    Changed at Thursday, 06-Dec-18 23:33:27 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision ea0c8739376a02968b21f609155a19cad36aa479

    Comments

    NFC: Move VisitStmt code to dumpStmt
    
    Summary: This call is duplicated in Visits of all direct subclasses of Stmt.
    
    Reviewers: aaron.ballman
    
    Subscribers: cfe-commits
    
    Differential Revision: https://reviews.llvm.org/D55338
    

    Changed files

    • clang/lib/AST/ASTDumper.cpp
  13. [lit] Use the build.py script in the case-insensitive test

    Changed by Stella Stamenova - stilisohnoyoudont@microsoft.com
    Changed at Thursday, 06-Dec-18 23:25:37 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 05fefd8ee184f27eac88108957e3eb132ef0856e

    Comments

    [lit] Use the build.py script in the case-insensitive test
    
    This makes the test build correctly regardless of whether we use VS or ninja to run the tests
    

    Changed files

    • lldb/lit/Breakpoint/case-insensitive.test
  14. Add more expected content to match in test

    Changed by Stephen Kelly - steveireohnoyoudont@gmail.com
    Changed at Thursday, 06-Dec-18 23:23:10 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 476cf00dd429913287dfc7aa3907282c869e8bc7

    Comments

    Add more expected content to match in test
    

    Changed files

    • clang/test/AST/ast-dump-decl.m
  15. [pecoff] Use PATH_MAX instead of MAX_PATH

    Changed by Stella Stamenova - stilisohnoyoudont@microsoft.com
    Changed at Thursday, 06-Dec-18 23:22:46 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 1e2737cdbd0779bd306a86c9e8d4e5bfaf97ac58

    Comments

    [pecoff] Use PATH_MAX instead of MAX_PATH
    
    PATH_MAX is defined on all platforms while MAX_PATH is Windows-specific
    

    Changed files

    • lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp
  16. Use relative line offsets in test

    Changed by Stephen Kelly - steveireohnoyoudont@gmail.com
    Changed at Thursday, 06-Dec-18 22:51:51 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 3debcabcb13823db010cd4b027eb508c5521969d

    Comments

    Use relative line offsets in test
    

    Changed files

    • clang/test/AST/dump.cpp
  17. [frontend][darwin] warn_stdlibcxx_not_found: supress warning for preprocessed input

    Changed by Alex Lorenz - arphamanohnoyoudont@gmail.com
    Changed at Thursday, 06-Dec-18 22:45:58 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 66dc290b9efb03b2f34aace54b070c710d04f181

    Comments

    [frontend][darwin] warn_stdlibcxx_not_found: supress warning for preprocessed input
    
    Addresses second post-commit feedback for r335081 from Nico
    

    Changed files

    • clang/lib/Frontend/CompilerInvocation.cpp
    • clang/test/Frontend/warning-stdlibcxx-darwin.cpp
  18. Run `git ls-files '*.gn' '*.gni' | xargs -n 1 gn format`.

    Changed by Nico Weber - nicolasweberohnoyoudont@gmx.de
    Changed at Thursday, 06-Dec-18 22:40:05 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 5e89597c4d2c9be542b3de3908e7f57b0072a95a

    Comments

    Run `git ls-files '*.gn' '*.gni' | xargs -n 1 gn format`.
    

    Changed files

    • llvm/utils/gn/secondary/llvm/lib/Support/BUILD.gn
  19. [gn build] merge r348505.

    Changed by Nico Weber - nicolasweberohnoyoudont@gmx.de
    Changed at Thursday, 06-Dec-18 22:36:16 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 9f66a2cf8042b6204f82237b86154f3fb382d0e7

    Comments

    [gn build] merge r348505.
    

    Changed files

    • llvm/utils/gn/secondary/llvm/lib/DebugInfo/CodeView/BUILD.gn
  20. [X86] Directly create ADC/SBB nodes instead of using ADD/SUB with (and SETCC_CARRY, 1)

    Changed by Craig Topper - craig.topperohnoyoudont@intel.com
    Changed at Thursday, 06-Dec-18 22:26:59 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 37a7fd285871b3be56058a7d835a8104df237b0b

    Comments

    [X86] Directly create ADC/SBB nodes instead of using ADD/SUB with (and SETCC_CARRY, 1)
    
    This addresses a FIXME and avoids depending on an isel pattern match I think. I've remove the isel patterns too since he have no lit tests left that cover them. Hopefully that really means they are unused.
    
    I'm trying to decide if we need SETCC_CARRY. This removes one of its usages.
    
    Differential Revision: https://reviews.llvm.org/D55355
    

    Changed files

    • llvm/lib/Target/X86/X86ISelLowering.cpp
    • llvm/lib/Target/X86/X86InstrCompiler.td
  21. [DAGCombiner] don't group bswap with casts in logic hoisting fold

    Changed by Sanjay Patel - spatelohnoyoudont@rotateright.com
    Changed at Thursday, 06-Dec-18 22:10:44 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 75bfa2556eaf7f5d18b320cec3ceedeebf940750

    Comments

    [DAGCombiner] don't group bswap with casts in logic hoisting fold
    
    This was probably organized as it was because bswap is a unary op.
    But that's where the similarity to the other opcodes ends. We should
    not limit this transform to scalars, and we should not try it if
    either input has other uses. This is another step towards trying to
    clean this whole function up to prevent it from causing infinite loops
    and memory explosions.
    
    Earlier commits in this series:
    rL348501
    rL348508
    rL348518
    

    Changed files

    • llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    • llvm/test/CodeGen/X86/bswap-vector.ll
  22. [analyzer] Rely on os_consumes_this attribute to signify that the method call consumes a reference for "this"

    Changed by George Karpenkov - ekarpenkovohnoyoudont@apple.com
    Changed at Thursday, 06-Dec-18 22:07:12 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 862c390d44c6ca75a46b0537ce910d3f00789287

    Comments

    [analyzer] Rely on os_consumes_this attribute to signify that the method call consumes a reference for "this"
    
    Differential Revision: https://reviews.llvm.org/D55158
    

    Changed files

    • clang/include/clang/StaticAnalyzer/Core/RetainSummaryManager.h
    • clang/lib/StaticAnalyzer/Core/RetainSummaryManager.cpp
    • clang/test/Analysis/osobject-retain-release.cpp
  23. [attributes] Add an attribute os_consumes_this, with similar semantics to ns_consumes_self

    Changed by George Karpenkov - ekarpenkovohnoyoudont@apple.com
    Changed at Thursday, 06-Dec-18 22:06:59 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 0f516a32c556d34eac2f642dffd6e2b219d09d1b

    Comments

    [attributes] Add an attribute os_consumes_this, with similar semantics to ns_consumes_self
    
    The attribute specifies that the call of the C++ method consumes a
    reference to "this".
    
    Differential Revision: https://reviews.llvm.org/D55155
    

    Changed files

    • clang/include/clang/Basic/Attr.td
    • clang/include/clang/Basic/AttrDocs.td
    • clang/lib/Sema/SemaDeclAttr.cpp
    • clang/test/Sema/attr-osobject.cpp
  24. [analyzer] Fix an infinite recursion bug while checking parent methods in RetainCountChecker

    Changed by George Karpenkov - ekarpenkovohnoyoudont@apple.com
    Changed at Thursday, 06-Dec-18 22:06:44 UTC
    Repository https://fuchsia.googlesource.com/third_party/llvm-project
    Branch
    Revision 9a7d1debf716a11786a29b13686fe657fd31d324

    Comments

    [analyzer] Fix an infinite recursion bug while checking parent methods in RetainCountChecker
    
    Differential Revision: https://reviews.llvm.org/D55351
    

    Changed files

    • clang/lib/StaticAnalyzer/Core/RetainSummaryManager.cpp
    • clang/test/Analysis/osobject-retain-release.cpp
  25. [x86] add test for vector bitwise-logic-of-bswaps; NFC

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

    Comments

    [x86] add test for vector bitwise-logic-of-bswaps; NFC
    

    Changed files

    • llvm/test/CodeGen/X86/bswap-vector.ll