- Dec 06, 2022
-
-
Artem Dergachev authored
This reverts commit 200007ec.
-
Leonard Chan authored
This reverts commit aacf17aa. Fixed by using the right register class for the movk.
-
LLVM GN Syncbot authored
-
Artem Dergachev authored
This is the initial commit for -Wunsafe-buffer-usage, a warning that helps codebases (especially modern C++ codebases) transition away from raw buffer pointers. The warning is implemented in libAnalysis as it's going to become a non-trivial analysis, mostly the fixit part where we try to figure out if we understand a variable's use pattern well enough to suggest a safe container/view as a replacement. Some parts of this analsysis may eventually prove useful for any similar fixit machine that tries to change types of variables. The warning is disabled by default. RFC/discussion in https://discourse.llvm.org/t/rfc-c-buffer-hardening/65734 Differential Revision: https://reviews.llvm.org/D137346
-
Jason Molenda authored
DynamicLoaderDarwinKernel::SearchForKernelNearPC() searches for a Darwin kernel mach-o header starting at $pc and working backwards, stopping on the first memory read error encountered. The kernel, and the kexts linked in to the kernel, have grown over the years and the original 32MB scan limit is giving a high chance of failing to find the kernel if we're in a random kext. In non-kernel environments, firmware and bare board typically, we will hit a memory read error on an unmapped page quickly so this doesn't add a lot of random memory read requests in those environments. We only check at one megabyte boundaries, so worst case this is 128 reads at the start of a gdb-remote connection. The check for a memory read error & stopping was a more recent addition (a few years ago), so I kept the scan region a bit small.
-
Lei Zhang authored
Reviewed By: ThomasRaoux Differential Revision: https://reviews.llvm.org/D139244
-
Leonard Chan authored
This reverts commit 7358c29a. This broke an upstream builder: https://lab.llvm.org/buildbot/#/builders/16/builds/39356
-
Felipe de Azevedo Piovezan authored
The tests in this patch expose failures of LLDBs expression evaluator when a standard library is compiled with debug symbols. This is the case for RelWithDebugInfo builds of llvm-project (with libcxx). Until these bugs are fixed, we force these tests to use the system's standard library. Differential Revision: https://reviews.llvm.org/D139361
-
Jakub Kuderski authored
This adds existing extensions as implied by SPIR-V 1.6. Also clean up the surrounding code. Fixes: https://github.com/llvm/llvm-project/issues/59348. Reviewed By: antiagainst Differential Revision: https://reviews.llvm.org/D139369
-
Lei Zhang authored
Along the way, make the default pattern fail instead of crashing when an elementwise op is not supported yet. Reviewed By: kuhar Differential Revision: https://reviews.llvm.org/D139280
-
Leonard Chan authored
This reverts commit bec8a372. This causes many of these errors to appear when rebuilding runtimes part of fuchsia's toolchain: ld.lld: error: /usr/local/google/home/paulkirth/llvm-upstream/build/lib/x86_64-unknown-linux-gnu/libunwind.a(libunwind.cpp.o) is incompatible with elf64-x86-64 This can be reproduced by making a complete toolchain, saving any source file with no changes, then rerunning ninja distribution.
-
Leonard Chan authored
This addresses https://github.com/llvm/llvm-project/issues/57750. For some globals, the tag wasn't propagated correctly because the necessary movk wasn't emitted sometimes. Differential Revision: https://reviews.llvm.org/D138615
-
Lei Zhang authored
spirv.MatrixTimesScalar is allowed to use cooperative matrix. Reviewed By: kuhar Differential Revision: https://reviews.llvm.org/D139279
-
Peiming Liu authored
[mlir][sparse] Refactoring: abstract sparse tensor memory scheme into a SparseTensorDescriptor class. This patch abstracts sparse tensor memory scheme into a SparseTensorDescriptor class. Previously, the field accesses are performed in a relatively error-prone way, this patch hides the hairy details behind a SparseTensorDescriptor class to allow users access sparse tensor fields in a more cohesive way. Reviewed By: aartbik Differential Revision: https://reviews.llvm.org/D138627
-
Hanhan Wang authored
We can compute the offsets and sizes for the slice of input because the iteration domain is defined over outer loops. If the dimension is tiled, the i-th index is the product of offset_i and inner_tile_i. Different from tiling a pad op, we do not have to deal with reading zero data from input. Because the tiling sizes are indicated to packed outer dimensions. We will read either the entire tile or partial tile for each packed tile. The scf.if and tensor.generate ops are not needed in this context. Co-authored-by:
Lorenzo Chelini <l.chelini@icloud.com> Reviewed By: rengolin, mravishankar Differential Revision: https://reviews.llvm.org/D138631
-
Jez Ng authored
This reverts commit 38d6202a. Differential Revision: https://reviews.llvm.org/D137368
-
Argyrios Kyrtzidis authored
[lldb] Make sure the value of `eSymbolContextVariable` is not conflicting with `RESOLVED_FRAME_CODE_ADDR` Differential Revision: https://reviews.llvm.org/D139066
-
Jeff Niu authored
This patch removes the implementation of TypedAttr and ElementsAttr from DenseArrayAttr and, in doing so, removes the need store a shaped type. The attribute now stores a size (number of elements), an MLIR type as a discriminator, and a raw byte array. The intent of DenseArrayAttr was not to be a drop-in replacement for DenseElementsAttr. It was meant to be a simple container of integers or floats that map to C++ types. The ElementsAttr implementation on DenseArrayAttr had many holes in it, and fixing those holes would require evolving DenseArrayAttr in a way that is incompatible with its original purpose. Reviewed By: rriddle Differential Revision: https://reviews.llvm.org/D137606
-
Simon Pilgrim authored
Fixes superfluous diffs identified in D139253
-
Simon Pilgrim authored
Fixes superfluous diffs identified in D139253
-
Simon Pilgrim authored
-
Simon Pilgrim authored
-
Simon Pilgrim authored
Fixes superfluous diffs identified in D139253
-
Simon Pilgrim authored
Reported by D138359 - the overrides matched the base class schedule WriteMicrocoded definition
-
Jez Ng authored
This was causing an uncaught exception issue in one of our programs. The issue was fairly subtle / rare as it required two identical LSDAs that were referenced by a pair of non-identical compact unwind encodings. Reviewed By: #lld-macho, smeenai Differential Revision: https://reviews.llvm.org/D139269
-
Rob Suderman authored
Rounding of tosa.resize did not handle rounding to the nearest pixel correctly. Rather than dividing the scale by 2 we should double the partial pixel to guarantee we include a check on the lowest bit. Reviewed By: NatashaKnk Differential Revision: https://reviews.llvm.org/D139162
-
Roy Jacobson authored
[Clang] Don't consider default constructors ineligible if the more constrained constructor is a template Partially solves https://github.com/llvm/llvm-project/issues/59206: We now mark trivial constructors as eligible even if there's a more constrained templated default constructor. Although technically non-conformant, this solves problems with pretty reasonable uses cases like ``` template<int n> struct Foo { constexpr Foo() = default; template<class... Ts> Foo(Ts... vals) requires(sizeof...(Ts) == n) {} }; ``` where we currently consider the default constructor to be ineligible and therefor inheriting/containing classes have non trivial constructors. This is aligned with GCC: https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=c75ebe76ae12ac4020f20a24f34606a594a40d15 This doesn't change `__is_trivial`. Although we're technically standard conformant in this regard, GCC/MSVC exhibit different behaviors that seem to make more sense. An issue has been filed to CWG and we await their response. Reviewed By: erichkeane, #clang-language-wg Differential Revision: https://reviews.llvm.org/D139038
-
Tarun Prabhu authored
The implementation follows the pattern used in comparable intrinsics. Change the runtime API for Norm2 so it does not expect a mask argument since the Norm2 intrinsic does not accept a mask in Fortran. Differential Revision: https://reviews.llvm.org/D138150
-
Krzysztof Parzyszek authored
-
Nico Weber authored
merge-fdata doesn't depend on any libraries except Support, but it includes headers that require Attributes.inc to exist. Add a dep that ensures that it does exist. Corresponds to the intrinsics_gen dep in CMake.
-
Philip Reames authored
This is a continuation of the series of patches adding lane wise support for scalable vectors in various knownbit-esq routines. The basic idea here is that we track a single lane for scalable vectors which corresponds to an unknown number of lanes at runtime. This is enough for us to perform lane wise reasoning on many arithmetic operations. Differential Revision: https://reviews.llvm.org/D137190
-
Krzysztof Parzyszek authored
If there is an entry P that has C bits set, it could become one of C different possibilities. If P occurs more than C times, then there are no valid completions.
-
Peter Steinfeld authored
Patch D138274 removed some Flang cmake configuration information that we need for our internal builds. This change restores them. Differential Revision: https://reviews.llvm.org/D139355
-
Nicolai Hähnle authored
Goal is to get a cleaner diff in an upcoming functional change.
-
Fangrui Song authored
after D131589 Reviewed By: jhenderson Differential Revision: https://reviews.llvm.org/D139282
-
Ramkumar Ramachandra authored
Make systematic use of notifyMatchFailure. Signed-off-by:
Ramkumar Ramachandra <r@artagnon.com> Reviewed By: rsuderman Differential Revision: https://reviews.llvm.org/D139190
-
River Riddle authored
This was donked up in the last patch that only considered aliases for things actually getting printed.
-
River Riddle authored
Calculating the position of the region trailing objects isn't free, given that it's the last trailing object, and inlining the size check removes the need for users to explicitly add size checks for micro-optimization.
-
Roman Lebedev authored
-
Chris Bieneman authored
This broke when some headers got moved aorund.
-