- Sep 27, 2022
-
-
Lorenzo Chelini authored
It introduces a circular build dependence: DialectUtils <- ArithmeticUtils <- ArithDialect <- DialectUtils This reverts commit 27224fe7.
-
Sanjay Patel authored
This is a test to verify that we do not crash with the problem noted in issue #57986. The root problem should be fixed with a prior change to InstSimplify.
-
Sanjay Patel authored
The test shows that we would fail to consistently fold the instruction based on the max value operand. This is also the root cause for issue #57986, but I'll add an instcombine test + assert for that exact problem in another commit.
-
Sanjay Patel authored
This shows the root problem that leads to the crash in issue #57986.
-
Jan Svoboda authored
The "-fno-integrated-as" on AIX expands the driver invocation into multiple jobs. This patch makes new test accommodates for that.
-
Krzysztof Parzyszek authored
Simplify (sext_inreg (extractu ...)) -> (extract ...) where appropriate.
-
Mahesh Ravishankar authored
The current approach for handling `iter_args` was to replace all uses of the value that is used as `init` value with the corresponding region block argument within the `scf.for`. This is not always correct. Instead a more deliberate approach needs to be taken to handle these. If the slice being fused represents a slice of the destination operand of the untiled op, then - Make the destination of the fused producer the `init` value of the loop nest - For the tiled and fused producer op created, replace the slice of the destination operand with a slice of the corresponding region iter arg of the innermost loop of the generated loop nest Differential Revision: https://reviews.llvm.org/D134411
-
Sanjay Patel authored
This makes the code a little shorter and should be easier to extend for a pattern like in issue #42100.
-
Sanjay Patel authored
It's still possible that there's a simpler way to specify the conditions needed for this set of folds, but "getStrictPred" converts >= to > for example, so there's no need to explicitly check that.
-
Michael Buch authored
Since we don't compile with `gmodules` implicitly via debug-info test replication, we should mark all implicit `gmodules` tests with the appropriate category so the API tests get actually run as intended. Differential Revision: https://reviews.llvm.org/D134574
-
Michael Buch authored
Currently, by default LLDB runs an API test with 3 variants, one of which, depending on platform, is `gmodules`. However, most tests don't actually make use of any specific `gmodules` feature since they compile a single `main.cpp` file without imporint types from other modules. Instead of running all tests an extra time with `-gmodules` enabled, we plan to test `gmodules` features with dedicated API tests that explicitly opt-into compiling with `-gmodules`. One of the main benefits of this will be a reduction in total API test-suite run-time (by around 1/3). This patch adds a flag to `debug_info_categories` that indicates whether a category is eligible to be replicated by `lldbtest`. Keeping `gmodules` a debug-info category is desirable because `builder.py` knows how to inject the appropriate Makefile flags into the build command for debug-info categories already. Whereas for non-debug-info categories we'd have to teach it how to. The category is inferred from the test-name debug-info suffix currently. Differential Revision: https://reviews.llvm.org/D134524
-
Jakub Kuderski authored
-
Matt Arsenault authored
-
Matt Arsenault authored
-
Matt Arsenault authored
Using these queries with a context instruction and without a cache seems to be about 2x slower than with it so this theoretically improves compile time.
-
Jakub Kuderski authored
This reverts commit ce82530c. Undo accidental Vulkan/SPIR-V name changes. Tested with `ninja check-mlir check-mlir-mlir-spirv-cpu-runner check-mlir-mlir-vulkan-runner`.
-
Thomas Raoux authored
Make sure we consider other subviews of the same buffer when doing store to load forwarding or dead store elimination. Differential Revision: https://reviews.llvm.org/D134576
-
Dave Lee authored
Some time ago, a refactor (1153dc96) broke completion for assigning settings values (`settings set`). This was most annoying for enum settings, where you'd have to get the valid enum names separately. This restores the logic in the post-refactor completion function, as well as adding a test to catch future regressions. Differential Revision: https://reviews.llvm.org/D134515
-
Krzysztof Parzyszek authored
-
Thomas Raoux authored
Improve hoisting logic to support cases where the read being hoisted comes from a transfer_write with disjoint indices. Differential Revision: https://reviews.llvm.org/D134624
-
Lang Hames authored
ELF undefined symbols can have non-default visibility.
-
Craig Topper authored
Promote f16 to f32 and use the f32 libcall. I deleted rv64zfh-half-intrinsics-strict.ll because it only existed due to this issue breaking rv32. Differential Revision: https://reviews.llvm.org/D134579
-
Jan Svoboda authored
This makes `llvm::PointerIntPair<Module *, 3>` from f35230ae work across platforms. Reviewed By: srj Differential Revision: https://reviews.llvm.org/D134653
-
Felipe de Azevedo Piovezan authored
The coroutine tests require a standard library implementation of coroutines, which was only made available some time _after_ Clang 13. The first such Clang tested by the LLDB matrix bot is 15.0.1 The TestObjCExceptions test forces the use of the system's libcxx. For the lldb matrix bot, the first Clang version compatible with the bot's libraries is 13.0. Differential Revision: https://reviews.llvm.org/D134645
-
Jeff Niu authored
-
Mathieu Fehr authored
SelfOwningTypeID was inherited privately, so it was not possible to access the TypeID of an attribute or a type definition. Reviewed By: Mogball Differential Revision: https://reviews.llvm.org/D134593
-
Richard Howell authored
When initializing FunctionNoProtoType types, zero out the type qualifiers. This will ensure the ODR hash remains stable as it hashes the values for these qualifiers for all function types. Reviewed By: aaron.ballman Differential Revision: https://reviews.llvm.org/D133586
-
Jakub Kuderski authored
This reverts commit a2052b87. This commit renamed some Vulkan identifiers that shouldn't have been renamed, e.g., `SPV_KHR_storage_buffer_storage_class`.
-
Changpeng Fang authored
Summary: With opaque pointer support, the "ptr" type is introduced and thus BitCast is not necessary in some cases. This work takes care of this change, and recognizes the new address patterns to do appropriate optimizations. Reviewers: arsenm Differential Revision: https://reviews.llvm.org/D134596
-
Tarun Prabhu authored
This calls the corresponding runtime functions when appropriate. The implementation follows the pattern of the SUM and PRODUCT intrinsics. Differential Revision: https://reviews.llvm.org/D129616
-
Quentin Colombet authored
Teach the pass that simplifies extract_strided_metadata(other_op(memref)) how to get rid of extract_strided_metadata when they are fed by allocLikeOp. For the simplification to happen the allocLikeOp needs to have been normalized. I.e., no weird offset and strides. When this is the case, we replace: ``` base, offset, sizes, strides = extract_strided_metadata(allocLikeOp(allocSizes)) ``` With ``` base = reinterpret_cast allocLikeOp(allocSizes) to a flat memref<eltTy> offset = 0 sizes = allocSizes strides#i = prod(allocSizes#j, for j in {i+1..rank-1}) ``` The computation involving dynamic sizes are expanded in affine.apply. Differential Revision: https://reviews.llvm.org/D134577 -
Lorenzo Chelini authored
The utility function should live in `StaticValueUtils.h` as it provides a convenient way to convert a vector of OpFoldResults into a vector of Values. Reviewed By: nicolasvasilache Differential Revision: https://reviews.llvm.org/D134451
-
Pengxuan Zheng authored
Currently LLD does not generate a map file if no /out flag (e.g., /out:a.exe) is present. This is because LLD derives the map file's name from the default output file name is no output file name is specified explicitly on the command line. However, in this case, the default output file name has not been set by LLD yet when LLD tries to set the name of the map file. This patch fixes this corner case by moving the logic handling map file flags to a place after the default output file name is set. Reviewed By: thakis Differential Revision: https://reviews.llvm.org/D134559
-
- Sep 26, 2022
-
-
Nathan Sidwell authored
I noticed a duplicate macro definition in xmmintrin.h. Reviewed By: RKSimon Differential Revision: https://reviews.llvm.org/D134545
-
Nicolas Vasilache authored
As experience with memref::ExtractStridedMetadataOp grows we are still missing a simple way to extract the pointer held by a memref and lower to different backednds (LLVM, SPIRV, library calls). This revision introduces a memref.extract_aligned_pointer_as_index that returns an index containing the aligned pointer of the strided memref. This operation is intended to be used solely as step during lowering, it has no side effects. A reverse operation that creates a memref from an index interpreted as a pointer is explicitly discouraged. Differential Revision: https://reviews.llvm.org/D134651
-
Zain Jaffal authored
Optimizations can be used to eliminate unecessary overflow checks. This patch introduces some test cases where the checks can be safely removed Reviewed By: fhahn, fcloutier Differential Revision: https://reviews.llvm.org/D134038
-
Jakub Kuderski authored
Tested with `check-mlir` and `check-mlir-integration`. Fixes: https://github.com/llvm/llvm-project/issues/56863 Reviewed By: antiagainst Differential Revision: https://reviews.llvm.org/D134649
-
Erich Keane authored
This reverts commit 192d69f7. This fixes the condition to check whether this is a situation where we are in a recovery-expr'ed concept a little better, so we don't access an inactive member of a union, which should make the bots happy. Differential Revision: https://reviews.llvm.org/D134542
-
Nicolas Vasilache authored
`memref.extract_strided_metadata` can forward constants independently of the exsistence of other operations such as subview or reshape. Differential Revision: https://reviews.llvm.org/D134603
-
Kazu Hirata authored
This patch fixes: mlir/lib/Dialect/Linalg/Transforms/TilingInterfaceImpl.cpp:99:16: error: 'getValue' is deprecated: Use value instead. [-Werror,-Wdeprecated-declarations]
-