- Oct 25, 2022
-
-
Manish Gupta authored
Differential Revision: https://reviews.llvm.org/D136313
-
Augusto Noronha authored
-
Aaron Ballman authored
This adds test coverage and updates the related entries for five more C99 DRs.
-
Aaron Ballman authored
I intended to fix this when landing the changes, but forgot to add the file to the commit.
-
Alex Brachet authored
This was reverted because it was breaking when targeting Darwin which tried to export these symbols which are now hidden. It should be safe to just stop attempting to export these symbols in the clang driver, though Apple folks will need to change their TAPI allow list described in the commit where these symbols were originally exported https://github.com/llvm/llvm-project/commit/f5380185623be243ba0f1b18d4bd594ac5cc7163 Bug: https://github.com/llvm/llvm-project/issues/58265 Differential Revision: https://reviews.llvm.org/D135340
-
Peiming Liu authored
[mlir][sparse] add some APIs for merger to query the tensor id for output tensor and synthetic tensor. Reviewed By: aartbik Differential Revision: https://reviews.llvm.org/D136630
-
Erich Keane authored
This reverts commit 52930162. Now with updating the ASTBitcodes to show that this AST is incompatible from the last.
-
Valentin Clement authored
Load the pointer of the dynamic type descriptor from the original box and update the destination descriptor with this pointer. Reviewed By: PeteSteinfeld Differential Revision: https://reviews.llvm.org/D136618
-
Zhiyao Ma authored
Instead of using constant pools, use movw movt pair. Differential Revision: https://reviews.llvm.org/D136203
-
rkayaith authored
This moves the commandline option registration into its own function, so that users can register translations without registering the options. Reviewed By: cota Differential Revision: https://reviews.llvm.org/D136561
-
David Green authored
D136227 showed that tests for this case in getReductionPatternCost were missing.
-
Michael Jones authored
This allows the client to set compile flags to disable the passes that the string to float function uses. A client may be willing to trade off performance for a reduction in code size, and this allows for that fine-tuning. Reviewed By: sivachandra Differential Revision: https://reviews.llvm.org/D136495
-
Michał Górny authored
Fix INSTALL_INTERFACE path for exported MLIRSparseTensorEnums target to include the include directory. Otherwise, CMake attempts to find it relatively to CMAKE_INSTALL_PREFIX rather than the include directory, resulting in flang failing to configure with the following error: CMake Error at /usr/lib/llvm/16/lib64/cmake/llvm/AddLLVM.cmake:581 (add_library): Cannot find source file: /usr/lib/llvm/16/mlir/Dialect/SparseTensor/IR/Enums.h Tried extensions .c .C .c++ .cc .cpp .cxx .cu .mpp .m .M .mm .ixx .cppm .h .hh .h++ .hm .hpp .hxx .in .txx .f .F .for .f77 .f90 .f95 .f03 .hip .ispc Call Stack (most recent call first): cmake/modules/AddFlang.cmake:64 (llvm_add_library) lib/Lower/CMakeLists.txt:3 (add_flang_library) Differential Revision: https://reviews.llvm.org/D136590 -
Carlos Alberto Enciso authored
The test case 'CompareElements' caused a failure in: https://lab.llvm.org/buildbot#builders/5/builds/28502 https://lab.llvm.org/buildbot#builders/168/builds/9671 SUMMARY: AddressSanitizer: 2144 byte(s) leaked in 14 allocation(s). The 'Insert' function conditionally adds the 'Child' logical element to a logical scope. The not added logical elements, caused the memory leak. Just record them to be destroyed when the logical reader is destroyed. Changed 'EXPECT_NE' with 'ASSERT_NE'. Missing in the main review.
-
Jonas Devlieghere authored
The diagnostic events were heavily inspired by the progress events and several comments incorrectly referenced "progress" rather than "diagnostic" events.
-
Mahesh Ravishankar authored
Drop the `createPadScalarOp` from Utils.h since it is a duplicate of the `build` method added here. Differential Revision: https://reviews.llvm.org/D136493
-
Fangrui Song authored
Mach-O ld64 supports -w to suppress warnings. GNU ld 2.40 will support the option as well (https://sourceware.org/bugzilla/show_bug.cgi?id=29654). This feature has some small value. E.g. when analyzing a large executable with relocation overflow issues, we may use --noinhibit-exec --emit-relocs to get an output file with static relocations despite relocation overflow issues. -w can significantly improve the link time as printing the massive warnings is slow. Reviewed By: peter.smith Differential Revision: https://reviews.llvm.org/D136569
-
Guozhi Wei authored
This is a pre-commit test case for D136396. Differential Revision: https://reviews.llvm.org/D136501
-
zhijian authored
Reviewers: James Henderson, Esme Yi Differential Revision: https://reviews.llvm.org/D134883
-
Roman Lebedev authored
[X86] `DAGTypeLegalizer::ModifyToType()`: when widening w/ zeros, insert into undef and `and`-mask the padding away We can expect that the sequence of inserting-of-extracts-into-undef will be successfully lowered back into widening of the source vector, but it seems that at least for X86 mask vectors, we have a really hard time recovering from inserting-into-zero. I've looked into alternative fix injection points, and they are much more involved, by the time of `LowerBUILD_VECTORvXi1()`/`LowerINSERT_VECTOR_ELT()` the constants might be obscured, so it does not seem like we can easily deal with this by lowering into bit math later on, some other pieces are missing. Instead, it seems like just clearing the padding away via an `AND`-mask is at least not a worse choice. Why create a problem where there wasn't one. Though yes, it is possible that there are cases where constants originate from the source IR, so some other fix may still be needed. Reviewed By: pengfei Differential Revision: https://reviews.llvm.org/D136046
-
Erich Keane authored
This reverts commit b7c92260. This seems to cause some problems with some modules related things, which makes me think I should have updated the version-major in ast-bit-codes? Going to revert to confirm this was a problem, then change that and re-try a commit.
-
Alexey Bataev authored
Improve O(N^2) to O(N) in some cases, reduce number of allocations by reserving memory. Also, improve analysis of loads reduction values to avoid analysis of not vectorizable cases.
-
Paul Robinson authored
Differential Revision: https://reviews.llvm.org/D136619
-
zhongyunde authored
The following sequence should be folded into in0 * in1 In0Lo = in0 & 0xffffffff; In0Hi = in0 >> 32; In1Lo = in1 & 0xffffffff; In1Hi = in1 >> 32; m01 = In1Hi * In0Lo; m10 = In1Lo * In0Hi; m00 = In1Lo * In0Lo; addc = m01 + m10; ResLo = m00 + (addc >> 32); Reviewed By: spatel, RKSimon Differential Revision: https://reviews.llvm.org/D136015 -
Craig Topper authored
This reverts commit 65aaecca. There was an ordering problem in the calculation of the partial remainder. Original commit message: If the divisor is even, we can first shift the dividend and divisor right by the number of trailing zeros. Now the divisor is odd and we can do the original algorithm to calculate a remainder. Then we shift that remainder left by the number of trailing zeros and add the bits that were shifted out of the dividend. Differential Revision: https://reviews.llvm.org/D135541
-
Simon Pilgrim authored
-
Simon Pilgrim authored
-
Peiming Liu authored
Address comments in D136372 Reviewed By: aartbik, springerm Differential Revision: https://reviews.llvm.org/D136510
-
Felipe de Azevedo Piovezan authored
The test TestObjCDirectMethods loads the Objective C runtime, which doesn't work well with custom a libcxx, resulting in two copies of the standard library being loaded at runtime. Like what was done for `TestObjCExceptions`, this commit forces the usage of the system's library instead. The minimum required Clang version is set to the oldest Clang that can compile the libraries available in the lldb-matrix bots. Differential Revision: https://reviews.llvm.org/D136600
-
Felipe de Azevedo Piovezan authored
This requirement dates back to ten years ago and the test seems to work nowadays with either libc++ or libstdc++. Differential Revision: https://reviews.llvm.org/D136608
-
Michał Górny authored
Differential Revision: https://reviews.llvm.org/D136607
-
Corentin Jabot authored
Implement P2513 This change allows initializing an array of unsigned char, or char from u8 string literals. This was done both to support legacy code and for compatibility with C where char8_t will be typedef to unsigned char. This is backported to C++20 as per WG21 guidance. Reviewed By: aaron.ballman Differential Revision: https://reviews.llvm.org/D136449
-
- Oct 24, 2022
-
-
Kevin P. Neal authored
This teaches the SCCP Solver how to constant fold more intrinsics. Constant folding appears to be just as good as D115737 but much, much lower in code change impact as suggested by nikic. The constrained floating-point intrinsics all take at least one metadata argument and were the motivation for the change. Differential Revision: https://reviews.llvm.org/D136466
-
Ahmed Bougacha authored
Differential Revision: https://reviews.llvm.org/D132385
-
Ahmed Bougacha authored
-
Kadir Cetinkaya authored
Introduces walkUsed, a very simple version of the public API to enable incremental development on rest of the pieces. Differential Revision: https://reviews.llvm.org/D136293
-
Ahmed Bougacha authored
(sign|resign) + (auth|resign) can be folded by omitting the middle sign+auth component if the key and discriminator match. Differential Revision: https://reviews.llvm.org/D132383
-
Amy Kwan authored
When lowering vector shuffles into the xxsplti32dx instruction on Power10, we canonicalize the right operand to be a BUILD_VECTOR and as a result, get the commuted vector shuffle node. However, a vector shuffle will not always be returned as the result for a commuted vector shuffle. In such a scenario, this patch updates the original cast of a shuffle into a dyn_cast<> and checks if the shuffle is a valid vector shuffle node prior to obtaining the commuted shuffle mask. This patch also adds a new test case that demonstrates this scenario (primarily seen on 32-bit), and was originally a crash prior to this fix. Differential Revision: https://reviews.llvm.org/D135024
-
Craig Topper authored
This reverts commit f6a7b478. I received a report that this fails on 32-bit X86.
-
Dmitry Preobrazhensky authored
Differential Revision: https://reviews.llvm.org/D136370
-