Started 10 mo ago
Took 1 hr 28 min on green-dragon-23

Success Build #2973 (Aug 8, 2019 10:44:07 PM)

Build Artifacts
test_logs.tgz49.03 KB view
  • : 368395
  • : 368393
  • : 368359
  • : 364589
  • : 368317
  1. [X86] Remove handler for (U/S)(ADD/SUB)SAT from ReplaceNodeResults. Remove TypeWidenVector check from code that handles X86ISD::VPMADDWD and X86ISD::AVG.

    More unneeded code since we now legalize narrow vectors by widening. (detail)
    by ctopper
  2. [X86] Remove ISD::SETCC handling from ReplaceNodeResults.

    This is no longer needed since we widen v2i32 instead of promoting. (detail)
    by ctopper
  3. [clang][NFC] Consolidating usage of "FinalPhase" in Driver::BuildActions.

    I am working to remove this concept of the "FinalPhase" in the clang driver,
    but it is used in a lot of different places to do argument handling for
    different combinations of phase pipelines and arguments. I am trying to
    consolidate most of the uses of "FinalPhase" into its own separate scope.
    Eventually, in a subsequent patch I will move all of this stuff to a separate
    function, and have more of the complication phase list construction setup into

    Differential Revision: (detail)
    by zer0
  4. [PowerPC] [Clang] Port SSE3, SSSE3 and SSE4 intrinsics to PowerPC

    Port existing headers which include x86 intrinsics implementation to
    PowerPC platform (using Altivec), along with tests. Also, tests about
    including these intrinsic headers are combined.

    The headers are mainly developed by Steven Munroe, with contributions
    from Paul Clarke, Bill Schmidt, Jinsong Ji and Zixuan Wu.

    Reviewed By: Jinsong Ji

    Differential Revision: (detail)
    by chaofan
  5. [X86] Simplify ISD::LOAD handling in ReplaceNodeResults and ISD::STORE handling in LowerStore now that v2i32 is widened to v4i32. (detail)
    by ctopper
  6. [X86] Merge v2f32 and v2i32 gather/scatter handling in ReplaceNodeResults/LowerMSCATTER now that v2i32 is also widened like v2f32. (detail)
    by ctopper
  7. [X86] Now unreachable handling for f64->v2i32/v4i16/v8i8 bitcasts from ReplaceNodeResults.

    We rely on the generic type legalizer for this now. (detail)
    by ctopper
  8. [X86] Simplify ReplaceNodeResults handling for FP_TO_SINT/UINT for vectors to only handle widening. (detail)
    by ctopper
  9. [X86] Simplify ReplaceNodeResults handling for SIGN_EXTEND/ZERO_EXTEND/TRUNCATE for vectors to only handle widening. (detail)
    by ctopper
  10. [X86] Simplify ReplaceNodeResults handling for UDIV/UREM/SDIV/SREM for vectors to only handle widening. (detail)
    by ctopper
  11. [X86] Remove vector promotion handling from the ReplaceNodeResults ISD::MUL handling code.

    We now widen illegal vector types so we don't need this anymore. (detail)
    by ctopper
  12. [analyzer] CastValueChecker: Model castAs(), getAs()

    Summary: Thanks to Kristóf Umann for the great idea!

    Reviewed By: NoQ

    Differential Revision: (detail)
    by charusso
  13. [analyzer] ConditionBRVisitor: Fix HTML PathDiagnosticPopUpPieces

    A condition could be a multi-line expression where we create the highlight
    in separated chunks. PathDiagnosticPopUpPiece is not made for that purpose,
    it cannot be added to multiple lines because we have only one ending part
    which contains all the notes. So that it cannot have multiple endings and
    therefore this patch narrows down the ranges of the highlight to the given
    interesting variable of the condition. It prevents HTML-breaking injections.

    Reviewed By: NoQ

    Differential Revision: (detail)
    by charusso
  14. [clang-scan-deps] Add minimizer support for C++20 modules.

    This only adds support to the minimizer, it doesn't actually capture the dependencies yet. (detail)
    by mspencer
  15. DebugInfo/DWARF: Provide some (pretty half-hearted) error handling access when parsing units

    This isn't the most robust error handling API, but does allow clients to
    opt-in to getting Errors they can handle. I suspect the long-term
    solution would be to move away from the lazy unit parsing and have an
    explicit step that parses the unit and then allows access to the other
    APIs that require a parsed unit.

    llvm-dwarfdump could be expanded to use this (or newer/better API) to
    demonstrate the benefit of it - but for now lld will use this in a
    follow-up cl which ensures lld can exit non-zero on errors like this (&
    provide more descriptive diagnostics including which object file the
    error came from).

    (error access to later errors when parsing nested DIEs would be good too
    - but, again, exposing that without it being a hassle for every consumer
    may be tricky) (detail)
    by dblaikie
  16. Change the return type of UpgradeARCRuntimeCalls to void

    Nothing is using the function return. (detail)
    by ahatanak
  17. Remove else-after-return (detail)
    by dblaikie
  18. Fix -DBUILD_SHARED_LIBS=ON build after rL368358

    Differential Revision: (detail)
    by sbc
  19. Fix llvm.aarch64.irg properties.

    IRG does not access any memory.
    Replace IntrInaccessibleMemOnly with IntrNoMem | IntrHasSideEffects.

    Reviewers: chill

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

    Tags: #llvm

    Differential Revision: (detail)
    by eugenis
  20. [InstCombine][NFC] Added comments about constants in tests for pow->exp2 fold (detail)
    by xbolva00
  21. [lldb][NFC] Modernize IRForTarget::CreateResultVariable (detail)
    by Raphael Isemann
  22. Added Delta IR Reduction Tool

    Summary: Tool parses input IR file, and runs the delta debugging algorithm to reduce the functions inside the input file.

    Reviewers: alexshap, chandlerc

    Subscribers: mgorny, llvm-commits

    Tags: #llvm

    Differential Revision:

    llvm-svn: 368071 (detail)
    by diegotf30
  23. Linker: Add support for GlobalIFunc.

    GlobalAlias and GlobalIFunc ought to be treated the same by the IR
    linker, so we can generalize the code to be in terms of their common
    base class GlobalIndirectSymbol.

    Differential Revision: (detail)
    by pcc
  24. Mark clang-scan-deps test as requiring thread support

    Otherwise the test calls a pure virtual method and crashes. Perhaps this
    could be improved. (detail)
    by rnk
  25. [lldb][NFC] Move to StringRef in some places in IRForTarget (detail)
    by Raphael Isemann

    This test doesn't make sense. Change to be consistent with what we did
    in GDBRemoteCommunication.cpp.

    Reviewers: labath

    Subscribers: lldb-commits

    Tags: #lldb

    Differential Revision: (detail)
    by hhb
  27. [LICM] Support unary FNeg in LICM

    Differential Revision: (detail)
    by mcinally
  28. [X86] Improve codegen of v8i64->v8i16 and v16i32->v16i8 truncate with avx512vl, avx512bw, min-legal-vector-width<=256 and prefer-vector-width=256

    Under this configuration we'll want to split the v8i64 or v16i32 into two vectors. The default legalization will try to truncate each of those 256-bit pieces one step to 128-bit, concatenate those, then truncate one more time from the new 256 to 128 bits.

    With this patch we now truncate the two splits to 64-bits then concatenate those. We have to do this two different ways depending on whether have widening legalization enabled. Without widening legalization we have to manually construct X86ISD::VTRUNC to prevent the ISD::TRUNCATE with a narrow result being promoted to 128 bits with a larger element type than what we want followed by something like a pshufb to grab the lower half of each element to finish the job. With widening legalization we just get the right thing. When we switch to widening by default we can just delete the other code path.

    Differential Revision: (detail)
    by ctopper
  29. Fix up fd limit diagnosis code

    Apparently Windows returns the "invalid argument" error code when the
    path contains invalid characters such as '<'. The
    test/Preprocessor/include-likely-typo.c test does this, so it was
    failing after r368322.

    Also, the diagnostic requires two arguments, so add the filename. (detail)
    by rnk
  30. [lldb][NFC] Directly use StringRef instead of temporary std::string (detail)
    by Raphael Isemann
  31. SymbolFileDWARF: Unconditionally scan through clang modules. NFCish

    When looking up a type by name, also scan through any referenced Clang
    modules regardsless of whether a type with this name has been
    found. This is NFCish (= a potential performance regression) for Clang
    projects, but necessary in mixed Swift and Objective-C projects (and
    tested in swift-lldb).

    This only affects projects compiled with -gmodules that were not run
    through dsymutil. (detail)
    by Adrian Prantl
  32. [SelectionDAG][X86] Move setcc mask splitting for mload/mstore/mgather/mscatter from DAGCombiner to the type legalizer.

    We may be able to look to how VSELECT is handled to further
    improve this, but this appears to be neutral or an improvement
    on the test cases we have. (detail)
    by ctopper
  33. [LegalizeTypes] Remove SplitVSETCC helper and just call SplitVecRes_SETCC. (detail)
    by ctopper
  34. [Attributor][NFC] Include only what is needed (detail)
    by jdoerfert
  35. Fix a comment which was incorrect. (detail)
    by jingham
  36. [MBP] Disable aggressive loop rotate in plain mode

    Patch introduced more aggressive loop layout optimization which depends on profile information. If profile information is not available, the statically estimated profile information(generated by BranchProbabilityInfo.cpp) is used. If user program doesn't behave as BranchProbabilityInfo.cpp expected, the layout may be worse.

    To be conservative this patch restores the original layout algorithm in plain mode. But user can still try the aggressive layout optimization with -force-precise-rotation-cost=true.

    Differential Revision: (detail)
    by carrot
  37. [analyzer] Fix scan-build's plist output in plist-html mode.

    r366941 accidentally made it delete all plist files
    as soon as they're produced. (detail)
    by dergachev
  38. [clang] add REQUIRES: linux to driver test case

    The test case explicitly leverages linux, so should include it as
    a test requirement. (detail)
    by bcain
  39. gn build: Merge r368331. (detail)
    by pcc
  40. [clang] add REQUIRES to driver test case

    The test case explicitly leverages x86, so should include it as
    a test requirement. (detail)
    by bcain
  41. Remove unused and undocumented data_offset parameter (NFC)

    Value::GetValueAsData() takes an undocumented parameter called
    data_offset that is always 0.

    Differential Revision: (detail)
    by Adrian Prantl
  42. lit: Bump version to 0.10.0

    Reviewers: hans

    Subscribers: hans, delcypher, llvm-commits

    Tags: #llvm

    Differential Revision: (detail)
    by tstellar
  43. [clang] Add no-warn support for Wa (detail)
    by bcain
  44. [llvm-mc] Add reportWarning() to MCContext

    Adding reportWarning() to MCContext, so that it can be used from
    the Hexagon assembler backend. (detail)
    by bcain
  45. [clang][NFC] Move matcher ignoringElidableConstructorCall's tests to appropriate file.

    `ignoringElidableConstructorCall` is a traversal matcher, but its tests are
    grouped with narrowing-matcher tests. This revision moves them to the correct

    Reviewers: gribozavr

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: (detail)
    by ymandel
  46. [X86] Make CMPXCHG16B feature imply CMPXCHG8B feature.

    This fixes znver1 so that it properly enables CMPXHG8B. We can
    probably remove explicit CMPXCHG8B from CPUs that also have
    CMPXCHG16B, but keeping this simple to allow cherry pick to 9.0.

    Fixes PR42935. (detail)
    by ctopper
  47. Recommit Devirtualize destructor of final class.

    Original patch commited as r364100, reverted as r364359, recommitted as r365509,
    reverted as r365850. (detail)
    by yamauchi
  48. clang: Diag running out of file handles while looking for files

    clang would only print "file not found" when it's unable to find a
    header file.  If the reason for that is a file handle leak, that's not a
    very useful error message.  For errors that aren't in a small whitelist
    ("file not found", "file is directory"), print an error with the
    strerror() output.

    This changes behavior in corner cases: If clang was out of file handles
    while looking in one -I dir but then suddenly wasn't when looking in the
    next -I dir, and both directories contained a file with the desired
    name, previously we'd silently return the file from the second
    directory. For this reason, it's important to ignore "is a directory"
    for this new diag: if a file foo/foo exists and -I -Ifoo are passed, an
    include of "foo" should successfully open file "foo" in directory "foo/"
    instead of complaining that "./foo" is a directory.

    No test since we mostly hit this when there's a handle leak somewhere,
    and currently there isn't one. I manually tested this with the repro
    steps in comment 2 on the bug below.

    Fixes PR42524.

    Differential Revision: (detail)
    by nico
  49. [globalisel][legalizer] Attempt to write down the minimal legalization rules

    There aren't very many requirements on the legalization rules but we should
    document them.

    Reviewers: aditya_nandakumar, volkan, bogner, paquette, aemerson, rovka, arsenm, Petar.Avramovic

    Subscribers: wdng, kristof.beyls, llvm-commits

    Tags: #llvm

    Differential Revision:

    # Conflicts:
    # llvm/docs/GlobalISel.rst (detail)
    by dsanders
  50. [AArch64] Do not emit '#' before immediates in inline asm

    The A64 assembly language does not require the '#' character to
    introduce constant immediate operands.  Avoid the '#' since the AArch64
    asm parser does not accept '#' before the lane specifier and rejects the
      __asm__ ("fmla v2.4s, v0.4s, v1.s[%0]" :: "I"(0x1))

    Fix a test to not expect the '#' and add a new test case with the above


    Reviewers: peter.smith, kristof.beyls

    Subscribers: javed.absar, hiraditya, llvm-commits, srhines

    Tags: #llvm

    Differential Revision: (detail)
    by pirama
  51. [clang] Update `ignoringElidableConstructorCall` matcher to ignore `ExprWithCleanups`.

    The `ExprWithCleanups` node is added to the AST along with the elidable
    CXXConstructExpr.  If it is the outermost node of the node being matched, ignore
    it as well.

    Reviewers: gribozavr

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: (detail)
    by ymandel
  52. Re-commit "[PowerPC][NFC][MachinePipeliner] Add some regression testcases""

    Remove sms-cpy1.ll first while I investigate the problem. (detail)
    by jsji
  53. [libc++] Fix environment variable passing in libc++'s `SSHExecutor` `lit` utility

    Quote the value of environment variables when passing them to the SSH
    client in SSHExecutor in libc++'s lit utilities. Without the quotes,
    an environment variable like FOO="buzz bar" gets passed incorrectly
    like this, ssh env FOO=buzz bar, which causes bar to be treated as a
    command to run, not part of the environment variable value.

    We ran into this when using SSHExecutor to do bringup of our CUDA
    libcu++ port on an embedded aarch64 system.

    Patch by Bryce Adelstein Lelbach.

    Differential Revision: (detail)
    by Louis Dionne
  54. lit: Use a License classifier that pypi will accept

    'OSI Approved :: Apache-2.0 with LLVM exception' is not a valid
    classifier.  'OSI Approved :: Apache Software License' is the closest
    fit for the new license, so we've decided to use this one.

    The classifiers seem to only be used for searching on the pypi website,
    so this does not actually change the license of the code.
    We still pass 'Apache-2.0 with LLVM exception' as the license to setup(),
    and this appears alongside the classifier on the pypi webpage for lit.

    Reviewers: chandlerc, ddunbar, joerg

    Reviewed By: joerg

    Subscribers: delcypher, llvm-commits

    Tags: #llvm

    Differential Revision: (detail)
    by tstellar
  55. [ObjC][ARC] Upgrade calls to ARC runtime functions to intrinsic calls if
    the bitcode has the arm64 retainAutoreleasedReturnValue marker

    The ARC middle-end passes stopped optimizing or transforming bitcode
    that has been compiled with old compilers after we started emitting
    calls to ARC runtime functions as intrinsic calls instead of normal
    function calls in the front-end and made changes to teach the ARC
    middle-end passes about those intrinsics (see r349534). This patch
    converts calls to ARC runtime functions that are not intrinsic functions
    to intrinsic function calls if the bitcode has the arm64
    retainAutoreleasedReturnValue marker. Checking for the presence of the
    marker is necessary to make sure we aren't changing ARC function calls
    that were originally MRR message sends (see r349952).


    Differential Revision: (detail)
    by ahatanak
  56. [lldb][NFC] Simplify return in MaybeHandleVariable

    This function anyway returns true, no need to do this extra work. (detail)
    by Raphael Isemann
  57. [X86] XFormVExtractWithShuffleIntoLoad - handle shuffle mask scaling

    If the target shuffle mask is from a wider type, attempt to scale the mask so that the extraction can attempt to peek through.

    Fixes the regression mentioned in rL368307 (detail)
    by rksimon
  58. [X86] SimplifyDemandedVectorElts - attempt to recombine target shuffle using DemandedElts mask

    If we don't demand all elements, then attempt to combine to a simpler shuffle.

    At the moment we can only do this if Depth == 0 as combineX86ShufflesRecursively uses Depth to track whether the shuffle has really changed or not - we'll need to change this before we can properly start merging combineX86ShufflesRecursively into SimplifyDemandedVectorElts.

    The insertps-combine.ll regression is because XFormVExtractWithShuffleIntoLoad can't see through shuffles of different widths - this will be fixed in a follow-up commit. (detail)
    by rksimon
  59. Enable assembly output of local commons for AIX

    This patch enable assembly output of local commons for AIX using .lcomm
    directives. Adds a EmitXCOFFLocalCommonSymbol to MCStreamer so we can emit the
    AIX version of .lcomm assembly directives which include a csect name. Handle the
    case of BSS locals in PPCAIXAsmPrinter by using EmitXCOFFLocalCommonSymbol. Adds
    a test for generating .lcomm on AIX Targets.

    Reviewers: cebowleratibm, hubert.reinterpretcast, Xiangling_L, jasonliu, sfertile

    Reviewed By: sfertile

    Subscribers: wuzish, nemanjai, hiraditya, kbarton, MaskRay, jsji, llvm-commits

    Tags: #llvm

    Differential Revision: (detail)
    by daltenty
  60. [ARM] Add support for MVE pre and post inc loads and stores

    This adds pre- and post- increment and decrements for MVE loads and stores. It
    uses the builtin pre and post load/store detection, unlike Neon. Loads are
    selected with the code in tryT2IndexedLoad, stores are selected with tablegen
    patterns. The immediates have a +/-7bit range, multiplied by the size of the

    Differential Revision: (detail)
    by dmgreen
  61. [ARM] MVE big endian loads/stores

    This adds some missing patterns for big endian loads/stores, allowing unaligned
    loads/stores to also be selected with an extra VREV, which produces better code
    than aligning through a stack. Also moves VLDR_P0 to not be LE only, and
    adjusts some of the tests to show all that working.

    Differential Revision: (detail)
    by dmgreen
  62. [RISCV] Allow ABI Names in Inline Assembly Constraints

    Clang will replace references to registers using ABI names in inline
    assembly constraints with references to architecture names, but other
    frontends do not. LLVM uses the regular assembly parser to parse inline asm,
    so inline assembly strings can contain references to registers using their
    ABI names.

    This patch adds support for parsing constraints using either the ABI name or
    the architectural register name. This means we do not need to implement the
    ABI name replacement code in every single frontend, especially those like
    Rust which are a very thin shim on top of LLVM IR's inline asm, and that
    constraints can more closely match the assembly strings they refer to.

    Reviewers: asb, simoncook

    Reviewed By: simoncook

    Subscribers: hiraditya, rbar, johnrusso, JDevlieghere, apazos, sabuasal, niosHD, kito-cheng, shiva0217, jrtc27, MaskRay, zzheng, edward-jones, rogfer01, MartinMosbeck, brucehoult, the_o, rkruppe, PkmX, jocewei, psnobl, benna, Jim, s.egerton, llvm-commits

    Tags: #llvm

    Differential Revision: (detail)
    by lenary
  63. [FIX][NFC] Update clang sema test

    Try to fix Sema test for default alignment for when
    compiling to ARM, but not to android, due
    r9427aa2d543b (detail)
    by dnsampaio
  64. [RISCV] Minimal stack realignment support

    Currently the RISC-V backend does not realign the stack. This can be an issue even for the RV32I/RV64I ABIs (where the stack is 16-byte aligned), though is rare. It will be much more comment with RV32E (though the alignment requirements for common data types remain under-documented...).

    This patch adds minimal support for stack realignment. It should cope with large realignments. It will error out if the stack needs realignment and variable sized objects are present.

    It feels like a lot of the code like getFrameIndexReference and determineFrameLayout could be refactored somehow, as right now it feels fiddly and brittle. We also seem to allocate a lot more memory than GCC does for equivalent C code.

    Reviewers: asb

    Reviewed By: asb

    Subscribers: wwei, jrtc27, s.egerton, MaskRay, Jim, lenary, hiraditya, rbar, johnrusso, simoncook, apazos, sabuasal, niosHD, kito-cheng, shiva0217, zzheng, edward-jones, rogfer01, MartinMosbeck, brucehoult, the_o, rkruppe, PkmX, jocewei, psnobl, benna, llvm-commits

    Tags: #llvm

    Differential Revision: (detail)
    by lenary
  65. Implement hh_mm_ss from P1466R3. Reviewed as (detail)
    by marshall
  66. [FileCheck] Add missing includes in header

    Reviewers: jhenderson, chandlerc, jdenny, probinson, grimar, arichardson, rnk

    Subscribers: hiraditya, llvm-commits

    Tags: #llvm

    Differential Revision: (detail)
    by thopre
  67. Add llvm.licm.disable metadata

    For some targets the LICM pass can result in sub-optimal code in some
    cases where it would be better not to run the pass, but it isn't
    always possible to suppress the transformations heuristically.

    Where the front-end has insight into such cases it is beneficial
    to attach loop metadata to disable the pass - this change adds the
    llvm.licm.disable metadata to enable that.

    Differential Revision: (detail)
    by timcorringham
  68. [OPENMP]Add support for analysis of linear variables and step.

    Added support for basic analysis of the linear variables and linear step
    expression. Linear loop iteration variables must be excluded from this
    analysis, only non-loop iteration variables must be analyzed.

    Reviewers: NoQ

    Subscribers: guansong, cfe-commits, caomhin, kkwli0

    Tags: #clang

    Differential Revision: (detail)
    by abataev
  69. [lldb][CMake] Disable modules in Xcode projects

    Summary: Apparently, module-enabled builds clash with Xcode's analysis.

    Reviewers: aprantl, jingham, davide, teemperor

    Reviewed By: davide

    Subscribers: mgorny, lldb-commits, #lldb

    Tags: #lldb

    Differential Revision: (detail)
    by stefan.graenitz
  70. [X86][SSE] matchBinaryPermuteShuffle - split INSERTPS combines

    We need to prefer INSERTPS with zeros over SHUFPS, but fallback to INSERTPS if that fails. (detail)
    by rksimon
  71. [Reassociate] add more tests with negative FP constants; NFC (detail)
    by spatel
  72. ObjectFileELF: Convert a unit test to a lit test

    It is much easier to test this functionality via lldb-test. (detail)
    by labath
  73. [ARM] Set default alignment to 64bits

    The maximum alignment used by ARM arch
    is 64bits, not 128.

    This could cause overaligned memory
    access for 128 bit neon vector that
    have unpredictable behaviour.

    This fixes:

    Reviewers: ostannard, dmgreen, srhines, danalbert, pirama, peter.smith

    Reviewed By: pirama, peter.smith

    Subscribers: phosek, thegameg, thakis, llvm-commits, carwil, peter.smith, javed.absar, kristof.beyls, cfe-commits

    Tags: #clang, #llvm

    Differential Revision: (detail)
    by dnsampaio
  74. [pstl] Add a __pstl_config_site header to record the CMake configuration

    This commit adds a __pstl_config_site header that contains the value of
    macros specified at CMake configuration time. It works similarly to
    libc++'s __config_site header, except we always include it as a separate
    file instead of concatenating it to the main configuration header.

    It is necessary to thread the includes for that header into libc++'s
    lit configuration, otherwise we'd be requiring an installation step
    prior to running the test suite. (detail)
    by Louis Dionne
  75. Fix check in tools/gold/X86/strip_names.ll regarding unnamed args

    After r367755 value numbers are printed for unnamed
    function arguments. The tools/gold/X86/strip_names.ll
    was not updated in that commit, so this patch can be
    seen as a follow up to r367755. (detail)
    by bjope
  76. [clang-format] fix crash involving invalid preprocessor line

    This (invalid) fragment is crashing clang-format:
    #if 1
    int x;
    int y;

    The reason being that the parser expects a token after `#elif`, and the
    subsequent parsing of the next line does not check if `CurrentToken` is null.

    Reviewers: gribozavr

    Reviewed By: gribozavr

    Subscribers: cfe-commits

    Tags: #clang

    Differential Revision: (detail)
    by krasimir
  77. Fix LLDB_CONFIGURATION_DEBUG builds for the GetSymbolVendor removal

    fix one usage that is ifdefed-out in non-debug builds. (detail)
    by labath
  78. [X86][SSE] Add x64 load use test case (detail)
    by rksimon
  79. [TargetLowering] SimplifyDemandedBits - call SimplifyMultipleUseDemandedBits for ISD::EXTRACT_VECTOR_ELT

    This patch attempts to peek through vectors based on the demanded bits/elt of a particular ISD::EXTRACT_VECTOR_ELT node, allowing us to avoid dependencies on ops that have no impact on the extract.

    In particular this helps remove some unnecessary scalar->vector->scalar patterns.

    The wasm shift patterns are annoying - @tlively has indicated that the wasm vector shift codegen are to be refactored in the near-term and isn't considered a major issue.

    Differential Revision: (detail)
    by rksimon
  80. [MCA] Remove dependency from InstrBuilder in mca::Context. NFC

    InstrBuilder is not required to construct the default pipeline. (detail)
    by adibiagio
  81. [MIPS GlobalISel] Select jump_table and brjt

    G_JUMP_TABLE and G_BRJT appear from translation of switch statement.
    Select these two instructions for MIPS32, both pic and non-pic.

    Differential Revision: (detail)
    by petar.avramovic
  82. [llcm-readobj] - Fix BB after t368272.

    Seems I forgot to update this test case. (detail)
    by grimar
  83. [yaml2obj/obj2yaml] - Add a basic support for extended section indexes.

    In some cases a symbol might have section index == SHN_XINDEX.
    This is an escape value indicating that the actual section header index
    is too large to fit in the containing field.
    Then the SHT_SYMTAB_SHNDX section is used. It contains the 32bit values
    that stores section indexes.

    ELF gABI says that there can be multiple SHT_SYMTAB_SHNDX sections,
    i.e. for example one for .symtab and one for .dynsym

    In this patch I am only supporting a single SHT_SYMTAB_SHNDX associated
    with a .symtab. This is a more or less common case which is used a few tests I saw in LLVM.

    I decided not to create the SHT_SYMTAB_SHNDX section as "implicit",
    but implement is like a kind of regular section for now.
    i.e. tools do not recreate this section or its content, like they do for
    symbol table sections, for example. That should allow to write all kind of
    possible broken test cases for our needs and keep the output closer to requested.

    Differential revision: (detail)
    by grimar
  84. Add a missing include to SymbolFilePDBTests.cpp

    This should _really_ fix the pdb unit tests. (detail)
    by labath
  85. [Extract] Fixed SemicolonExtractionPolicy for SwitchStmt and SwitchCase

    Reviewers: arphaman, sammccall

    Subscribers: dexonsmith, cfe-commits

    Tags: #clang

    Differential Revision: (detail)
    by sureyeaah
  86. Remove xcode-specific Config.h

    Now that the xcode project is removed, we no longer need/use the
    hand-maintained Config.h file, as everything is configured through

    This patch deletes that file and reverts some of the changes from
    r300372, which were made to support this use case.

    Reviewers: sgraenitz, beanz

    Subscribers: mgorny, lldb-commits

    Differential Revision: (detail)
    by labath
  87. Fix PDB unit tests for the GetSymbolVendor deletion (detail)
    by labath
  88. [ARM] Select VFMA (detail)
    by samtebbs
  89. Remove Module::GetSymbolVendor

    This patch removes the GetSymbolVendor function, and the various
    mentions of the SymbolVendor in the Module class. The implementation of
    GetSymbolVendor is "inlined" into the GetSymbolFile class which I
    created earlier.

    After this patch, the SymbolVendor class still exists inside the Module
    object, but only as an implementation detail -- a fancy holder for the
    SymbolFile. That will be removed in the next patch.

    Reviewers: clayborg, JDevlieghere, jingham, jdoerfert

    Subscribers: jfb, lldb-commits

    Differential Revision: (detail)
    by labath
  90. [ELF] - An attemp to fix builld bit after r368260


    The error is:
    error: 'static' function 'defaultWarningHandler' declared in header file should be declared 'static inline' [-Werror,-Wunneeded-internal-declaration]
    static Error defaultWarningHandler(const Twine &Msg) { return createError(Msg); }
                 ^ (detail)
    by grimar
  91. [llvm-readobj/libObject] - Introduce a custom warning handler for `ELFFile<ELFT>` methods.

    Currently, we have a code duplication in llvm-readobj which was introduced in D63266.
    The duplication was introduced to allow llvm-readobj to dump the partially
    broken object. Methods in ELFFile<ELFT> perform a strict validation of the inputs,
    what is itself good, but not for dumper tools, that might want to dump the information,
    even if some pieces are broken/unexpected.

    This patch introduces a warning handler which can be passed to ELFFile<ELFT> methods
    and can allow skipping the non-critical errors when needed/possible.

    For demonstration, I removed the duplication from llvm-readobj and implemented a warning using
    the new custom warning handler. It also deduplicates the strings printed, making the output less verbose.

    Differential revision: (detail)
    by grimar
  92. [diagtool] Use `operator<<(Colors)` to print out colored output.

    r368131 introduced this new API to print out messages in colors.
    If the colored output is disabled, `operator<<(Colors)` becomes nop.
    No functionality change intended.

    Differential Revision: (detail)
    by ruiu
  93. [X86] Remove -x86-experimental-vector-widening-legalization command line option and all its uses.

    This option is now defaulted to true and we don't want to support
    turning it off so remove the option. (detail)
    by ctopper
  94. [ARM] Tighten up VLDRH.32 with low alignments

    VLDRH needs to have an alignment of at least 2, including the
    widening/narrowing versions. This tightens up the ISel patterns for it and
    alters allowsMisalignedMemoryAccesses so that unaligned accesses are expanded
    through the stack. It also fixed some incorrect shift amounts, which seemed to
    be passing a multiple not a shift.

    Differential Revision: (detail)
    by dmgreen

Started by upstream project LLDB Incremental build number 94
originally caused by:

This run spent:

  • 7.1 sec waiting;
  • 1 hr 28 min build duration;
  • 1 hr 28 min total from scheduled to completion.
Test Result (no failures)