Use FlatSymbolRefAttr instead of StringAttr

Author: bcardosolopes

mlir/include/mlir/Dialect/LLVMIR/LLVMAttrDefs.td

@@ -1220,8 +1220,8 @@ def LLVM_PoisonAttr : LLVM_Attr<"Poison", "poison">;
 /// Folded into from LLVM::DSOLocalEquivalentOp.
 def LLVM_DSOLocalEquivalentAttr : LLVM_Attr<"DSOLocalEquivalent",
                                             "dso_local_equivalent"> {
-  let parameters = (ins "StringAttr":$name);
-  let assemblyFormat = "$name";
+  let parameters = (ins "FlatSymbolRefAttr":$sym);
+  let assemblyFormat = "$sym";
 }
 
 //===----------------------------------------------------------------------===//

mlir/lib/Dialect/LLVMIR/IR/LLVMDialect.cpp

@@ -2170,8 +2170,7 @@ DSOLocalEquivalentOp::verifySymbolUses(SymbolTableCollection &symbolTable) {
 /// Fold a dso_local_equivalent operation to a dedicated dso_local_equivalent
 /// attribute.
 OpFoldResult DSOLocalEquivalentOp::fold(FoldAdaptor) {
-  return DSOLocalEquivalentAttr::get(getContext(),
-                                     getFunctionNameAttr().getAttr());
+  return DSOLocalEquivalentAttr::get(getContext(), getFunctionNameAttr());
 }
 
 //===----------------------------------------------------------------------===//

mlir/lib/Target/LLVMIR/ModuleImport.cpp

@@ -1253,7 +1253,10 @@ FailureOr<Value> ModuleImport::convertConstant(llvm::Constant *constant) {
     Type type = convertType(dsoLocalEquivalent->getType());
     return builder
         .create<DSOLocalEquivalentOp>(
-            loc, type, dsoLocalEquivalent->getGlobalValue()->getName())
+            loc, type,
+            FlatSymbolRefAttr::get(
+                builder.getContext(),
+                dsoLocalEquivalent->getGlobalValue()->getName()))
         .getResult();
   }