Skip to content

Commit a59faf5

Browse files
author
Razvan Lupusoru
committed
Fix formatting
1 parent 8ce5728 commit a59faf5

File tree

2 files changed

+17
-16
lines changed

2 files changed

+17
-16
lines changed

flang/lib/Optimizer/OpenACC/Support/RegisterOpenACCExtensions.cpp

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -47,18 +47,18 @@ void registerOpenACCExtensions(mlir::DialectRegistry &registry) {
4747
PartialEntityAccessModel<fir::ArrayCoorOp>>(*ctx);
4848
fir::CoordinateOp::attachInterface<
4949
PartialEntityAccessModel<fir::CoordinateOp>>(*ctx);
50-
fir::DeclareOp::attachInterface<
51-
PartialEntityAccessModel<fir::DeclareOp>>(*ctx);
50+
fir::DeclareOp::attachInterface<PartialEntityAccessModel<fir::DeclareOp>>(
51+
*ctx);
5252
});
5353

5454
// Register HLFIR operation interfaces
55-
registry.addExtension(+[](mlir::MLIRContext *ctx,
56-
hlfir::hlfirDialect *dialect) {
57-
hlfir::DesignateOp::attachInterface<
58-
PartialEntityAccessModel<hlfir::DesignateOp>>(*ctx);
59-
hlfir::DeclareOp::attachInterface<
60-
PartialEntityAccessModel<hlfir::DeclareOp>>(*ctx);
61-
});
55+
registry.addExtension(
56+
+[](mlir::MLIRContext *ctx, hlfir::hlfirDialect *dialect) {
57+
hlfir::DesignateOp::attachInterface<
58+
PartialEntityAccessModel<hlfir::DesignateOp>>(*ctx);
59+
hlfir::DeclareOp::attachInterface<
60+
PartialEntityAccessModel<hlfir::DeclareOp>>(*ctx);
61+
});
6262

6363
registerAttrsExtensions(registry);
6464
}

mlir/unittests/Dialect/OpenACC/OpenACCUtilsTest.cpp

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -614,8 +614,8 @@ TEST_F(OpenACCUtilsTest, getBaseEntityFromSubview) {
614614
SmallVector<OpFoldResult> sizes = {b.getIndexAttr(5), b.getIndexAttr(10)};
615615
SmallVector<OpFoldResult> strides = {b.getIndexAttr(1), b.getIndexAttr(1)};
616616

617-
OwningOpRef<memref::SubViewOp> subviewOp = memref::SubViewOp::create(
618-
b, loc, baseMemref, offsets, sizes, strides);
617+
OwningOpRef<memref::SubViewOp> subviewOp =
618+
memref::SubViewOp::create(b, loc, baseMemref, offsets, sizes, strides);
619619
Value subview = subviewOp->getResult();
620620

621621
// Test that getBaseEntity returns the base memref, not the subview
@@ -651,21 +651,22 @@ TEST_F(OpenACCUtilsTest, getBaseEntityChainedSubviews) {
651651
SmallVector<OpFoldResult> sizes1 = {b.getIndexAttr(50), b.getIndexAttr(80)};
652652
SmallVector<OpFoldResult> strides1 = {b.getIndexAttr(1), b.getIndexAttr(1)};
653653

654-
OwningOpRef<memref::SubViewOp> subview1Op = memref::SubViewOp::create(
655-
b, loc, baseMemref, offsets1, sizes1, strides1);
654+
OwningOpRef<memref::SubViewOp> subview1Op =
655+
memref::SubViewOp::create(b, loc, baseMemref, offsets1, sizes1, strides1);
656656
Value subview1 = subview1Op->getResult();
657657

658658
// Create second subview (subview of subview)
659659
SmallVector<OpFoldResult> offsets2 = {b.getIndexAttr(5), b.getIndexAttr(10)};
660660
SmallVector<OpFoldResult> sizes2 = {b.getIndexAttr(20), b.getIndexAttr(30)};
661661
SmallVector<OpFoldResult> strides2 = {b.getIndexAttr(1), b.getIndexAttr(1)};
662662

663-
OwningOpRef<memref::SubViewOp> subview2Op = memref::SubViewOp::create(
664-
b, loc, subview1, offsets2, sizes2, strides2);
663+
OwningOpRef<memref::SubViewOp> subview2Op =
664+
memref::SubViewOp::create(b, loc, subview1, offsets2, sizes2, strides2);
665665
Value subview2 = subview2Op->getResult();
666666

667667
// Test that getBaseEntity on the nested subview returns the first subview
668-
// (since our implementation returns the immediate source, not the ultimate base)
668+
// (since our implementation returns the immediate source, not the ultimate
669+
// base)
669670
Value baseEntity = getBaseEntity(subview2);
670671
EXPECT_EQ(baseEntity, subview1);
671672

0 commit comments

Comments
 (0)