- Apr 19, 2024
-
-
Michael Flanders authored
Fixes #89135.
-
Anthony Ha authored
This PR adds a check within `PutFile` to exit early when both local and destination files have matching MD5 hashes. If they differ, or there is trouble getting the hashes, the regular code path to put the file is run. As I needed this to talk to an `lldb-server` which runs the gdb-remote protocol, I enabled `CalculateMD5` within `Platform/gdb-server` and also found and fixed a parsing bug within it as well. Before this PR, the client is incorrectly parsing the response packet containing the checksum; after this PR, hopefully this is fixed. There is a test for the parsing behavior included in this PR. --------- Co-authored-by:Anthony Ha <antha@microsoft.com>
-
Cyndy Ishida authored
-
alx32 authored
Currently, when moving symbols from one `InputSection` to another (like in ICF) we directly update the symbol's `isec`, `unwindEntry` and `size`. By doing this we lose the original information. This information will be needed in a future change. Since when moving symbols we always set the symbol's `wasCoalesced` and `isec-> replacement`, we can just use this info to conditionally get the information we need at access time.
-
Craig Topper authored
-
Durgadoss R authored
-
Arthur Eubanks authored
This means that the pass is unusable without some sort of profile. We can revisit this decision later if we want to support running this pass without a profile.
-
Jorge Gorbe Moya authored
-
Jordan Rupprecht authored
The `LLDB_TEST_USE_VENDOR_PACKAGES` has defaulted to `Off` for a while. Either installing `pexpect` or skipping those tests with `-DLLDB_TEST_USER_ARGS=--skip-category=pexpect` seems to be enough that we can fully remove this option. This patch removes the `LLDB_TEST_USE_VENDOR_PACKAGES` cmake configuration as well as the associated code to add `third_party/Python/module` to the python path. I'll do the actual deletion of `third_party/Python/module` in a followup PR in the (unlikely, I hope) event this commit needs to be reverted.
-
Charitha Saumya authored
[mlir][vector] Add support for linearizing Extract, ExtractStridedSlice, Shuffle VectorOps in VectorLinearize (#88204) This PR adds support for converting `vector.extract_strided_slice` and `vector.extract` operations to equivalent `vector.shuffle` operations that operates on linearized (1-D) vectors. `vector.shuffle` operations operating on n-D (n > 1) are also converted to equivalent shuffle operations working on linearized vectors.
-
Joe Nash authored
Tablegen classes MnemonicAlias, Requires, and VOPC_Real, all define a field 'Predicates'. The prior formulation resulted in the instantiated record inheriting from all three to only have the Predicate set in Requires, i.e. Gen.AssemblerPredicate. This breaks the design of GCNPredicateControl (which is a parent class of VOPC_Real) that allows multiple predicates such as SubtargetPredicate and OtherPredicates to be set on an Instruction. MnemonicAlias does not need to be defined in the same record as VOPC_Real, so we can separate the definitions and remove Requires to avoid the issue. NFCI, but it enables future changes, such as setting multiple predicates on a VOPC_Real.
-
Emma Pilkington authored
These mostly are checking for various reserved bits being set. The diagnostics for gpu-dependent reserved bits have a bit more context since they seem like the most likely ones to be observed in practice. This commit also improves the error handling mechanism for MCDisassembler::onSymbolStart(). Previously it had a comment stream parameter that was just being ignored by llvm-objdump, now it returns errors using Expected<T>.
-
Mark de Wever authored
This uses a period instead of a comma; the latter was a typo.
-
Vyacheslav Levytskyy authored
This PR fixes parsing of builtins return types in general and fixes return type when sampling an image with OpImageSampleExplicitLod in particular.
-
Fred Grim authored
the assembly jazz doesn't work on arm. oops
-
Alexey Bataev authored
Need to check that at least single bit is cleared for unsigned nodes before reducing their size. Otherwise they might be treated as signed in signed nodes.
-
Valery Pykhtin authored
Usage: `update_any_test_check.py @my_list_of_tests` where my_list_of_tests is a file containing list of tests to update.
-
Krzysztof Parzyszek authored
This removes the last use of genOmpObjectList2, which has now been removed. --------- Co-authored-by:Sergio Afonso <safonsof@amd.com>
-
Alexey Bataev authored
-
Dmitriy Smirnov authored
This PR increases accurasy of FFT2D/RFFT2D calculation by removing periodic part of sin/cos
-
Simon Pilgrim authored
512-bit GFNI doesn't require AVX512BW
-
Fred Grim authored
This adds some additional bits into a ProcessInfo structure that will be of use in filling structs in an elf core file. This is a demand for implementing process save-core
-
Kazu Hirata authored
-
Bharathi Ramana Joshi authored
The pull request includes the following changes. 1. Refactors the interface to `PresburgerSpace::identifiers` to `setId` and a const `getId`, instead of previous `getId` which returned a mutable reference. `resetIds` does not need to be called to use identifiers, `setId` calls `resetIds` if identifiers are not enabled. 2. Deprecates `FlatAffineRelation` by refactoring all usages of `FlatAffineRelation` to `IntegerRelation`. To achieve this, `FlatAffineRelation::compose` is refactored into `IntegerRelation::mergeAndCompose`. 3. Deletes unneeded overrides of virtual functions `hasConsistentState`, `clearAndCopyFrom` and `fourierMotzkinEliminate` from `FlatLinearValueConstraints` as these were only used through `FlatAffineRelation` and we now use `IntegerRelation`'s member functions instead. 4. Fixes an existing bug in FlatLinearValueConstraints' constructor which caused identifiers set by superclass FlatLinearConstraints' constructor to be erased. 5. Fixes `IntegerRelation::convertVarKind` not preserving identifiers.
-
jeffreytan81 authored
There are users reporting saving minidump from lldb-dap does not work. Turns out our stack trace request always evaluate a function call which caused JIT object file like "__lldb_caller_function" to be created which can fail minidump builder to get its module size. This patch fixes "getModuleFileSize" for ObjectFileJIT so that module list can be saved. I decided to create a lldb-dap test so that this end-to-end functionality can be validated from our tests (instead of only command line lldb). The patch also improves several other small things in the workflow: 1. It logs any minidump stream failure so that it is easier to find out what stream saving fails. In future, we should show process and error to end users. 2. It handles error from "getModuleFileSize" llvm::Expected<T> otherwise it will complain the error is not handled. --------- Co-authored-by:jeffreytan81 <jeffreytan@fb.com>
-
Slava Zakharin authored
This reverts commit 92029845.
-
Craig Topper authored
Fixes #88958.
-
- Apr 18, 2024
-
-
Simon Pilgrim authored
Once #89115 has landed, we can handle per-element rotates as well using (V)GF2P8MULB
-
Simon Pilgrim authored
Once #89115 has landed, we can handle per-element shifts as well using (V)GF2P8MULB
-
LLVM GN Syncbot authored
-
Craig Topper authored
[DAGCombiner][AArch64] Make combineCarryDiamond avoid creating UADDO_CARRY with carry in larger than setcc result type. (#89121) In the attach test case we were creating a UADDO_CARRY with i1 carry out and i41 carry in. i41 exceeds is larger than the setcc result type for AArch64 which is i32. i41 needs to be promoted to i64 since it is larger than i32. The type legalizer tried to use promoteTargetBoolean, but that can only promote from a type smaller than setcc result type. The easiest fix here is to force the carryin type to match the carryout type at the type of creation. This should ensure the node won't exceeed setcc result type as long as the output type doesn't. I think we should explore requiring the types to match for this node. Fixes #88966
-
Jay Foad authored
Fixes #84212
-
Mark de Wever authored
This adds the local_info type and its formatting options. The usage of the local_info object will be done in separate patches. Implements parts of: - P0355 Extending to Calendars and Time Zones - P1361 Integration of chrono with text formatting
-
Scott Egerton authored
[AMDGPU] Support wide register or subregister access when emitting s_singleuse_vdst instructions. (#88520) Both single use producer and consumer instructions using wide/sub registers are now correctly tracked and eligible for being marked as single use.
-
Zequan Wu authored
If lldb finds the dynamic linker in the search path or if the binary is linked staticlly, it will fail at `lldbutil.run_break_set_by_symbol` because the breakpoint is resolved. Otherwise, it's not resolved at this point. But we don't care if it's resolved or not. This test cares about if the breakpoint is hit or not after launching. This changes the num_expected_locations to -2, which means don't assert on if this breakpoint resolved or not.
-
Matt Arsenault authored
-
Matt Arsenault authored
This was copy-paste from the identical add test, and didn't change to test the or from the name.
-
kadir çetinkaya authored
Some FS implementations rely on snapshots available in the context.
-
Craig Topper authored
The usage of IsEABI was only valid for RV32E. For RV64E, the stack adjust base needs to be 32 when ra,s0-s1 are being saved. Since it takes more than 16 bytes to save 3 64-bit registers. The spec lists the rv32e behavior explicitly, but not rv64e. My assumption is that the only thing that changes with rv64e is which registers can be used in the register list, but not how the register list affects the stack_adj_base. -
kadir çetinkaya authored
Building ASTs with compile flags that are incompatible to the ones used for the Preamble are not really supported by clang and can trigger crashes. In an ideal world, we should be re-using not only TargetOpts, but the full ParseInputs from the Preamble to prevent such failures. Unfortunately current contracts of ThreadSafeFS makes this a non-safe change for certain implementations. As there are no guarantees that the same ThreadSafeFS is going to be valid in the Context::current() we're building the AST in.
-