Skip to content

Commit 1b059e4

Browse files
authored
Bump llvm to f926bcf9068c808b643a56322b7ef6910eb36599 (#2012)
1 parent dc34a42 commit 1b059e4

File tree

13 files changed

+20
-19
lines changed

13 files changed

+20
-19
lines changed

CMakeLists.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -188,7 +188,7 @@ cmake_dependent_option(AIE_ENABLE_AIRBIN
188188
# If we need runtime libs, then statically link them.
189189
set(CMAKE_MSVC_RUNTIME_LIBRARY "MultiThreaded$<$<CONFIG:Debug>:Debug>")
190190

191-
add_flag_if_supported("-Werror=sign-compare" WERROR_SIGN_COMPARE)
191+
# add_flag_if_supported("-Werror=sign-compare" WERROR_SIGN_COMPARE)
192192
add_flag_if_supported("-Werror=unused" WERROR_USED)
193193

194194
# What happens when you have a non-void function with no return?

lib/Dialect/AIEVec/Transforms/ConvertVectorToAIEVec.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ struct RedundantLoadStoreOptimizationPass
8888
patterns.add<SetInboundsToReadOp, SetInboundsToWriteOp>(
8989
patterns.getContext());
9090

91-
(void)applyPatternsAndFoldGreedily(op, std::move(patterns));
91+
(void)applyPatternsGreedily(op, std::move(patterns));
9292
IRRewriter rewriter(&getContext());
9393
vector::transferOpflowOpt(rewriter, op);
9494
}

lib/Dialect/AIEVec/Transforms/DynamicSizeNoImplicitBroadcast.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ struct DynamicSizeNoImplicitBroadcastPass
113113

114114
patterns.add<DynamicSizeNoImplicitBroadcastPattern>(patterns.getContext());
115115

116-
(void)applyPatternsAndFoldGreedily(op, std::move(patterns));
116+
(void)applyPatternsGreedily(op, std::move(patterns));
117117
}
118118
};
119119

lib/Dialect/AIEVec/Transforms/VectorToVectorConversions.cpp

+3-3
Original file line numberDiff line numberDiff line change
@@ -758,7 +758,7 @@ struct VectorBroadcastLoweringPass
758758
patterns.add<ConvertLeadingUnitDimInsertToReshapePattern>(
759759
patterns.getContext());
760760

761-
(void)applyPatternsAndFoldGreedily(op, std::move(patterns));
761+
(void)applyPatternsGreedily(op, std::move(patterns));
762762
}
763763
};
764764

@@ -872,7 +872,7 @@ struct HoistCastOpToDataSourcePass
872872

873873
patterns.add<HoistCastOpToDataSourcePattern>(patterns.getContext());
874874

875-
(void)applyPatternsAndFoldGreedily(op, std::move(patterns));
875+
(void)applyPatternsGreedily(op, std::move(patterns));
876876
}
877877
};
878878

@@ -899,7 +899,7 @@ struct ReorderOperationsPass
899899
extInElemTy);
900900
});
901901

902-
(void)applyPatternsAndFoldGreedily(op, std::move(patterns));
902+
(void)applyPatternsGreedily(op, std::move(patterns));
903903
}
904904
};
905905

lib/Dialect/AIEX/Transforms/AIEMaterializeBDChains.cpp

+4-4
Original file line numberDiff line numberDiff line change
@@ -125,8 +125,8 @@ struct AIEMaterializeBDChainsPass
125125
RewritePatternSet patterns_0(ctx);
126126
patterns_0.insert<DMAStartBdChainForOpPattern>(ctx);
127127
DMAConfigureTaskOp::getCanonicalizationPatterns(patterns_0, ctx);
128-
if (failed(applyPatternsAndFoldGreedily(device, std::move(patterns_0),
129-
rewriter_config))) {
128+
if (failed(applyPatternsGreedily(device, std::move(patterns_0),
129+
rewriter_config))) {
130130
signalPassFailure();
131131
}
132132

@@ -135,8 +135,8 @@ struct AIEMaterializeBDChainsPass
135135
rewriter_config.enableRegionSimplification =
136136
GreedySimplifyRegionLevel::Disabled;
137137
DMAConfigureTaskOp::getCanonicalizationPatterns(patterns_1, ctx);
138-
if (failed(applyPatternsAndFoldGreedily(device, std::move(patterns_1),
139-
rewriter_config))) {
138+
if (failed(applyPatternsGreedily(device, std::move(patterns_1),
139+
rewriter_config))) {
140140
signalPassFailure();
141141
}
142142
}

lib/Dialect/AIEX/Transforms/AIESubstituteShimDMAAllocations.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ struct AIESubstituteShimDMAAllocationsPass
6767
RewritePatternSet patterns(&getContext());
6868
patterns.insert<DMAConfigureTaskForOpPattern>(&getContext());
6969

70-
(void)applyPatternsAndFoldGreedily(device, std::move(patterns));
70+
(void)applyPatternsGreedily(device, std::move(patterns));
7171
}
7272
};
7373

lib/Targets/AIERT.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -439,7 +439,7 @@ LogicalResult AIERTControl::initBuffers(DeviceOp &targetOp) {
439439
bytes.begin());
440440
byteVec.insert(byteVec.end(), bytes.begin(), bytes.end());
441441
}
442-
} else if (denseInit.getElementType().isa<FloatType>()) {
442+
} else if (isa<FloatType>(denseInit.getElementType())) {
443443
for (auto floatVal : denseInit.getValues<APFloat>()) {
444444
APInt floatInt = floatVal.bitcastToAPInt();
445445
// Get the size in bytes

python/dialects/aie.py

+2-1
Original file line numberDiff line numberDiff line change
@@ -257,7 +257,8 @@ def __init__(self, tile, link_with=None, dynamic_objfifo_lowering=None):
257257
# Create an aie buffer of (shape x datatype) on given tile.
258258
# shape examples: [256], [256, 256], [256, 256,]
259259
# This class hides the BufferOp and instead pretends to be a MemRef
260-
class buffer(BufferOp, ShapedValue):
260+
@ShapedValue
261+
class buffer(BufferOp):
261262
def __init__(self):
262263
raise ValueError("Should never be called")
263264

python/iron/kernel.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ def resolve(
4444
loc: ir.Location | None = None,
4545
ip: ir.InsertionPoint | None = None,
4646
) -> None:
47-
if self._op == None:
47+
if not self._op:
4848
self._op = external_func(self._name, inputs=self._arg_types)
4949

5050
def __call__(self, *args, **kwargs):

python/requirements_extras.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
# This can't go in the normal requirements file because the way the wheels build parses requirements.txt
22
# does not support github packages
3-
git+https://github.com/makslevental/mlir-python-extras@b2b8d72
3+
git+https://github.com/makslevental/mlir-python-extras@8aba4e7

test/npu-xrt/sync_task_complete_token_bd_chaining/aie2.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ def sequence(input, output):
123123
row=0,
124124
direction=1,
125125
channel=0,
126-
issue_token=1,
126+
issue_token=True,
127127
repeat_count=0,
128128
)
129129
# Wait for the task completion token of the previously set off chain of BDs

test/python/execution_engine.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ def run(f):
3535

3636
def lowerToLLVM(module):
3737
pm = PassManager.parse(
38-
"builtin.module(convert-complex-to-llvm,finalize-memref-to-llvm,convert-func-to-llvm,reconcile-unrealized-casts)"
38+
"builtin.module(convert-complex-to-llvm,finalize-memref-to-llvm,convert-func-to-llvm,convert-arith-to-llvm,convert-cf-to-llvm,reconcile-unrealized-casts)"
3939
)
4040
pm.run(module.operation)
4141
return module

utils/clone-llvm.sh

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
##===----------------------------------------------------------------------===##
1414

1515
# The LLVM commit to use.
16-
LLVM_PROJECT_COMMIT=f334db92be168876b618db72dc93078ce23ffa89
17-
DATETIME=2024121901
16+
LLVM_PROJECT_COMMIT=f926bcf9068c808b643a56322b7ef6910eb36599
17+
DATETIME=2025011002
1818
WHEEL_VERSION=20.0.0.$DATETIME+${LLVM_PROJECT_COMMIT:0:8}
1919

2020
############################################################################################

0 commit comments

Comments
 (0)