Fix failures in other tests

It had to be checked for a lack of `"targets"` attributes in a GPU
module

Author: oojahooo

mlir/lib/Conversion/GPUToSPIRV/GPUToSPIRV.cpp

@@ -385,9 +385,11 @@ LogicalResult GPUModuleConversion::matchAndRewrite(
   if (auto attr = moduleOp->getAttrOfType<spirv::TargetEnvAttr>(
           spirv::getTargetEnvAttrName()))
     spvModule->setAttr(spirv::getTargetEnvAttrName(), attr);
-  for (const Attribute &targetAttr : moduleOp.getTargetsAttr())
-    if (auto spirvTargetEnvAttr = dyn_cast<spirv::TargetEnvAttr>(targetAttr))
-      spvModule->setAttr(spirv::getTargetEnvAttrName(), spirvTargetEnvAttr);
+  if (const ArrayAttr &targets = moduleOp.getTargetsAttr()) {
+    for (const Attribute &targetAttr : targets)
+      if (auto spirvTargetEnvAttr = dyn_cast<spirv::TargetEnvAttr>(targetAttr))
+        spvModule->setAttr(spirv::getTargetEnvAttrName(), spirvTargetEnvAttr);
+  }
 
   rewriter.eraseOp(moduleOp);
   return success();

mlir/lib/Conversion/GPUToSPIRV/GPUToSPIRVPass.cpp

@@ -61,9 +61,11 @@ struct GPUToSPIRVPass final : impl::ConvertGPUToSPIRVBase<GPUToSPIRVPass> {
 
 spirv::TargetEnvAttr
 GPUToSPIRVPass::lookupTargetEnvInTargets(gpu::GPUModuleOp moduleOp) {
-  for (const Attribute &targetAttr : moduleOp.getTargetsAttr())
-    if (auto spirvTargetEnvAttr = dyn_cast<spirv::TargetEnvAttr>(targetAttr))
-      return spirvTargetEnvAttr;
+  if (const ArrayAttr &targets = moduleOp.getTargetsAttr()) {
+    for (const Attribute &targetAttr : targets)
+      if (auto spirvTargetEnvAttr = dyn_cast<spirv::TargetEnvAttr>(targetAttr))
+        return spirvTargetEnvAttr;
+  }
 
   return {};
 }