================
@@ -460,6 +460,28 @@ mlir::LogicalResult 
CIRToLLVMAssumeOpLowering::matchAndRewrite(
   return mlir::success();
 }
 
+mlir::LogicalResult CIRToLLVMAssumeAlignedOpLowering::matchAndRewrite(
+    cir::AssumeAlignedOp op, OpAdaptor adaptor,
+    mlir::ConversionPatternRewriter &rewriter) const {
+  SmallVector<mlir::Value, 3> opBundleArgs{adaptor.getPointer()};
+
+  auto alignment = rewriter.create<mlir::LLVM::ConstantOp>(
+      op.getLoc(), rewriter.getI64Type(), op.getAlignment());
----------------
andykaylor wrote:

I was hoping that using the adaptor would save a round trip on the attribute 
creation (because `getAlignment()` returns a literal value which is then 
converted back to an attribute by the `ConstantOp` builder), but after closer 
inspection it doesn't look like it does. However, I think maybe using 
`getAlignmentAttr()` with either `op` or `adaptor` would work.

https://github.com/llvm/llvm-project/pull/152152
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to