Started 4 mo 8 days ago
Took 1 hr 11 min on green-dragon-23

Success Build #2966 (Aug 7, 2019 10:38:09 AM)

Build Artifacts
test_logs.tgz48.87 KB view
Revisions
  • http://llvm.org/svn/llvm-project/llvm/trunk : 368191
  • http://llvm.org/svn/llvm-project/cfe/trunk : 368188
  • http://llvm.org/svn/llvm-project/lldb/trunk : 368182
  • http://llvm.org/svn/llvm-project/debuginfo-tests/trunk : 364589
  • http://llvm.org/svn/llvm-project/libcxx/trunk : 368088
Changes
  1. [DataLayout] Check StackNatural and FunctionPtr alignments.

    MaybeAlignment asserts that the passed in value is == 0 or a power of 2.

    Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=16272

    Reviewers: michaelplatings, gchatelet, jakehehrlich, jfb

    Reviewed By: gchatelet

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D65858 (detail)
    by fhahn
  2. DebugInfo/DWARF: Normalize DWARFObject members on the DWARF spec section names

    Some of these names were abbreviated, some were not, some pluralised,
    some not. Made the API difficult to use - since it's an exact 1:1
    mapping to the DWARF sections - use those names (changing underscore
    separation for camel casing). (detail)
    by dblaikie
  3. [clang][NFC] Fix typo in matcher comment

    Also updates corresponding html doc. (detail)
    by ymandel
  4. Support: Remove needless allocation when getMainExecutable() calls readlink()

    We built a StringRef from a string literal which we then converted to a
    std::string to call c_str().  Just use a pointer to the string literal
    instead of a StringRef.

    No behavior change.

    Differential Revision: https://reviews.llvm.org/D65890 (detail)
    by nico
  5. Recommit r368081 "[X86] Add more extract subvector cost model tests for smaller element sizes and smaller than 128-bit vectors." (detail)
    by ctopper
  6. Recommit r368079 "[X86] Remove uses of the -x86-experimental-vector-widening-legalization flag from test/CodeGen/X86/" (detail)
    by ctopper
  7. Recommit r367901 "[X86] Enable -x86-experimental-vector-widening-legalization by default."

    The assert that caused this to be reverted should be fixed now.

    Original commit message:

    This patch changes our defualt legalization behavior for 16, 32, and
    64 bit vectors with i8/i16/i32/i64 scalar types from promotion to
    widening. For example, v8i8 will now be widened to v16i8 instead of
    promoted to v8i16. This keeps the elements widths the same and pads
    with undef elements. We believe this is a better legalization strategy.
    But it carries some issues due to the fragmented vector ISA. For
    example, i8 shifts and multiplies get widened and then later have
    to be promoted/split into vXi16 vectors.

    This has the potential to cause regressions so we wanted to get
    it in early in the 10.0 cycle so we have plenty of time to
    address them.

    Next steps will be to merge tests that explicitly test the command
    line option. And then we can remove the option and its associated
    code. (detail)
    by ctopper
  8. [Driver] Expand the executable path in the target create output

    Resolve the path in the target create output. This is nice when passing
    relative paths to the lldb command line driver.

      $ lldb ./binary
      (lldb) target create "./binary"
      Current executable set to '/absolute/path/to/binary' (x86_64).

    This change only affects the target create output and does not change
    the debugger's behavior. It doesn't resolve symbolic links so it won't
    cause confusing when debugging something like clang++ that's symlinked
    to clang.

    Differential revision: https://reviews.llvm.org/D65611 (detail)
    by Jonas Devlieghere
  9. [ExecutionContext] Return the target/process byte order.

    Currently ExecutionContext::GetByteOrder() always returns the host byte
    order. This seems like a simple mistake: the return keyword appears to
    have been omitted by accident. This patch fixes that and adds a unit
    test.

    Bugreport: https://llvm.org/PR37950

    Differential revision: https://reviews.llvm.org/D48704 (detail)
    by Jonas Devlieghere
  10. [ARM] Expand CTPOP intrinsic for MVE (detail)
    by oliverlars
  11. gn build: Merge r368158 (detail)
    by nico
  12. gn build: Merge r368119 (detail)
    by nico
  13. [InstCombine] Add a TODO comment (detail)
    by foad
  14. [InstCombine] Propagate fast math flags through selects

    Summary:
    In SimplifySelectsFeedingBinaryOp, propagate fast math flags from the
    outer op into both arms of the new select, to take advantage of
    simplifications that require fast math flags.

    Reviewers: mcberg2017, majnemer, spatel, arsenm, xbolva00

    Subscribers: wdng, javed.absar, kristof.beyls, hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D65658 (detail)
    by foad
  15. [UpdateTestChecks] Update tests option

    Summary:
    Port of new feature introduced https://reviews.llvm.org/D65610 to other update scripts.

    - update_*_checks.py: add an alias -u for --update-only
    - port --update-only to other update_*_test_checks.py scripts
    - update script aborts if the test file was generated by another update_*_test_checks.py utility

    Reviewers: lebedev.ri, RKSimon, MaskRay, reames, gbedwell

    Reviewed By: MaskRay

    Subscribers: llvm-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D65793 (detail)
    by xbolva00
  16. Remove LLVM mutexes from clang in favor of std::mutex

    None of those need to be recursive mutexes. No functionality change
    intended. (detail)
    by d0k
  17. [OPENMP]Set default version to OpenMP 4.5.

    Since clang fully supports OpenMP 4.5, set the default version to 4.5
    instead of 3.1. (detail)
    by abataev
  18. [EarlyCSE] Add support for unary FNeg to EarlyCSE

    Differential Revision: https://reviews.llvm.org/D65815 (detail)
    by mcinally
  19. [OPENMP]Add standard macro value _OPENMP for OpenMP 5.0.

    According to the OpenMP standard, compiler must define _OPENMP macro,
    which has value in format yyyymm, where yyyy is the year of the standard
    and mm is the month of the standard. For OpenMP 5.0 this value must be
    set to 201811. (detail)
    by abataev
  20. ProcessElfCore: Remove linux and freebsd NT_*** constants

    These are already defined in llvm/BinaryFormat/ELF.h. Leaving the NetBSD
    and OpenBSD constants as-is, as they have no llvm counterparts. (detail)
    by labath
  21. [RISCV][NFC] Document RISC-V-specific assembly constraints (detail)
    by lenary
  22. GlobalISel: factor common code from translateCall and translateInvoke. NFC. (detail)
    by Tim Northover
  23. [X86] EltsFromConsecutiveLoads - early out for non-byte sized memory (PR42909)

    Don't attempt to merge loads for types that aren't modulo 8-bits. (detail)
    by rksimon
  24. [AArch64][WinCFI] Do not pair callee-save instructions in LoadStoreOptimizer

    Prevent the LoadStoreOptimizer from pairing any load/store instructions with
    instructions from the prologue/epilogue if the CFI information has encoded the
    operations as separate instructions.  This would otherwise lead to a mismatch
    of the actual prologue size from the size as recorded in the Windows CFI.

    Reviewers: efriedma, mstorsjo, ssijaric

    Reviewed By: efriedma

    Differential Revision: https://reviews.llvm.org/D65817 (detail)
    by s.desmalen
  25. [ASTImporter] Do not import FunctionTemplateDecl in record twice.

    Summary:
    For functions there is a check to not duplicate the declaration if it is in a
    record (class). For function templates there was no similar check, if a
    template (in the same class) was imported multiple times the
    FunctionTemplateDecl was created multiple times with the same templated
    FunctionDecl. This can result in problems with the declaration chain.

    Reviewers: martong, a.sidorin, shafik, a_sidorin

    Reviewed By: a_sidorin

    Subscribers: rnkovacs, dkrupp, Szelethus, gamesh411, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D65203 (detail)
    by balazske
  26. [mips] Make a couple of class methods plain static functions. NFC (detail)
    by atanasyan
  27. [mips] Use isMicroMips() function to check enabled feature flag. NFC (detail)
    by atanasyan
  28. [Mips] Instruction `sc` now accepts symbol as an argument

    Function MipsAsmParser::expandMemInst() did not properly handle
    instruction `sc` with a symbol as an argument because first argument
    would be counted twice. We add additional checks and handle this case
    separately.

    Patch by Mirko Brkusanin.

    Differential Revision: https://reviews.llvm.org/D64252 (detail)
    by atanasyan
  29. ObjectFileELF: Remove NT_*** constants

    llvm now has definitions of those in BinaryFormat/ELF.h. Use those
    instead. (detail)
    by labath
  30. [Support] Base SmartMutex on std::recursive_mutex

    - Remove support for non-recursive mutexes. This was unused.
    - The std::recursive_mutex is now created/destroyed unconditionally.
      Locking is still only done if threading is enabled.
    - Alias SmartScopedLock to std::lock_guard.

    This should make no semantic difference on the existing APIs. (detail)
    by d0k
  31. Replace non-recursive sys::Mutex users with std::mutex

    Also remove a use of sys::MutexImpl, that's just evil. No functionality
    change intended. (detail)
    by d0k
  32. Remove support for 32-bit offsets in utility classes (5/5)

    Differential Revision: https://reviews.llvm.org/D65641 (detail)
    by ikudrin
  33. [TargetLowering] SimplifyDemandedBits - call SimplifyMultipleUseDemandedBits for ISD::VECTOR_SHUFFLE

    In particular this helps the SSE vector shift cvttps2dq+add+shl pattern by avoiding the need for zeros in shuffle style extensions to vXi32 types as we'll be shifting out those bits anyway (detail)
    by rksimon
  34. A more robust way of testing debug_line parser near the end of module

    Summary:
    While removing -z separate-code makes lld produce place the code at the
    end of a segment right now, it's possible that future changes to the
    linker will change that, thereby removing the coverage for the changes
    in r367983. This patch adds a linker script to one of the line table
    tests, which ensures that the code (and its line table) will be placed
    at the very end of a module.

    Reviewers: MaskRay

    Subscribers: lldb-commits

    Differential Revision: https://reviews.llvm.org/D65789 (detail)
    by labath
  35. [clang] Fix mismatched args constructing AddressSpaceAttr.

    Differential Revision: https://reviews.llvm.org/D65589 (detail)
    by antonbikineev
  36. [lldb][CMake] Workaround debugserver code-signing issue in generated Xcode project

    Summary:
    Explicitly code-sign the LLDB.framework copy of debugserver in the build-tree. This is necessary, because the Xcode-specific logic in `llvm_codesign` [1] has the side-effect that Xcode code-signs after post-build steps (here: after copying debugserver over into the framework). The special case for Xcode was necessary to avoid double-signing errors in the past (see D55116 and D55816).

    [1] https://github.com/llvm/llvm-project/blob/36fb93982f0e/llvm/cmake/modules/AddLLVM.cmake#L1676

    Reviewers: jingham, davide, JDevlieghere, teemperor

    Reviewed By: JDevlieghere

    Subscribers: beanz, mgorny, lldb-commits, #lldb

    Tags: #lldb

    Differential Revision: https://reviews.llvm.org/D65566 (detail)
    by stefan.graenitz
  37. [lldb][NFC] Remove commented out code in ClangASTContext::AddMethodToCXXRecordType (detail)
    by Raphael Isemann
  38. Replace llvm::MutexGuard/UniqueLock with their standard equivalents

    All supported platforms have <mutex> now, so we don't need our own
    copies any longer. No functionality change intended. (detail)
    by d0k
  39. [lldb][CMake] Fix one more detail in r368066

    Differential Revision: https://reviews.llvm.org/D65797 (detail)
    by stefan.graenitz
  40. gsl::Owner/gsl::Pointer: Add implicit annotations for some std types

    Summary:
    Hard code gsl::Owner/gsl::Pointer for std types. The paper mentions
    some types explicitly. Generally, all containers and their iterators are
    covered. For iterators, we cover both the case that they are defined
    as an nested class or as an typedef/using. I have started to test this
    implementation against some real standard library implementations, namely
    libc++ 7.1.0, libc++ 8.0.1rc2, libstdc++ 4.6.4, libstdc++ 4.8.5,
    libstdc++ 4.9.4, libstdc++ 5.4.0, libstdc++ 6.5.0, libstdc++ 7.3.0,
    libstdc++ 8.3.0 and libstdc++ 9.1.0.

    The tests are currently here
      https://github.com/mgehre/llvm-project/blob/lifetime-ci/lifetime-attr-test.sh
      https://github.com/mgehre/llvm-project/blob/lifetime-ci/lifetime-attr-test.cpp
    I think due to their dependency on a standard library, they are not a good fit
    for clang/test/. Where else could I put them?

    Reviewers: gribozavr, xazax.hun

    Subscribers: rnkovacs, cfe-commits

    Tags: #clang

    Differential Revision: https://reviews.llvm.org/D64448 (detail)
    by mgehre
  41. [ARM] Generate MVE VHADDs/VHSUBs (detail)
    by oliverlars
  42. [lldb][NFC] Fix typo in 368066

    Differential Revision: https://reviews.llvm.org/D65797 (detail)
    by stefan.graenitz
  43. [InstCombine] Recommit: Shift amount reassociation: shl-trunc-shl pattern

    This was initially committed in r368059 but got reverted in r368084
    because there was a faulty logic in how the shift amounts type mismatch
    was being handled (it simply wasn't).

    I've added an explicit bailout before we SimplifyAddInst() - i don't think
    it's designed in general to handle differently-typed values, even though
    the actual problem only comes from ConstantExpr's.

    I have also changed the common type deduction, to not just blindly
    look past zext, but try to do that so that in the end types match.

    Differential Revision: https://reviews.llvm.org/D65380 (detail)
    by lebedevri
  44. [obj2yaml] - MIPS: move and improve testing of the e_flags

    Mips/elf-flags.yaml and Mips/elf-abi.yaml are tests that intention was to
    show that yaml2obj/obj2yaml are able to read/dump MIPS specific e_flags.

    They were not complete, contained an excessive YAML parts and were placed
    at a wrong location.

    I removed them and created the obj2yaml/elf-mips-eflags.yaml instead.

    Differential revision: https://reviews.llvm.org/D65807 (detail)
    by grimar
  45. [ELF.h] - Check the case when e_shstrndx=SHN_XINDEX, but the corresponding sh_link is broken.

    When e_shstrndx is equal to SHN_XINDEX,
    the index of the section string table section should
    be taken from the sh_link field of the section
    header at index 0.

    If sh_link is broken, e.g. contains an index that is
    larger than number of sections, then error is reported.

    This error message was untested before.

    Differential revision: https://reviews.llvm.org/D65391 (detail)
    by grimar
  46. [llvm-readelf] --notes: move 'Data size' column left by 1

    readelf -n:

    ```
    // "Data size" is not left justified
      Owner                 Data size       Description
      GNU                  0x00000010       NT_GNU_ABI_TAG (ABI version tag)
    ```

    llvm-readelf -n (before):
    ```
    // "Data size" column shifted by 1
      Owner                 Data size        Description
      GNU                   0x00000010       NT_GNU_ABI_TAG (ABI version tag)
    ```

    llvm-readelf -n (after):
    ```
      Owner                Data size        Description
      GNU                  0x00000010       NT_GNU_ABI_TAG (ABI version tag)
    ```

    This change is made to reduce the diff with readelf -n, so that it is
    slightly easier to check what features readelf implements but we don't.

    Reviewed By: grimar, jhenderson

    Differential Revision: https://reviews.llvm.org/D65847 (detail)
    by maskray
  47. [unittests] Mark private gmock headers with IWYU pragmas. NFC

    Summary: To prevent clangd from adding #include of those headers.

    Reviewers: gribozavr

    Reviewed By: gribozavr

    Subscribers: kadircet, llvm-commits, cfe-commits

    Tags: #llvm

    Differential Revision: https://reviews.llvm.org/D65849 (detail)
    by ibiryukov
  48. Remove inclusion of a private gmock header from a test (detail)
    by gribozavr
  49. Re-submit r367649: Improve raw_ostream so that you can "write" colors using operator<<

    The original patch broke buildbots, perhaps because it changed the
    default setting whether colors are enabled or not. (detail)
    by ruiu
  50. [ARM][LowOverheadLoops] Revert after read/write
       
    Currently we check whether LR is stored/loaded to/from inbetween the
    loop decrement and loop end pseudo instructions. There's two problems
    here:
    - It relies on all load/store instructions being labelled as such in
      tablegen.
    - Actually any use of loop decrement is troublesome because the value
      doesn't exist!
       
    So we need to check for any read/write of LR that occurs between the
    two instructions and revert if we find anything.

    Differential Revision: https://reviews.llvm.org/D65792 (detail)
    by sam_parker
  51. Force check prof branch_weights consistency in SwitchInstProfUpdateWrapper

    This patch turns on the prof branch_weights metadata consistency
    check in SwitchInstProfUpdateWrapper.

    If this patch causes a failure then please before reverting do report
    the IR that hits the assertion and try identifying the pass that
    introduces the inconsistency. We have to fix all such passes.

    See also the upcoming change https://reviews.llvm.org/D61179
    in the Verifier.

    Reviewers: davidx, nikic, eraman, reames, chandlerc
    Reviewed By: davidx
    Differential Revision: https://reviews.llvm.org/D64061 (detail)
    by yrouban
  52. [RISCV] Remove duplicated logic when determining the target ABI

    We were calculating twice ilp32/lp64. Do this in one place instead.

    Differential Revision: https://reviews.llvm.org/D48357 (detail)
    by rogfer01
  53. [X86] Allow any 8-bit immediate to be used with bt/btc/btr/bts memory aliases.

    We have aliases that disambiguate memory forms of bt/btc/btr/bts
    without suffixes to the 32-bit form. These aliases should have
    been updated when the instructions were updated in r356413. (detail)
    by ctopper
  54. [X86] Use isInt<8> to simplify some code. NFC (detail)
    by ctopper
  55. Detect HAVE_SYS_TYPES_H in lldb

    Summary:
    After rL368069 I noticed that HAVE_SYS_TYPES_H is not defined in
    Platform.h, or anywhere else in lldb. This change fixes that.

    Reviewers: labath

    Subscribers: mgorny, lldb-commits

    Tags: #lldb

    Differential Revision: https://reviews.llvm.org/D65822 (detail)
    by hhb
  56. [MachineCSE][NFC] Use 'profitable' rather than 'beneficial' to name method. (detail)
    by lkail
  57. [X86] Limit vpermil2pd/vpermil2ps immediates to 4 bits in the assembly parser.

    The upper 4 bits of the immediate byte are used to encode a
    register. We need to limit the explicit immediate to fit in the
    remaining 4 bits.

    Fixes PR42899. (detail)
    by ctopper
  58. Reverts commit r368117, r368115 and r368112

    This reverts commits:

      "Added Delta IR Reduction Tool"
      "[Bugpoint redesign] Added Pass to Remove Global Variables"
      "Added Tool as Dependency to tests & fixed warnings"

    Reduce/remove-funcs.ll is failing on bots. (detail)
    by phosek

Started by upstream project lldb-cmake build number 32879
originally caused by:

  • Started by timer

Started by upstream project lldb-cmake build number 32880
originally caused by:

  • Started by timer

Started by upstream project lldb-cmake build number 32881
originally caused by:

  • Started by timer

Started by upstream project lldb-cmake build number 32882
originally caused by:

  • Started by timer

This run spent:

  • 43 min waiting;
  • 1 hr 11 min build duration;
  • 1 hr 55 min total from scheduled to completion.
Test Result (no failures)