[Mlir-commits] [mlir] [mlir][affine] cleanup deprecated T.cast style functions (PR #71269)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Thu Nov 9 17:16:35 PST 2023


================
@@ -438,14 +438,15 @@ SparseTensorEncodingAttr::tranlateShape(ArrayRef<int64_t> srcShape,
     // Do constant propagation on the affine map.
     AffineExpr evalExp =
         simplifyAffineExpr(exp.replaceDims(dimRep), srcShape.size(), 0);
-    if (auto c = evalExp.dyn_cast<AffineConstantExpr>()) {
+    // use llvm namespace here to avoid ambiguity
+    if (auto c = llvm::dyn_cast<AffineConstantExpr>(evalExp)) {
----------------
lipracer wrote:

> can we avoid the llvm:: prefix if possible? it does not seem to be needed below, so I would like to keep the shorter, consistent version

Because the compiler reports that it cannot find this function, I will check where the namespace is missing.

https://github.com/llvm/llvm-project/pull/71269


More information about the Mlir-commits mailing list