[Mlir-commits] [mlir] [mlir] Remove convertible identity restriction for memref.atomic_rmw … (PR #72262)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Tue Nov 14 13:20:16 PST 2023
================
@@ -1562,8 +1562,6 @@ struct AtomicRMWOpLowering : public LoadStoreOpLowering<memref::AtomicRMWOp> {
LogicalResult
matchAndRewrite(memref::AtomicRMWOp atomicOp, OpAdaptor adaptor,
ConversionPatternRewriter &rewriter) const override {
- if (failed(match(atomicOp)))
----------------
MaheshRavishankar wrote:
It seems like this only works for strided memref types. Can you check that `getStridesAndOffset` on the source type returns `success`, i.e. return failure when that method fails. I looked into the implementation of `getStridedElementPtr` and it asserts if thats not the case, so letting that through would lead to an assert
https://github.com/llvm/llvm-project/pull/72262
More information about the Mlir-commits
mailing list