clean up how we use attributes

Author: Jaddyen

mlir/lib/Dialect/EmitC/Transforms/WrapFuncInClass.cpp

@@ -62,11 +62,8 @@ class WrapFuncInClass : public OpRewritePattern<emitc::FuncOp> {
       Attribute argAttr = nullptr;
 
       fieldName = rewriter.getStringAttr("fieldName" + std::to_string(idx));
-      if (argAttrs && idx < argAttrs->size()) {
-        if (DictionaryAttr dictAttr =
-                dyn_cast<mlir::DictionaryAttr>((*argAttrs)[idx]))
-          argAttr = (*argAttrs)[idx];
-      }
+      if (argAttrs && idx < argAttrs->size())
+        argAttr = (*argAttrs)[idx];
 
       TypeAttr typeAttr = TypeAttr::get(val.getType());
       fields.push_back({fieldName, typeAttr});