[flang][cuda][NFC] Simplify IsCUDAShared (#157931)

clementval · web-flow · commit f0b80699ccb9 · 2025-09-10T19:19:55.000Z
diff --git a/flang/include/flang/Semantics/tools.h b/flang/include/flang/Semantics/tools.h
@@ -235,10 +235,8 @@ inline bool IsCUDADevice(const Symbol &sym) {
 
 inline bool IsCUDAShared(const Symbol &sym) {
   if (const auto *details{sym.GetUltimate().detailsIf<ObjectEntityDetails>()}) {
-    if (details->cudaDataAttr() &&
-        *details->cudaDataAttr() == common::CUDADataAttr::Shared) {
-      return true;
-    }
+    return details->cudaDataAttr() &&
+        *details->cudaDataAttr() == common::CUDADataAttr::Shared;
   }
   return false;
 }

Original file line number	Diff line number	Diff line change
`@@ -235,10 +235,8 @@ inline bool IsCUDADevice(const Symbol &sym) {`
`235`	`235`
`236`	`236`	`inline bool IsCUDAShared(const Symbol &sym) {`
`237`	`237`	`if (const auto *details{sym.GetUltimate().detailsIf<ObjectEntityDetails>()}) {`
`238`		`- if (details->cudaDataAttr() &&`
`239`		`- *details->cudaDataAttr() == common::CUDADataAttr::Shared) {`
`240`		`- return true;`
`241`		`- }`
	`238`	`+ return details->cudaDataAttr() &&`
	`239`	`+ *details->cudaDataAttr() == common::CUDADataAttr::Shared;`
`242`	`240`	`}`
`243`	`241`	`return false;`
`244`	`242`	`}`