Addressing further code review comments

Signed-off-by: Mohammadreza Ameri Mahabadian <[email protected]>

Author: mahabadm

mlir/lib/Dialect/SPIRV/IR/SPIRVOps.cpp

@@ -767,7 +767,7 @@ void mlir::spirv::AddressOfOp::getAsmResultNames(
 // spirv.EXTConstantCompositeReplicate
 //===----------------------------------------------------------------------===//
 
-// Returns type of attribute.  In case of a TypedAttr this will simply return
+// Returns type of attribute. In case of a TypedAttr this will simply return
 // the type. But for an ArrayAttr which is untyped and can be multidimensional
 // it creates the ArrayType recursively.
 static Type getValueType(Attribute attr) {

mlir/lib/Target/SPIRV/Serialization/Serializer.cpp

@@ -1187,6 +1187,9 @@ uint32_t Serializer::prepareConstantFp(Location loc, FloatAttr floatAttr,
   return resultID;
 }
 
+// Returns type of attribute. In case of a TypedAttr this will simply return
+// the type. But for an ArrayAttr which is untyped and can be multidimensional
+// it creates the ArrayType recursively.
 static Type getValueType(Attribute attr) {
   if (auto typedAttr = dyn_cast<TypedAttr>(attr)) {
     return typedAttr.getType();